-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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 VMs to F40, F39, D13 #22549
Conversation
@Luap99 PTAL when you get a moment. I'm seeing a bunch of pasta related "permission denied" failures on the debian integration tests (annotated log). Ed's script reports this should be passt version Edit: Nevermind, Paul responded in the related c/automation_images PR:
|
@Luap99 I noticed this is the same version of pasta in the new F40 CI VM image and no tasks failed there. I'm a bit uneasy thinking about it, but how would you feel if I:
|
Look like it is still not working... Not sure if the profile is wrong or if there is some special magic needed after changes |
Let me confirm the basics first, like is the file there, does it have correct permissions, etc. @edsantiago I'm seeing these weird Edit: in the Edit 2: Found #21373 Edit 3: Un-ping @edsantiago I found the source of the trouble and staged a fix in containers/automation_images#349 |
Signed-off-by: Chris Evich <[email protected]>
e581739
to
0810203
Compare
Keeping this a draft since I don't know if "Temporarily disable rootless debian e2e testing" will work. |
Signed-off-by: Chris Evich <[email protected]>
@mheon Question when you get a chance. With this PR, CI is setup to test F39 with BoltDB. Is this even supported and/or should we continue? Reminder: We have CI running on release-branches daily. There's also the upgrade test w/ the comment: # 2024-02: as long as possible/reasonable, try to keep
# one version < 4.8 so we can test boltdb. v4.3.1 is
# the lowest we can go right now, builds before that
# have netavark <1.4 which hangs on f39 kernel (#21863).
PODMAN_UPGRADE_FROM: v4.3.1 The other reason I ask is largely cosmetic and to simplify our scripts: Almost every Cirrus task name includes 'boltdb' or 'sqlite' at the end. Making them eyeball unfriendly. ISTM we could maybe get rid of that, and the |
Yes, it's still sensible. We've only just disabled the creation of BoltDB databases in F40 (F39 never gets 5.0, so it can still use BoltDB without restrictions, even if it's not the default). We'll want to keep it around on F40 for folks upgrading, as well. Probably obligated to keep it around until next year, when we can consider what to do about the old DB code. |
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.
LGTM
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cevich, Luap99 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 |
# FIXME: Fails on Debian rootless ref: #22625 | ||
# matrix: *platform_axis | ||
# FIXME: Fails on Debian rootless ref: #22625 |
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.
WTF????? Is this the pasta thing? Do we have a timeline for resolving this?
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.
I mean, IIRC the bug had something to do with the move to /var/tmp
, did anyone consider setting an explicit TMPDIR=/tmp
in the debian CI?
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.
Yes my tmpfs PR will fix it by using /tmp/... Otherwise I need to dig into apparmor
Pasta is completely broken on debian. I see no choice but to hold my nose and merge this. Good luck to everyone working on fixes. /lgtm |
Ya, it was totally FUBAR. Paul's got a handle on it 😄 |
Ref: containers/automation_images#349
Does this PR introduce a user-facing change?