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.

Sign up for GitHub

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

Jump to bottom

Helm: Skip recursive permission change #1744

Merged
merged 1 commit into from
Aug 29, 2023
Merged

Helm: Skip recursive permission change #1744

merged 1 commit into from
Aug 29, 2023

Conversation

cniackz
Copy link
Contributor

@cniackz cniackz commented Aug 29, 2023 •
edited
Loading

Objective:

To skip recursive permission change.

Reasoning:

When having large volumes with lots of small files, it can be very expensive the recursive permission change. And it can be skipped by setting fsGroupChangePolicy to "OnRootMismatch"

Documentation:

@cniackz cniackz requested a review from allanrogerr August 29, 2023 22:12
@cniackz cniackz self-assigned this Aug 29, 2023
@cniackz cniackz added the enhancement New feature or request label Aug 29, 2023
@cniackz cniackz changed the title Skip recursive permission change Helm: Skip recursive permission change Aug 29, 2023
@harshavardhana harshavardhana merged commit daa32d3 into minio:master Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pjuarezd pjuarezd pjuarezd approved these changes

@harshavardhana harshavardhana harshavardhana approved these changes

@allanrogerr allanrogerr Awaiting requested review from allanrogerr

@dvaldivia dvaldivia Awaiting requested review from dvaldivia

Assignees

@cniackz cniackz

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cniackz @harshavardhana @pjuarezd