Skip to content

Conversation

@bobfuru
Copy link
Contributor

@bobfuru bobfuru commented Aug 13, 2020

BZ1850510 - This is an update to #24276 and is in response to a new comment by the requestor that root_squash should be all_squash (https://bugzilla.redhat.com/show_bug.cgi?id=1850510#c19).

@huffmanca - PTAL. I'm not sure how I would address Alex's other option besides all_squash:

or the image_registry need to be adjusted in some way with a group which is dynamic in a sense that on a container restart/recreation will be modified.
Do you think changing to all_squash is a sufficient recommendation in this NFS note?

@qinpingli - PTAL for QE.

@bobfuru bobfuru added this to the Next Release milestone Aug 13, 2020
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 13, 2020
@openshift-docs-preview-bot

The preview will be available shortly at:

@qinpingli
Copy link

@wzheng1 Could you help take a look?

@wzheng1
Copy link

wzheng1 commented Aug 17, 2020

@dmage @ricardomaraschini My case is using "no_root_squash" to test, I am not sure whether "root_squash" works, could you please confirm?

@dmage
Copy link
Contributor

dmage commented Aug 18, 2020

I cannot confirm. Initially we had no_root_squash, and we tested it with no_root_squash.

@huffmanca why did we change it?

@bobfuru
Copy link
Contributor Author

bobfuru commented Sep 7, 2020

@huffmanca - wondering if you have any insights to share, also waiting to find out from Alex Gurenko if his attempts were successful to use root_squash + the supplementalGroup.

@bobfuru
Copy link
Contributor Author

bobfuru commented Oct 23, 2020

I'm going to close this PR. We have since updated image registry docs to remove NFS references, including these instructions to use supplementalGroup and root_squash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants