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

Add hint message in workload create/update/apply/get#163

Merged
rashedkvm merged 1 commit into
vmware-tanzu:mainfrom
warango4:issue150
Jun 8, 2022
Merged

Add hint message in workload create/update/apply/get#163
rashedkvm merged 1 commit into
vmware-tanzu:mainfrom
warango4:issue150

Conversation

@warango4
Copy link
Copy Markdown
Contributor

@warango4 warango4 commented Jun 7, 2022

Pull request

What this PR does / why we need it

Add hint message for commands that can be used for troubleshooting or just check status of a workload.

  • For create/update/apply: add hint so user can check logs and status through tail and get commands
  • For get: add hint so user can check logs through tail command

Which issue(s) this PR fixes

Fixes #150

Describe testing done for PR

  • Unit testing
  • Test in local cluster

Additional information or special notes for your reviewer

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

@warango4
Copy link
Copy Markdown
Contributor Author

warango4 commented Jun 8, 2022

image

@warango4
Copy link
Copy Markdown
Contributor Author

warango4 commented Jun 8, 2022

image

@warango4
Copy link
Copy Markdown
Contributor Author

warango4 commented Jun 8, 2022

image

@warango4
Copy link
Copy Markdown
Contributor Author

warango4 commented Jun 8, 2022

image

Comment thread pkg/commands/workload.go Outdated
Comment thread pkg/commands/workload.go
Comment thread pkg/commands/workload_apply.go
Add hint message for commands that can be used for troubleshooting or just check status of a workload.

- For create/update/apply: add hint so user can check logs and status through tail and get commands
- For get: add hint so user can check logs through tail command

Signed-off-by: Wendy Arango warango@vmware.com
@rashedkvm rashedkvm merged commit 00393ac into vmware-tanzu:main Jun 8, 2022
@warango4 warango4 deleted the issue150 branch June 13, 2022 14:18
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.

Append "next command recommendations" to the output from workload create/update/apply/get

5 participants