Skip to content

fix: improve kargo helm example#1

Open
rklonner wants to merge 4 commits intoakuity:mainfrom
rklonner:improve-instructions-of-kargo-helm-example
Open

fix: improve kargo helm example#1
rklonner wants to merge 4 commits intoakuity:mainfrom
rklonner:improve-instructions-of-kargo-helm-example

Conversation

@rklonner
Copy link
Copy Markdown

Hi,

I worked through the example and stumbled upon some issues that I would like to address in this PR.

as the instructions and Argo manifests do not mention a cluster with name demo
anywhere.

Signed-off-by: Robert Klonner <r@klonner.cc>
Signed-off-by: Robert Klonner <r@klonner.cc>
Signed-off-by: Robert Klonner <r@klonner.cc>
like mentioned in the official kargo docs:
https://docs.kargo.io/user-guide/installing-the-cli/

Otherwise the resulting file will contain the text "Not found"
instead of downloading the binary.

Signed-off-by: Robert Klonner <r@klonner.cc>
@rklonner
Copy link
Copy Markdown
Author

@jessesuen : The repo is not mentioning contribution possibilities directly but I hope you can have a look and see if you find these modifications useful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant