Document URL: https://docs.openshift.com/enterprise/3.0/install_config/install/docker_registry.html#storage-for-the-registry Section Number and Name: Storage for the Registry Describe the issue: https://docs.openshift.com/enterprise/3.0/install_config/install/docker_registry.html#storage-for-the-registry The preceding paragraph to the command([1] : $ oc volume deploymentconfigs/docker-registry \ --add --overwrite --name=registry-storage --mount-path=/registry \ --source='{"nfs": { "server": "<fqdn>", "path": "/path/to/export"}}') says: --- For production use, you should attach a remote volume or use persistent storage using PersistentVolume and PersistentVolumeClaim objects for storage for the registry. For example, to attach an existing NFS volume to the registry once it has been defined: --- But then the command does not show using a PV/PVC. Suggestions for improvement: You need to mention steps for pv and pvc. https://github.com/openshift/training/blob/master/07-Registry-and-Storage.md Additional information:
Miheer, I have created a PR for this BZ: https://github.com/openshift/openshift-docs/pull/1273 In your opinion, does this PR satisfy this BZ? Thanks.
I believe the issue is mainly with the "oc volume" command using --source (a JSON string) instead of --claim-name (a PVC). See: https://github.com/openshift/training/blob/master/07-Registry-and-Storage.md#attach-the-volume-to-the-registry oc volume usage: https://docs.openshift.com/enterprise/latest/dev_guide/volumes.html Discusses creating PVCs: https://docs.openshift.com/enterprise/latest/dev_guide/persistent_volumes.html
Alex, ok. I think I get it. Miheer, I've added an example for adding pvcs as well: https://github.com/bfallonf/openshift-docs/blob/BZ1277029/install_config/install/docker_registry.adoc#storage-for-the-registry Miheer, is this closer to what you asked for?
Hi Brice, Yes it looks good. Thanks and regards, Miheer Salunke
Miheer, Thanks for that. PR is onto peer review. B
Commit pushed to master at https://github.com/openshift/openshift-docs https://github.com/openshift/openshift-docs/commit/f281a5afa0096d50b57ea12458d0d20e6c1b022e Merge pull request #1273 from bfallonf/BZ1277029 [peer review] Bug 1277029: edit link to storage options