Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bugfix: modify volume inspect docs #1029

Merged
merged 1 commit into from
Apr 3, 2018

Conversation

rudyfly
Copy link
Collaborator

@rudyfly rudyfly commented Apr 2, 2018

Ⅰ. Describe what this PR did

modify volume inspect docs, it supports to inspect multi-volumes,
and supports to inspect volume with format.

Ⅱ. Does this pull request fix one issue?

NONE

Ⅲ. Describe how you did it

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

Signed-off-by: Rudy Zhang rudyflyzhang@gmail.com

@pouchrobot pouchrobot added areas/storage kind/bug This is bug report for project size/S labels Apr 2, 2018
cli/volume.go Outdated
Use: "inspect [OPTIONS] NAME",
Short: "Inspect a pouch volume",
Use: "inspect [OPTIONS] Volume [Volume...]",
Short: "Inspect pouch one or more volumes",
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess "Inspect one or more pouch volumes" is better.

Copy link
Contributor

Choose a reason for hiding this comment

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

thanks a lot for your work, i think we should also fix network inspect command !

@codecov-io
Copy link

codecov-io commented Apr 2, 2018

Codecov Report

Merging #1029 into master will increase coverage by 0.25%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1029      +/-   ##
==========================================
+ Coverage   14.91%   15.17%   +0.25%     
==========================================
  Files         133      135       +2     
  Lines        8483     8490       +7     
==========================================
+ Hits         1265     1288      +23     
+ Misses       7118     7101      -17     
- Partials      100      101       +1
Impacted Files Coverage Δ
cli/volume.go 9.14% <0%> (-0.41%) ⬇️
client/system.go
client/system_ping.go 77.77% <0%> (ø)
client/registry_login.go 100% <0%> (ø)
client/system_info.go 100% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update caf4ea8...eeb9267. Read the comment docs.

modify volume inspect docs, it supports to inspect  multi-volumes.

Signed-off-by: Rudy Zhang <rudyflyzhang@gmail.com>
@allencloud
Copy link
Collaborator

LGTM

@allencloud allencloud merged commit b1b2207 into AliyunContainerService:master Apr 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
areas/storage kind/bug This is bug report for project size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants