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

Documentation update for admin policy rm and docker volume inspect commands #1030

Merged
merged 1 commit into from
Mar 14, 2017

Conversation

pshahzeb
Copy link
Contributor

@pshahzeb pshahzeb commented Mar 13, 2017

  1. Usage of --name option instead of positional name for admin policy rm command

  2. Note for docker volume inspect command in case of zeroedthick and eagerzeroedthick disk formats. It shows allocated size to be total size plus the size of replicas

Fixes #922 #961

…mmands

1. Usage of --name option instead of positional name for admin policy rm command

2. Note for docker volume inspect command in case of zeroedthick and eagerzeroedthick disk formats
   It shows allocated size to be total size plus the size of replicas

Fixes #922 #961
@pshahzeb pshahzeb changed the title Documentaiton update for admin policy rm and docker volume inspect co… Documentaiton update for admin policy rm and docker volume inspect commands Mar 13, 2017
@pshahzeb pshahzeb changed the title Documentaiton update for admin policy rm and docker volume inspect commands Documentation update for admin policy rm and docker volume inspect commands Mar 13, 2017
@@ -127,6 +127,7 @@ docker volume inspect MyVolume
}
]
```
Note: For disk formats zeroedthick and zeroedthick, the allocated size would be total size plus the size of replicas.
Copy link
Contributor

Choose a reason for hiding this comment

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

don't we need to add this at known issue?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

IMO, it is not an issue. The disk format's nature results in this behavior.

Copy link
Contributor

@shuklanirdesh82 shuklanirdesh82 left a comment

Choose a reason for hiding this comment

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

Please check with @msterin or @pdhamdhere for adding the Note into known issue part or not.

Copy link
Contributor

@shaominchen shaominchen left a comment

Choose a reason for hiding this comment

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

LGTM.

@msterin
Copy link
Contributor

msterin commented Mar 14, 2017

lgtm

@msterin msterin merged commit d81eec7 into master Mar 14, 2017
@msterin msterin deleted the issue_961_922.pshahzeb branch March 14, 2017 00:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants