-
Notifications
You must be signed in to change notification settings - Fork 62
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
Added flag --layers to enable/disable functionality of showing each layers of image #639
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added some comments.
pkg/imgpkg/cmd/describe.go
Outdated
@@ -81,7 +84,7 @@ func (d *DescribeOptions) Run() error { | |||
ttyEnabledLogger := util.NewUILevelLogger(logLevel, util.NewLoggerNoTTY(d.ui)) | |||
if d.OutputType == "text" { | |||
p := bundleTextPrinter{logger: ttyEnabledLogger} | |||
p.Print(description) | |||
p.Print(description, d.Layers) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are not providing this option for the yaml output?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In yaml with empty object it will not show the layers in output only.
…yer of image Signed-off-by: kumari tanushree <ktanushree@vmware.com>
ffb1f49
to
e75c415
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.