surahman edited a comment on issue #3723:
URL: 
https://github.com/apache/incubator-heron/issues/3723#issuecomment-953211624


   This should be rather straightforward to implement for the CLI. We can use 
`KubernetesController.getConfigItemsByPrefix` to collect all the relevant 
options from the config. As per our discussions, we should be using [dynamic 
provisioning](https://kubernetes.io/docs/concepts/storage/dynamic-provisioning/)?
   
   Suggested commands are below based on the Spark example, anything else we 
should add?
   
   ```bash
   --config-property heron.kubernetes.persistentVolumeClaim.[volume 
name].options.claimName=OnDemand
   --config-property heron.kubernetes.persistentVolumeClaim.[volume 
name].options.storageClass=gp
   --config-property heron.kubernetes.persistentVolumeClaim.[volume 
name].options.sizeLimit=500Gi
   --config-property heron.kubernetes.persistentVolumeClaim.[volume 
name].mount.path=/data
   --config-property heron.kubernetes.persistentVolumeClaim.[volume 
name].mount.readOnly=false
   ```
   
   I can start work on this but I can only take it as far as where it needs to 
be wired into #3710. I will then need to `rebase` onto that PR and wire it in. 
I will need also to clean up the test suite and perform some other general 
merge-conflict like clean-up operations at that point.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to