Merged
Conversation
Check if fork remote exists before trying to update it. When the fork already exists, fedpkg fork doesn't create the git remote, causing git remote set-url to fail. Now we check with git remote get-url and use git remote add if the remote doesn't exist. This fixes the "No such remote" error during Pagure test execution. Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
For tests with "skip build" in the title, the build is skipped so we cannot retrigger it with "/packit-stg build". Instead, use the test comment "/packit-stg test" to trigger test runs. Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
GitLab webhook delivery can be delayed by over an hour during high load, causing the opened PR test to fail even though the webhook eventually arrives and statuses are created. For GitLab opened PR triggers, if the initial 3-minute check fails: - Log an error about the webhook delay - Wait an additional 60 minutes for the delayed webhook - If statuses eventually appear, log the delay and continue validation - Only fail if statuses don't appear after the full extended wait Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Uses StrictHostKeyChecking=accept-new to accept the key on first connection without prompting. Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Configure Sentry to use environment labels "validation-stg" and "validation-prod" instead of generic labels. The environment is derived from the DEPLOYMENT configuration, so errors are automatically tagged with the correct environment. Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Increase the delay between validation tests from 1 minute to 3 minutes for both GitHub and GitLab to avoid overloading packit-service instances. Otherwise all 4 short running workers and 2 long running workers are overwhelmed by tasks. Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
mfocko
approved these changes
Apr 14, 2026
| # Configure SSH to not prompt for host key verification | ||
| RUN mkdir -p /root/.ssh && \ | ||
| echo "Host pkgs.fedoraproject.org" >> /root/.ssh/config && \ | ||
| echo " StrictHostKeyChecking accept-new" >> /root/.ssh/config && \ |
Member
There was a problem hiding this comment.
We could probably pregenerate the KnownHosts, but I don't think it's worth it here.
| from validation.deployment import DEPLOYMENT | ||
|
|
||
| # Set environment based on deployment to distinguish in Sentry | ||
| environment = f"validation-{DEPLOYMENT.name}" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
More fixes for the validation tests.