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

add userStorageAccessMode for UserClaim #95

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

camrossi
Copy link

This PR would address #94

Copy link
Collaborator

@as51340 as51340 left a comment

Choose a reason for hiding this comment

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

Nice work @camrossi! Thanks for the contribution!

@as51340 as51340 added this to the Memgraph 2.23 milestone Dec 10, 2024
Copy link
Collaborator

@antejavor antejavor left a comment

Choose a reason for hiding this comment

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

Hi @camrossi, thanks a lot for this contribution 🚀

Before merging, can you update the README that we have on Github for values? https://github.com/memgraph/helm-charts/blob/main/charts/memgraph/README.md#configuration-options

It describes all configuration options there, after that we can merge!

@camrossi
Copy link
Author

@antejavor I added the doc for all the UserClaim options as none where present. Also I am wondering if the userMountPath shouldn't be set to something as leaving it empty does require the user to manually set it every time

Copy link
Collaborator

@antejavor antejavor left a comment

Choose a reason for hiding this comment

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

I think it is ok default, so we can leave it set.

Looks good to me! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants