Skip to content
This repository was archived by the owner on May 18, 2026. It is now read-only.

Add namespace to workload get overview section#357

Merged
shashwathi merged 1 commit into
vmware-tanzu:mainfrom
warango4:issue321
Oct 4, 2022
Merged

Add namespace to workload get overview section#357
shashwathi merged 1 commit into
vmware-tanzu:mainfrom
warango4:issue321

Conversation

@warango4
Copy link
Copy Markdown
Contributor

@warango4 warango4 commented Sep 29, 2022

Pull request

What this PR does / why we need it

Add namespace to workload get output in Overview section

Which issue(s) this PR fixes

Fixes #321

Describe testing done for PR

  • Updated unit testing

Additional information or special notes for your reviewer

Signed-off-by: Wendy Arango warango@vmware.com

@warango4
Copy link
Copy Markdown
Contributor Author

image

@warango4
Copy link
Copy Markdown
Contributor Author

image

Copy link
Copy Markdown
Contributor

@odinnordico odinnordico left a comment

Choose a reason for hiding this comment

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

Please update docs

Copy link
Copy Markdown
Contributor

@shashwathi shashwathi left a comment

Choose a reason for hiding this comment

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

looks good. Update docs as mentioned in other comment.

Copy link
Copy Markdown
Contributor

@shaheerkootteeri shaheerkootteeri left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread docs/commands-details/workload_get.md Outdated
Comment thread docs/commands-details/workload_get.md Outdated
Signed-off-by: Wendy Arango <warango@vmware.com>
@shashwathi shashwathi merged commit 279de91 into vmware-tanzu:main Oct 4, 2022
@warango4 warango4 deleted the issue321 branch October 5, 2022 16:20
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.

add namespace to ...workload get ... output

5 participants