Repository navigation
fix(kubernetes): preserve initialized volume permissions (task 33.09.06) - #33
Merged
NaveCohenMonday merged 1 commit intoOct 14, 2026
Merged
Conversation
Signed-off-by: Nave Cohen <nave@monday.com>
2 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Periodic CSI republish reapplies the default recursive fsGroup ownership change, replacing groups assigned by a running sandbox. Helpers then lose access to their working directory and archived tool results. Set
OnRootMismatchin both existing Kubernetes pod group setup paths so initialized volumes retain workload-owned groups.Task-ID: 33.09.06
Related Issue
No issue required: localized bug fix to existing volume permission setup, assigned directly by Nave and recorded in Workspace task 33.09.06.
Changes
The Kubernetes compute driver owns this pod setting. Per-command permission repairs would race kubelet; giving the helper the private runtime group would break isolation. This is the smallest generic change at the owning boundary.
Testing
mise run pre-commitpasses.mise run ciattempted; stopped on 22 existing Go SDK lint findings outside this change. Rust clippy passed. No unrelated Go changes included.Checklist