Skip to content

fix(chart): sane default storage (ReadWriteOnce + cluster default class)#24

Open
andrey-embarklabs wants to merge 1 commit into
NrgXnat:mainfrom
andrey-embarklabs:plt/xnat-chart-storage-defaults
Open

fix(chart): sane default storage (ReadWriteOnce + cluster default class)#24
andrey-embarklabs wants to merge 1 commit into
NrgXnat:mainfrom
andrey-embarklabs:plt/xnat-chart-storage-defaults

Conversation

@andrey-embarklabs

Copy link
Copy Markdown
Contributor

The five XNAT data volumes (and the bundled CNPG cluster) defaulted to accessMode ReadWriteMany on a storageClass literally named "default", which is unsatisfiable on a standard EKS/EBS cluster (EBS has no RWX, and few clusters have a class named "default") — PVCs sit Pending on a fresh single-replica install.

  • Default the data volumes to ReadWriteOnce.
  • Default storageClass to "" so PVCs use the cluster's default StorageClass; the templates omit storageClassName when empty rather than setting it to "" (which would disable dynamic provisioning).
  • Same for cnpg.cluster.storage.storageClass; the Cluster template now omits the field when empty.
  • Document that multi-replica XNAT or the container-service should set the shared volumes (build, archive) to ReadWriteMany on an RWX class.

The five XNAT data volumes (and the bundled CNPG cluster) defaulted to
accessMode ReadWriteMany on a storageClass literally named "default",
which is unsatisfiable on a standard EKS/EBS cluster (EBS has no RWX, and
few clusters have a class named "default") — PVCs sit Pending on a fresh
single-replica install.

- Default the data volumes to ReadWriteOnce.
- Default storageClass to "" so PVCs use the cluster's default
  StorageClass; the templates omit storageClassName when empty rather than
  setting it to "" (which would disable dynamic provisioning).
- Same for cnpg.cluster.storage.storageClass; the Cluster template now
  omits the field when empty.
- Document that multi-replica XNAT or the container-service should set the
  shared volumes (build, archive) to ReadWriteMany on an RWX class.

@kathrynalpert kathrynalpert left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is breaking for existing installs, right? We should make sure this is clear in the semver

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.

2 participants