Skip to content
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

[release-1.23] Set permissions on volume before publishing update #3

Closed

Conversation

furkatgofurov7
Copy link
Member

@furkatgofurov7 furkatgofurov7 commented Dec 28, 2022

This change fixes a race condition in release-1.23 branch that was caused by setting the file owner, group and mode non-atomically, after the updated files had been published.

Users who were running non-root containers, without GID 0 permissions, and had removed read permissions from other users by setting defaultMode: 0440 or similar, were getting intermittent permission denied errors when accessing files on secret or configmap volumes or service account tokens on projected volumes during update.

@furkatgofurov7 furkatgofurov7 changed the title This change fixes a race condition in release-1.23 branch [release-1.23] Set permissions on volume before publishing update Dec 28, 2022
that was caused by setting the file owner, group and mode non-atomically,
after the updated files had been published.

Users who were running non-root containers, without GID 0 permissions, and
had removed read permissions from other users by setting defaultMode: 0440 or
similar, were getting intermittent permission denied errors when accessing
files on secret or configmap volumes or service account tokens on projected
volumes during update.
@furkatgofurov7 furkatgofurov7 force-pushed the backport-of-#114464-upstream-release-1.23 branch from b1e9912 to 31fdd27 Compare December 28, 2022 12:36
@Rozzii
Copy link

Rozzii commented Mar 14, 2023

not needed anymore

@Rozzii Rozzii closed this Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants