-
Notifications
You must be signed in to change notification settings - Fork 760
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
chore(deps): update dependency containers/automation_images to v20240513 - autoclosed #5526
Conversation
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: renovate[bot] The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Ephemeral COPR build failed. @containers/packit-build please check. |
I'm having a very hard time grokking those test logs, but this:
...makes me suspect that we can't use tmpfs-/tmp on buildah: too diskhoggy. I've run into the same problem with 1minutetip. Only solution I can think of is to hardcode |
Plug for my podman-ginkgo-highlight GreaseMonkey script, which colorizes the PASS/FAIL lines. Still very human-unfriendly, but better. |
Didn't we already run on tmpfs (on fedora) before my changes? AFAICT you tested the tmpfs thing with #5470? |
Conformance tests run on Debian. Looking more closely, I see:
As it turns out, there's some really weird clunky code that downloads -5:26.0.0-1~debian.12~bookworm
+5:26.1.2-1~debian.12~bookworm (basically, 26.0 to 26.1) Conformance tests, IIRC, are very docker-dependent and break every time docker changes. WHEW. After all that, I think the TL;DR is
|
Ah that is a mess, would be great to get the distro name into the task names. Because if there is none I am going to assume it uses fedora stable. setting TMPDIR=/var/tmp sounds like a reasonable work around, actual test fixes for docker changes would likely be a @nalind task. |
Agreed, actual fix needs @nalind attention. Here's (what I believe is) the relevant error output, to save on click-and-hunt time:
|
I agree with Paul's suggestion it would be nice to have "Debian" in the task name. I can toss a quick PR to do this.
For some more context here, in case it helps: IIRC the conformance tests use to run on Fedora and Ubuntu, until it became impossible/hard to install docker on Fedora. We also use to install the "latest" Docker at runtime, but this wrecked havoc with testing on release branches and flake all the time. CI VM build was updated to cache the packages for runtime install well before the switch to Debian. Much duct-tape has been piled on since then no doubt. Point is, these machinations are all very very old. I would not at all be opposed to improvements, so long as any changes are likely to survive long-term release-branch testing. |
@@ -32,7 +32,7 @@ env: | |||
DEBIAN_NAME: "debian-13" |
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.
Also need to update:
FEDORA_NAME: "fedora-40"
PRIOR_FEDORA_NAME: "fedora-39"
DEBIAN_NAME: "debian-13"
Per consent in 2024-03-20 Planning, remove support for cgroups v1. And, per discovery on 2024-05-28, systemd 256 no longer even boots on a cgroups v1 system. This commit switches to VMs built in: containers/automation_images#338 ...in which Debian is now cgroups v2 with crun. Requires disabling two conformance tests which fail when run against docker 26.1; see containers#5526 for context on those. And, latest git on f40 refuses to serve non-root files when run as root ("dubious ownership"); so, in start_git_daemon(), chown the extracted files. Signed-off-by: Ed Santiago <[email protected]>
Per consent in 2024-03-20 Planning, remove support for cgroups v1. And, per discovery on 2024-05-28, systemd 256 no longer even boots on a cgroups v1 system. This commit switches to VMs built in: containers/automation_images#338 ...in which Debian is now cgroups v2 with crun. Requires disabling two conformance tests which fail when run against docker 26.1; see containers#5526 for context on those. Also requires disabling two bats tests on debian because something changed there in ulimits. I'll look into them later but right now this gives us breathing space. And, latest git on f40 refuses to serve non-root files when run as root ("dubious ownership"); so, in start_git_daemon(), chown the extracted files. Signed-off-by: Ed Santiago <[email protected]>
Per consent in 2024-03-20 Planning, remove support for cgroups v1. And, per discovery on 2024-05-28, systemd 256 no longer even boots on a cgroups v1 system. This commit switches to VMs built in: containers/automation_images#338 ...in which Debian is now cgroups v2 with crun. Requires disabling two conformance tests which fail when run against docker 26.1; see containers#5526 for context on those. Also requires disabling two bats tests on debian because something changed there in ulimits. I'll look into them later but right now this gives us breathing space. And, latest git on f40 refuses to serve non-root files when run as root ("dubious ownership"); so, in start_git_daemon(), chown the extracted files. Signed-off-by: Ed Santiago <[email protected]>
This PR contains the following updates:
20240411t124913z-f39f38d13
->20240513t140131z-f40f39d13
Release Notes
containers/automation_images (containers/automation_images)
v20240513t140131z-f40f39d13
Compare Source
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Never, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.