Skip to content

fix: add fallback volume when PVC creation is disabled#226

Merged
as51340 merged 1 commit intomemgraph:mainfrom
Linkurious:fix/statefulset-no-pvc-volume
Apr 20, 2026
Merged

fix: add fallback volume when PVC creation is disabled#226
as51340 merged 1 commit intomemgraph:mainfrom
Linkurious:fix/statefulset-no-pvc-volume

Conversation

@enys
Copy link
Copy Markdown
Contributor

@enys enys commented Apr 16, 2026

When persistentVolumeClaim.createStorageClaim is false, the lib-storage volumeMount was rendered without a corresponding volume definition, causing Kubernetes to reject the pod.

Fix: adds a volume entry in the volumes: section when createStorageClaim=false:

  • Uses existingClaim PVC if set
  • Falls back to emptyDir otherwise

From what I understand it could have been introduced by : https://github.com/memgraph/helm-charts/pull/12/changes

There's a couple of way to fix the issue, including adding a condition to the volume mount.
This pr needs a bit more review than my previous PR's I believe.

When persistentVolumeClaim.createStorageClaim is false, the lib-storage
volumeMount was rendered without a corresponding volume definition,
causing Kubernetes to reject the pod.

Now falls back to the existingClaim PVC if set, or emptyDir otherwise.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
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.

I am OK with this

@as51340 as51340 merged commit 88136a0 into memgraph:main Apr 20, 2026
4 of 5 checks passed
@as51340 as51340 added docs-changelog-only If changelog is enough for this change, no need to update docs. bug Something isn't working memgraph-ha Memgraph HA chart issue labels Apr 24, 2026
@as51340 as51340 modified the milestones: Memgraph HA 0.2.19, Memgraph standone 0.2.19 Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working docs-changelog-only If changelog is enough for this change, no need to update docs. memgraph-ha Memgraph HA chart issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants