images: Adjust centos-9-bootc for cockpit testing #7365
Merged
+7
−1
Conversation
This file contains 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
cockpituous
changed the title
9 tasks
|
image-refresh centos-9-bootc done: https://github.com/cockpit-project/bots/commits/image-refresh-centos-9-bootc-20250129-054713 |
cockpituous
changed the title
martinpitt
force-pushed
the
c9bootc-cockpit-deps
branch
from
12b7588 to
29d7495
Compare
cockpituous
changed the title
cockpituous
changed the title
cockpituous
changed the title
cockpituous
changed the title
martinpitt
force-pushed
the
c9bootc-cockpit-deps
branch
from
29d7495 to
e928515
Compare
|
OK, let's try without updating the kernel. |
cockpituous
changed the title
|
image-refresh centos-9-bootc done: https://github.com/cockpit-project/bots/commits/image-refresh-centos-9-bootc-20250129-074751 |
cockpituous
changed the title
martinpitt
force-pushed
the
c9bootc-cockpit-deps
branch
from
357491f to
3dc43fb
Compare
config.json cannot do that. Some tests expect this, so let's keep our images uniform.
This gives us the latest packages, so that the image stays compatible with the generated runtime dependencies from the rpm build (such as selinux-policy). However, exclude the kernel as that current causes kernel oopses.
This will allow us to install cockpit-ws.rpm and run networking and metrics tests. Also toss in strace as the ultimate debugging tool, and rsync for `build.js -r` and a more efficient bots API. See cockpit-project/cockpit#21561
martinpitt
force-pushed
the
c9bootc-cockpit-deps
branch
from
3dc43fb to
a434ce4
Compare
jelly
approved these changes
Jan 30, 2025
| RUN \ | ||
| dnf update -y --exclude='kernel*' && \ |
There was a problem hiding this comment.
Is that a known issue, sounds like it could use a # HACK unless we never want to support it?
There was a problem hiding this comment.
It's at least not necessary for us to support. I had to add it because the original container has a too old selinux-policy -- the rpm build in the mock of centos-9-stream depended on a newer version. I haven't reported the kernel failure yet, but added to my TODO list.
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
See cockpit-project/cockpit#21559