Conversation
Due to https://bugzilla.redhat.com/show_bug.cgi?id=2429501 This reverts the prior change to make the test non gating because the problem is we'd consistently fail to do a bootc install *for each test* which dramatically slowed down that job. We could fix that but it's easier to just disable the job. Signed-off-by: Colin Walters <walters@verbum.org>
There was a problem hiding this comment.
Code Review
This pull request disables the failing fedora-rawhide tests by commenting them out in the .packit.yaml configuration. This is a reasonable temporary solution to prevent CI from slowing down due to consistent failures. My review includes a suggestion to add a TODO comment to make it clear that this is a temporary change and to facilitate tracking its resolution. As a minor point, you might consider updating the pull request title from 'fedora 44' to 'fedora-rawhide' for better clarity and to accurately reflect the changes.
| - fedora-43-aarch64 | ||
| - fedora-rawhide-x86_64 | ||
| - fedora-rawhide-aarch64 | ||
| # https://bugzilla.redhat.com/show_bug.cgi?id=2429501 |
There was a problem hiding this comment.
For better tracking and to ensure these tests are re-enabled in the future, consider adding a TODO comment that explicitly states the condition for re-enabling them. This makes it easier for other developers to understand the context and follow up.
# TODO: Re-enable fedora-rawhide tests once https://bugzilla.redhat.com/show_bug.cgi?id=2429501 is fixed.
Due to https://bugzilla.redhat.com/show_bug.cgi?id=2429501
This reverts the prior change to make the test non gating because the problem is we'd consistently fail to do a bootc install for each test which dramatically slowed down that job.
We could fix that but it's easier to just disable the job.