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

Add -s as shorthand for --source-image#129

Merged
shashwathi merged 1 commit into
vmware-tanzu:mainfrom
warango4:issue126
May 26, 2022
Merged

Add -s as shorthand for --source-image#129
shashwathi merged 1 commit into
vmware-tanzu:mainfrom
warango4:issue126

Conversation

@warango4
Copy link
Copy Markdown
Contributor

@warango4 warango4 commented May 22, 2022

Pull request

What this PR does / why we need it

Remove -s as shorthand for --service-account and add it for --source-image

Which issue(s) this PR fixes

Fixes #126

Describe testing done for PR

  • Created workload in local cluster using -s and passing a source image after flag
    • Inside pet-clinic folder (https://github.com/sample-accelerators/spring-petclinic)
      • $ tanzu apps workload create petclinic-local --local-path . -s gcr.io/tanzu-framework-playground/petclinic-local --type web
      • Note: use your gcr (or whichever registry) repository to publish source

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

warango4 commented May 22, 2022

$ tanzu apps workload get petclinic-local

image

@shashwathi shashwathi requested a review from danfein May 23, 2022 18:00
Signed-off by: Wendy Arango warango@vmware.com
@shashwathi shashwathi merged commit 196cfd4 into vmware-tanzu:main May 26, 2022
@warango4 warango4 deleted the issue126 branch July 14, 2022 13:52
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.

Remove -s alias for the --service-account flag and apply the -s alias to --source-image instead

4 participants