-
Notifications
You must be signed in to change notification settings - Fork 2.7k
bump image sfx #26120
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
bump image sfx #26120
Conversation
Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: timcoding1988 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 |
Note DCO check is failing because your git config has the wrong name/author set. |
Restarted three failing tests, machine is always a bit flaky but two new failures I have never seen before, commands time out. We need to keep a close look on these ones: https://api.cirrus-ci.com/v1/artifact/task/5530264677384192/html/int-podman-fedora-41-rootless-host-boltdb.log.html#t--Podman-kube-play-test-with-reserved-CIDFile-annotation-in-yaml--1 |
c00a807
to
21dfaad
Compare
ba22af1
to
d0e85fd
Compare
machine linux test on aarch64 keep timing out, that is not a good sign. Something in the updates packages seem to have causes a slow down. We were already close to 30 mins, i.e. around 26 mins but I rerun more than three times now so it is pretty consistent. @timcoding1988 Can you add this to increase the timeout for the task. diff --git a/.cirrus.yml b/.cirrus.yml
index 828b395c8c..ec47abe79b 100644
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -796,7 +796,7 @@ podman_machine_aarch64_task:
depends_on: *build
ec2_instance:
<<: *standard_build_ec2_aarch64
- timeout_in: 30m
+ timeout_in: 40m
env:
TEST_FLAVOR: "machine-linux"
EC2_INST_TYPE: c6g.metal Hopefully it is just a bit above 30mins and not a hang or something. |
6ceaf22
to
8ff0804
Compare
The Fedora-41 root host boltdb failed with the below. It looks like it hung for 2 minutes trying to send a TERM signal. As everything else passed, including the test that had been problematic, I'm going to respin it.
|
8ff0804
to
ec9011b
Compare
ec9011b
to
5edf3e7
Compare
Signed-off-by: Tim Zhou <[email protected]>
5edf3e7
to
6b3beaf
Compare
[NON-BLOCKING] Packit jobs failed. @containers/packit-build please check. Everyone else, feel free to ignore. |
When a custom username is requested we must stil enable lingering for it. The linger setup in the coreos image with the hard coded core user doesn't really make sense as it doesn't work when another name is used. Signed-off-by: Paul Holzinger <[email protected]>
There is no reason to deifne a dummy service to run sleep just to get the podman.socket enabled. Enabeling a service is just creating the symlink so we can do that for the user sessions as well. Signed-off-by: Paul Holzinger <[email protected]>
This patch adds a new endpoint to the REST API called "artifacts" with the following methods: - Add - Extract - Inspect - List - Pull - Push - Remove This API will be utilised by the Podman bindings to add OCI Artifact support to our remote clients. Jira: https://issues.redhat.com/browse/RUN-2711 Signed-off-by: Lewis Roy <[email protected]>
I noticed this was missing, its a simple 1 to 1 mapping to --interface-name. Signed-off-by: Paul Holzinger <[email protected]>
podman system check --quick currently only skips layer contents, but practically it's not much quicker than without the flag. This changes the flag to also skip checking layer digests which speed up the check significantly. In some cases, it is useful to opt for a quicker check if we prioritize detecting and fixing severe corruption and can tolerate minor damage. The check option is derived from CRI-O's internal repair: https://github.com/cri-o/cri-o/blob/9e4d86d82370ad44c14649f28ebdd8f94aaa28ca/internal/lib/container_server.go#L860 Signed-off-by: Sonny Sasaka <[email protected]>
Signed-off-by: Ashley Cui <[email protected]>
Users of WSL images currently get podman from the copr rpm on the release PR with Epoch: 102. This is a problem if the user is looking to update the image with packages from official Fedora repos. This commit limits Epoch: 102 to only the podman rpms on rhcontainerbot/podman-next copr. All other rpms, including other copr rpms, will use the default Epoch. Signed-off-by: Lokesh Mandvekar <[email protected]>
We disabled it in the machine os image so it can no longer be used currently. Signed-off-by: Paul Holzinger <[email protected]>
Do not write /etc/environment.d files, something is broken in the new image that causes the boot to fail when any basic var is set there. Signed-off-by: Paul Holzinger <[email protected]>
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
It has not been in use since commit f218f84 and should have been removed there. It seems somehow it is causing a bug since our env file is empty. In that case it triggers a segfault and since that happens from within pam we are unable to login in any way. I reported the issue[1] but because we don't need this just remove it so we don't have to wait for a fix. [1] https://bugzilla.redhat.com/show_bug.cgi?id=2370858 Signed-off-by: Paul Holzinger <[email protected]>
This reverts commit 0b8dd90. Signed-off-by: Paul Holzinger <[email protected]>
This is not used in the code so it can be deleted. Signed-off-by: Paul Holzinger <[email protected]>
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Two incomptable changes, they removed the BridgeNfIP6tables and BridgeNfIptables fields so we must drop them. As they are not important ones that should not cause problems. Second, they moved to using DockerOCIImageConfig from another new module. The json format did not chnage so this is not an external API break. Signed-off-by: Paul Holzinger <[email protected]>
Signed-off-by: Paul Holzinger <[email protected]>
Includes one breaking change for the flag as BuildOutputs now accept a slice. Signed-off-by: Paul Holzinger <[email protected]>
Newer versions expect to use a new test helper. Signed-off-by: Nalin Dahyabhai <[email protected]> Signed-off-by: Paul Holzinger <[email protected]>
The buildah test code chnages so I had to manually resolve the conflict. Signed-off-by: Paul Holzinger <[email protected]>
The --output option is not supported with remote. Signed-off-by: Paul Holzinger <[email protected]>
Signed-off-by: Tim Zhou <[email protected]>
91ea24a
to
95b18b2
Compare
you need to rebase against main and not merge main into your branch to avoid the history clutter |
Does this PR introduce a user-facing change?