Skip to content

Use current process ID to investigate proc filesystem to workaround in-container non-root owned symlinks #2990

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

Merged
merged 1 commit into from
Mar 24, 2025

Conversation

craigcomstock
Copy link
Contributor

@craigcomstock craigcomstock commented Feb 28, 2025

Ticket: CFE-3429
Changelog: title
Signed-off-by: Craig Comstock [email protected]

@cf-bottom
Copy link

Thanks for submitting a PR! Maybe @nickanderson can review this?

@craigcomstock craigcomstock marked this pull request as draft March 3, 2025 13:56
Copy link

@cf-bottom cf-bottom left a comment

Choose a reason for hiding this comment

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

Please use a company e-mail instead of c***@unreasonablefarm.org

@craigcomstock
Copy link
Contributor Author

craigcomstock commented Mar 21, 2025

The associated core PR has been merged. The exotic failures that occurred with that PR are flakes and so don't block merging this PR.

@craigcomstock craigcomstock requested a review from cf-bottom March 21, 2025 19:04
…n-container non-root owned symlinks

Ticket: CFE-3429
Changelog: title
Signed-off-by: Craig Comstock <[email protected]>
@craigcomstock craigcomstock merged commit d38e3a3 into cfengine:master Mar 24, 2025
4 checks passed
@craigcomstock craigcomstock deleted the CFE-3429/master branch March 24, 2025 12:48
@craigcomstock craigcomstock added the cherry-pick? Fixes which may need to be cherry-picked to LTS branches label Mar 24, 2025
@craigcomstock
Copy link
Contributor Author

only need to cherry pick back to 3.24.x: #2998

@craigcomstock craigcomstock removed the cherry-pick? Fixes which may need to be cherry-picked to LTS branches label Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants