Skip to content

Conversation

Luap99
Copy link
Member

@Luap99 Luap99 commented Jul 22, 2025

Images from containers/automation_images#411

Does this PR introduce a user-facing change?

None

Copy link
Contributor

openshift-ci bot commented Jul 22, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 22, 2025
@Luap99
Copy link
Member Author

Luap99 commented Jul 22, 2025

Uh, subuid/gids files seems to no longer be populated on rawhide...

Images from containers/automation_images#411

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
@Luap99 Luap99 changed the title update CI images to 2025-07-21 update CI images to 2025-07-29 Jul 29, 2025
@Luap99 Luap99 marked this pull request as draft July 29, 2025 10:37
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 29, 2025
Copy link

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

@Luap99
Copy link
Member Author

Luap99 commented Jul 29, 2025

@giuseppe cgroup weight tests are failing here due containers/crun#1767 I think. I assume we just need to update the test here?

@giuseppe
Copy link
Member

@Luap99 yes, we need to update the tests. Could we make it accept both values for the time being so it easier to bisect if needed?

@Luap99
Copy link
Member Author

Luap99 commented Jul 29, 2025

@Luap99 yes, we need to update the tests. Could we make it accept both values for the time being so it easier to bisect if needed?

yes looks like I would need to do it anyway as the update is not yet on f41 as the test passed there.

But regardless these images are currently unusable for podman CI to the useradd subuid thing, I guess I could add the work around to add our own ranges to get this image in but I was hoping this stuff would have been reverted by now...

@mtrmac
Copy link
Collaborator

mtrmac commented Jul 29, 2025

But regardless these images are currently unusable for podman CI to the useradd subuid thing

We are not in a rush for Sequoia at all, let’s wait for a few days.

@Luap99
Copy link
Member Author

Luap99 commented Jul 29, 2025

@mtrmac yeah I know, it is not about rushing.

I like to test each new image version so we become aware of all the new failures as early as possible. Just like I happen to find the useradd bug here last week. And this time around we have new cgroup changes. In case external stuff slipped in we can report it early and thus get it fixed earlier.
I don't care about getting this merged ASAP in general.

@Luap99 Luap99 closed this Aug 20, 2025
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. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. release-note-none
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants