Skip to content

Conversation

Luap99
Copy link
Member

@Luap99 Luap99 commented Sep 9, 2025

containers/automation_images#416

Does this PR introduce a user-facing change?

None

@openshift-ci openshift-ci bot added release-note-none approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Sep 9, 2025
@Luap99
Copy link
Member Author

Luap99 commented Sep 9, 2025

debian images are broken with some for of keryring leak. I have seen this in fedora openQA before and it was tied to the systemd update there https://bodhi.fedoraproject.org/updates/FEDORA-2025-a0ce059969 which then was traced bug to a kernel bug AFAICT systemd/systemd#38724

@Luap99 Luap99 changed the title New VM images 2025-09-09 New VM images 2025-09-10 Sep 10, 2025
Copy link

[NON-BLOCKING] Packit jobs failed. @containers/packit-build please check. Everyone else, feel free to ignore.

The full error message is:
failed to create idmapped mount: mount_setattr /tmp/CI_SKPI/podman_bats.RX6gD9/rootfs: invalid argument

So in order to match this account for the variable part.

Signed-off-by: Paul Holzinger <[email protected]>
On debian with an older we get this error instead:
fsconfig() failed: tmpfs: Unknown parameter 'noswap'

So handle that case as well to skip the test correctly.

Signed-off-by: Paul Holzinger <[email protected]>
The ls call for the directory now fails with EACCES, see
https://github.com/containers/crun/issues/1876

Signed-off-by: Paul Holzinger <[email protected]>
@Luap99
Copy link
Member Author

Luap99 commented Sep 10, 2025

Note the debian images now use a 6.1 kernel instead to work around the memory leak on the 6.16 kernel that we hit otherwise. Thus I had to fix the skips for older kernels as they were not working properly.

@Luap99
Copy link
Member Author

Luap99 commented Sep 10, 2025

This also bring in go 1.24 so we can bump the repo here #27017

@Luap99
Copy link
Member Author

Luap99 commented Sep 10, 2025

@timcoding1988 @containers/podman-maintainers PTAL

@baude
Copy link
Member

baude commented Sep 10, 2025

yeah LGTM

@timcoding1988
Copy link
Collaborator

LGTM

Copy link
Contributor

openshift-ci bot commented Sep 10, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Luap99, timcoding1988

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@baude
Copy link
Member

baude commented Sep 10, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 10, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 906f87b into containers:main Sep 10, 2025
81 of 89 checks passed
@Luap99 Luap99 deleted the new-images branch September 10, 2025 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release-note-none
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants