[GitHub] ruromero commented on issue #153: Add support for persistent storage.

[GitBox <gi...@apache.org>]

ruromero commented on issue #153: Add support for persistent storage.
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/153#issuecomment-363561121
 
 
   @dgrove-oss as you have added the PVC reference in the Deployment objects I'd add the PVC definition in the same file as each deployment.
   As any user might want to use different persistent storage (e.g. NFS, iSCSI, etc.) and/or of different size. It is fine to provide an example with hostPath but as they have the same accessMode and size (almost), they will end up being bound with mismatching names. E.g. pv-zookeeper-datalog-01 with pv-apigateway-01. It can also be the case where the PVCs requesting 1Gi are bound to the 2Gi PVs leaving the other PVCs of 2Gi without any available PV.
   Therefore the PV name is not meaningful unless you force it with [StorageClass](https://kubernetes.io/docs/concepts/storage/persistent-volumes/#class) or [selector.matchLabels](https://kubernetes.io/docs/concepts/storage/persistent-volumes/#persistentvolumeclaims)

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services