Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CI to Fedora 41, disable updates by default #3464

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

pmatilai
Copy link
Member

We basically always want to run the latest stable release, but the newer the release the more update churn there is, and that sort of instability is bad for CI. To counter that, disable updates by default. If and when we need specific updates to components, we now do it explicitly. This seems like the best of both worlds really.

Technically this is two different changes, one to bump the Fedora version and another to disable updates, but if they are separated then time goes backwards for that one commit so I think it's better to just have this as one commit.

We basically always want to run the latest stable release, but the newer
the release the more update churn there is, and that sort of instability
is bad for CI. To counter that, disable updates by default. If and when
we need specific updates to components, we now do it explicitly. This
seems like the best of both worlds really.

Technically this is two different changes, one to bump the Fedora
version and another to disable updates, but if they are separated then
time goes backwards for that one commit so I think it's better to just
have this as one commit.
@pmatilai pmatilai requested a review from a team as a code owner November 22, 2024 08:43
@pmatilai pmatilai requested review from ffesti and removed request for a team November 22, 2024 08:43
@dmnks
Copy link
Contributor

dmnks commented Nov 22, 2024

Indeed, and oh, good point about disabling the "updates" repo as well (something that's easy to forget 😄)

@dmnks
Copy link
Contributor

dmnks commented Nov 22, 2024

This also gives us better reproducibility of test failures in CI, hopefully, since we can now compare (or even pull) the image ID locally.

@dmnks dmnks merged commit 5612c77 into rpm-software-management:master Nov 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants