Bump CI VMs: new pasta, crun, rawhide kernel #22082
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: edsantiago The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
bot
added
the
approved
|
crun is now 1.14.4 (dot-FOUR) everywhere. This should help us out on buildah; some tests are being skipped there because of a too-old crun on debian. |
|
Cockpit tests failed for commit d78866aa5e315cfc84394add7172bf258203c7a3. @martinpitt, @jelly, @mvollmer please check. |
openshift-ci
bot
added
the
do-not-merge/work-in-progress
|
Converted to draft while the pasta issue gets sorted out |
|
Cockpit tests failed for commit 11248931491a872767a79d4ca1f5e8269baa9161. @martinpitt, @jelly, @mvollmer please check. |
|
Cockpit tests failed for commit d55dbaf8ca8bd749e0b15292b20a5ad64da68a43. @martinpitt, @jelly, @mvollmer please check. |
openshift-ci
bot
removed
the
do-not-merge/work-in-progress
There was a problem hiding this comment.
@containers/podman-maintainers PTAL.
Highlights of new VMs:
- pasta 03-20 on all Fedoras (02-20 on Debian)
- crun 1.14.4 (dot FOUR) everywhere
- kernel 6.9 on rawhide
Nothing else supernoteworthy IMO, but see auto-images 337 for full list.
| @@ -38,7 +38,8 @@ var _ = Describe("Podman run device", func() { | |||
| session.WaitWithDefaultTimeout() | |||
| Expect(session).Should(ExitCleanly()) | |||
| if !isRootless() { | |||
| session = podmanTest.Podman([]string{"run", "-q", "--security-opt", "label=disable", "--device", "/dev/kmsg", "--cap-add", "SYS_ADMIN", ALPINE, "head", "-n", "1", "/dev/kmsg"}) | |||
| // Kernel 6.9.0 (2024-03) requires SYSLOG | |||
| session = podmanTest.Podman([]string{"run", "-q", "--security-opt", "label=disable", "--device", "/dev/kmsg", "--cap-add", "SYS_ADMIN,SYSLOG", ALPINE, "head", "-n", "1", "/dev/kmsg"}) | |||
There was a problem hiding this comment.
I did not test if rawhide requires both, or just the new one. Since these tests have to run everywhere, and both works, I'm leaving it this way.
openshift-ci
bot
added
the
do-not-merge/hold
|
Re-pushed with a smaller, safer fix for the Be aware, these VMs include a new Debian flake, #22165. It will probably not affect PRs, because it happens (so far) only in |
edsantiago
force-pushed
the
new-vms
branch
6 times, most recently
from
6dd6002 to
acfd099
Compare
Pasta 03-20, crun 1.14.4, and /tmp on tmpfs
One unexplained difference: fc39 and rawhide now create:
/run/log/journal/SOMETHING/system.journal
...and the SOMETHING is o-rwx. This triggers journalctl to spit out a warning:
Hint: You are currently not seeing messages from the system.
Users in groups 'adm', 'systemd-journal', 'wheel' can see all messages.
Pass -q to turn off this notice.
...which in turn causes ExitCleanly() to fail.
It is not clear who/what is creating this journal directory, or
why it allofasudden started just now. Workaround is to add -q
to journalctl in one test.
VM package info:
containers/automation_images#337 (comment)
Signed-off-by: Ed Santiago <[email protected]>
|
Closing in favor of #22202 |
stale-locking-app
bot
added
the
locked - please file new issue/PR
Pasta 03-20, crun 1.14.4
One unexplained difference: fc39 and rawhide now create:
...and the SOMETHING is o-rwx. This triggers journalctl to spit out a warning:
...which in turn causes ExitCleanly() to fail.
It is not clear who/what is creating this journal directory, or
why it allofasudden started just now.
VM package info: containers/automation_images#337 (comment)
Signed-off-by: Ed Santiago [email protected]