-
Notifications
You must be signed in to change notification settings - Fork 16
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
ci: add Fedora 37 #111
ci: add Fedora 37 #111
Conversation
|
Ah, sure, we need newer versions of osbuild and composer. The old ones don't have F37 RPMs. Fixed. |
Do we actually need to keep testing on F36? 🤔 |
@thozza We release into both F36 and F37, so we probably should. Does it make sense? |
fair... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but IMO it would make sense to update the PR to use the latest osbuild and composer releases...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks ❤️
@ondrejbudai integration test is failing on el8.6, specifically the upload to AWS. There are no useful logs, no journal. 😢 |
CGImport seems to be slow in certain cases. Let's increase the timeout, otherwise CGImport for the 10 GiB RAW AMI might exceed it. Fun fact: Fedora's Koji has the timeout configured to 10800 seconds. 🤯
This commit needs to do three things: - update the terraform definition - add fedora-37 to the gitlab-ci file - bump osbuild and osbuild-composer versions, so we have F37 RPMs for them I also removed the commented out aarch64 runners to clean the file a bit.
I fixed it by increasing the apache timeout in the kojihub contianer. Apparently, we were hitting it while CGImporting the big raw image for AWS. I'm pretty sure that this is happening in the CI for quite some time, but it was masked due to infamous osbuild/osbuild-composer@972da81 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your effort!
This commit needs to do two things:
I also removed the commented out aarch64 runners to clean the file a bit.