Skip to content

feat: Manually chown in the case when fsGroup not respected#243

Merged
as51340 merged 2 commits into
mainfrom
feat/manual-chown
May 19, 2026
Merged

feat: Manually chown in the case when fsGroup not respected#243
as51340 merged 2 commits into
mainfrom
feat/manual-chown

Conversation

@as51340
Copy link
Copy Markdown
Collaborator

@as51340 as51340 commented May 18, 2026

Some storage classes don't support fsGroup. In that case, user should manually chown on the volumes used. This feature allows easier chown by just setting a switch. Inspired by mysql-operator.

@as51340 as51340 self-assigned this May 18, 2026
@as51340 as51340 added docs-needed Set if you think documentation needs to be updated because of this change memgraph-ha Memgraph HA chart issue feature New non-breaking feature labels May 18, 2026
@as51340
Copy link
Copy Markdown
Collaborator Author

as51340 commented May 18, 2026

Tracking

  • [Link to Epic/Issue]

Standard development

  • Update / add chart templates, values, and NOTES.txt as needed
  • Run helm lint and helm template locally against the changed chart(s)
  • Verify the chart installs and upgrades cleanly on a test cluster

Labels checklist

  • Add a docs label (exactly one): docs-changelog-only, docs-needed, or docs-not-needed
  • Add at least one component label: memgraph, memgraph-ha, memgraph-lab, or infrastructure
  • Add at least one type label: bug, feature, or infrastructure
  • Assign the PR to a milestone
    • If not known, set for a later milestone

Documentation checklist

  • Write a release note, including added/changed clauses
    An init container can now be added by enabling fixOwnershipInitContainer value which will chown to memgraphUserId:memgraphGroupId in situations when used storage class doesn't support fsGroup. #243
    • What has changed? What does it mean for a user? What should a user do with it? [#{{PR_number}}]({{link to the PR}})
  • Documentation PR link
    • Is back linked to this development PR

Copy link
Copy Markdown
Contributor

@mattkjames7 mattkjames7 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 we should specify a version for busybox, but otherwise looks good

Comment thread charts/memgraph-high-availability/values.yaml Outdated
@as51340 as51340 added this to the 19th May 2026 milestone May 19, 2026
@as51340 as51340 merged commit 828edb5 into main May 19, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-needed Set if you think documentation needs to be updated because of this change feature New non-breaking feature memgraph-ha Memgraph HA chart issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants