Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Update docker-volume-cli.md #795

Merged
merged 2 commits into from
Dec 6, 2016
Merged

Update docker-volume-cli.md #795

merged 2 commits into from
Dec 6, 2016

Conversation

shuklanirdesh82
Copy link
Contributor

@shuklanirdesh82 shuklanirdesh82 commented Dec 5, 2016

using 'vsan-policy-name' instead of 'policy' to keep the documentation in sync with the code and consistent with https://github.com/vmware/docker-volume-vsphere/blob/8717d753b606fad41597d074e5ee07a6893ee2f2/docs/user-guide/admin-cli.md#list-4

/CC @kerneltime

using 'vsan-policy-name' instead of 'policy' to keep the documentation in sync with the code and consistent with https://github.com/vmware/docker-volume-vsphere/blob/8717d753b606fad41597d074e5ee07a6893ee2f2/docs/user-guide/admin-cli.md#list-4
Copy link
Contributor

@kerneltime kerneltime left a comment

Choose a reason for hiding this comment

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

Please rename it to CLI invocation ### vsan-policy-name vs ### policy (vsan-policy-name)

@@ -20,13 +20,13 @@ The volume units can be ```kb, mb, gb and tb```

The default volume size is 100mb

### policy
### policy (vsan-policy-name)
Copy link
Contributor

Choose a reason for hiding this comment

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

Rename this to vsan-policy-name, it should reflect the CLI invocation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure!

@kerneltime kerneltime merged commit db3a74e into master Dec 6, 2016
@shuklanirdesh82 shuklanirdesh82 deleted the shuklanirdesh82-patch-2 branch December 6, 2016 17:18
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants