[rstudio-connect] wrong type for var

[Almenon]

.Values.sharedStorage.storageClassName has the type bool and default false.

helm/charts/rstudio-connect/README.md

Line 178 in 666e86d

| sharedStorage.storageClassName | bool | `false` | The type of storage to use. Must allow ReadWriteMany |

However, it's passed to the PersistentVolumeClaim kind, spec.storageClassName, which is a string.

https://github.com/rstudio/helm/blob/666e86da88124a40e823a9e23925702d151b8076/charts/rstudio-connect/templates/pvc.yaml#L14C3-L14C19

I haven't tested it, but based on the API reference this is probably a bug.

[bschwedler]

Thank you for reporting this!

It looks like we also have a couple of other variables that we need to update for the correct type.

Setting up a JSON schema as in #35 should help us catch & document these in a more automated fashion.

[colearendt]

Also worth noting that "mutable types" are kinda a weird / hard thing in helm.

While we do false as a proxy for "not set" currently in a handful of places (and we should not), there are also some places where we allow either a {} or a string, for instance. Helm is not a fan of this, so we will need to do some exploration on those items.

[Almenon]

Looks like same issue with package manager FYI