Skip to content

allow SINGLE_WRITER to support ReadWriteOncePod#52

Merged
hlts2 merged 1 commit into
civo:masterfrom
ddollar:read-write-once-pod
Nov 18, 2025
Merged

allow SINGLE_WRITER to support ReadWriteOncePod#52
hlts2 merged 1 commit into
civo:masterfrom
ddollar:read-write-once-pod

Conversation

@ddollar
Copy link
Copy Markdown
Contributor

@ddollar ddollar commented Nov 11, 2025

I'm trying to use the ReadWriteOncePod access mode for a PersistentVolumeClaim

This should be exactly like ReadWriteOnce but can only be mounted once per pod. With ReadWriteOnce two pods on the same node could both write to the same volume, which would be not a good thing for many use cases such as databases.

I tried making this change and rolling a custom civo-csi image in my Civo cluster but I got the following error when trying to create the PVC

Warning  ProvisioningFailed    12s (x5 over 27s)  csi.civo.com_civo-csi-controller-0_48a8b16a-4214-4849-b034-61fc2c4e7a6e  failed to provision volume with StorageClass "civo-volume": rpc error: code = InvalidArgument desc = CreateVolume access mode isn't supported

@davenull
Copy link
Copy Markdown

This change seems appropriate after looking at it in context. I will pass this up to the team and make sure there is nothing else we need to change to make this work properly, but I don't see much that this can affect negatively at the moment. Thank you for the PR!

Copy link
Copy Markdown
Member

@hlts2 hlts2 left a comment

Choose a reason for hiding this comment

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

Hi @ddollar , thank you for creating this PR. I’ve reviewed the code changes, and they look fine. Therefore, I have approved this PR—thank you!

Regarding the error you encountered, resolving it will require updating the versions of some dependent plugins. This will need separate testing, so please bear with us a little while longer until a fix is ready. Thank you!

@hlts2 hlts2 merged commit b1e4b85 into civo:master Nov 18, 2025
2 of 3 checks passed
@hlts2 hlts2 mentioned this pull request Nov 20, 2025
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