Skip to content

[release-1.41] fix call to chown#6763

Merged
lsm5 merged 2 commits intocontainers:release-1.41from
nalind:fix-chown-1.41
Apr 16, 2026
Merged

[release-1.41] fix call to chown#6763
lsm5 merged 2 commits intocontainers:release-1.41from
nalind:fix-chown-1.41

Conversation

@nalind
Copy link
Copy Markdown
Member

@nalind nalind commented Apr 2, 2026

What type of PR is this?

/kind bug

What this PR does / why we need it:

Cherry picks #6683

How to verify it

Which issue(s) this PR fixes:

Originally fixed #6682.

Special notes for your reviewer:

Does this PR introduce a user-facing change?

None

stilwelb added 2 commits April 2, 2026 11:21
Signed-off-by: Brad Stilwell <stilwelb@us.ibm.com>
Signed-off-by: Brad Stilwell <stilwelb@us.ibm.com>
@nalind nalind added the No New Tests Allow PR to proceed without adding regression tests label Apr 2, 2026
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 2, 2026
@TomSweeneyRedHat
Copy link
Copy Markdown
Member

LGTM
We really need to get Head and Shoulders into the CI pipeline to cure the flakes. Test restarted.

@nalind
Copy link
Copy Markdown
Member Author

nalind commented Apr 3, 2026

These testing farm runs are running tests from the main branch against code in this branch:
https://artifacts.dev.testing-farm.io/9e0a7fd0-1d32-4d5c-a6f1-02507489e52c/
https://artifacts.dev.testing-farm.io/1b2dc115-2fbb-470f-bbe6-5f78a164f65a/
https://artifacts.dev.testing-farm.io/12a4d1b8-8622-4f20-a94c-235040563c1d/
https://artifacts.dev.testing-farm.io/6cd3b68b-2061-4022-8a99-72b42ab1bfc2/
Those tests expect functionality and test helpers that aren't here. I don't think they could ever pass.

@lsm5
Copy link
Copy Markdown
Member

lsm5 commented Apr 16, 2026

These testing farm runs are running tests from the main branch against code in this branch: https://artifacts.dev.testing-farm.io/9e0a7fd0-1d32-4d5c-a6f1-02507489e52c/ https://artifacts.dev.testing-farm.io/1b2dc115-2fbb-470f-bbe6-5f78a164f65a/ https://artifacts.dev.testing-farm.io/12a4d1b8-8622-4f20-a94c-235040563c1d/ https://artifacts.dev.testing-farm.io/6cd3b68b-2061-4022-8a99-72b42ab1bfc2/ Those tests expect functionality and test helpers that aren't here. I don't think they could ever pass.

I can update the testing-farm config to not use main branch.

Copy link
Copy Markdown
Member

@lsm5 lsm5 left a comment

Choose a reason for hiding this comment

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

LGTM

@lsm5 lsm5 merged commit f7322b7 into containers:release-1.41 Apr 16, 2026
35 of 39 checks passed
@nalind nalind deleted the fix-chown-1.41 branch April 16, 2026 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No New Tests Allow PR to proceed without adding regression tests size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants