Skip to content

[release-1.42] chroot.bats(chroot with overlay root): ensure we can overlay#6666

Merged
Luap99 merged 1 commit intocontainers:release-1.42from
nalind:test-overlay-over-overlay-1.42
Feb 5, 2026
Merged

[release-1.42] chroot.bats(chroot with overlay root): ensure we can overlay#6666
Luap99 merged 1 commit intocontainers:release-1.42from
nalind:test-overlay-over-overlay-1.42

Conversation

@nalind
Copy link
Member

@nalind nalind commented Feb 5, 2026

What type of PR is this?

/kind failing-test

What this PR does / why we need it:

When setting up a chroot using overlay, if the intended upper directory is already on an overlayfs, mount a tmpfs onto it so that we can finish setting up the chroot that's an overlayfs that we'll actually test in, and copy the base image into the storage that it'll use.

How to verify it

Should continue passing here, and in the environment which prompted #6629.

Which issue(s) this PR fixes:

Special notes for your reviewer:

Cherry-picked from #6636.

Does this PR introduce a user-facing change?

None

When setting up a chroot using overlay, if the intended upper directory
is already on an overlayfs, mount a tmpfs onto it so that we can finish
setting up the chroot that's an overlayfs that we'll actually test in,
and copy the base image into the storage that it'll use.

Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Feb 5, 2026
@nalind nalind changed the title [release-1.42] integration test updates [release-1.42] chroot.bats(chroot with overlay root): ensure we can overlay Feb 5, 2026
Copy link
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Luap99 Luap99 merged commit ea8b091 into containers:release-1.42 Feb 5, 2026
31 of 37 checks passed
@nalind nalind deleted the test-overlay-over-overlay-1.42 branch February 5, 2026 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants