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

Run gitlab CI build jobs on Fedora 41 #1058

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

achilleas-k
Copy link
Member

@achilleas-k achilleas-k commented Nov 22, 2024

Schutzfile: switch Fedora 41 repos to released


Schutzfile: define common gitlab-ci-runner


imgtestlib: read common CI runner from Schutzfile

Use the common CI runner defined in the Schutzfile in all gitlab CI
generators.


gitlab: bump gitlab CI runners to Fedora 41


schutzbot: update terraform commit ID


thozza
thozza previously approved these changes Nov 22, 2024
Copy link
Member

@thozza thozza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing the 🧹

I've added one idea, but don't consider it a blocker.

test/scripts/imgtestlib.py Outdated Show resolved Hide resolved
@achilleas-k
Copy link
Member Author

Added the CI runner definition to the Schutzfile and updated the terraform SHA to the newest which includes Fedora 41 runner definitions.

Copy link
Member

@thozza thozza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing! 😍

@achilleas-k
Copy link
Member Author

achilleas-k commented Nov 26, 2024

A lot of jobs timed out in GitLab. I'm hoping it's because this PR was running at the same time as #1066, both of which caused a rebuild of every test image (this one because the runner changed and the other because the osbuild commit ID changed).

I hit the rebase button here now, which means a full rebuild again, but it also means we wont need one on the merge queue.

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