Skip to content
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

deploy - correct go install output #586

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DamianZaremba
Copy link

Upon an invalid kind version, the following message is displayed:
Error: failed to deploy cluster: failed to check for dependencies: kind version check failed: got 0.19.0, want 0.24.0. install with go install sigs.k8s.io/[email protected]

Executing the instruction fails, due to an in-valid version tag: go: sigs.k8s.io/[email protected]: sigs.k8s.io/[email protected]: invalid version: unknown revision 0.24.0

The v is being stripped off by parseVersion as changed in 1d152a.

Since parseVersion explicitly requires a v prefix, it should be safe to just add this back in the human output.

Upon an invalid `kind` version, the following message is displayed:
Error: failed to deploy cluster: failed to check for dependencies: kind version check failed: got 0.19.0, want 0.24.0. install with `go install sigs.k8s.io/[email protected]`

Executing the instruction fails, due to an in-valid version tag:
`go: sigs.k8s.io/[email protected]: sigs.k8s.io/[email protected]: invalid version: unknown revision 0.24.0`

The `v` is being stripped off by `parseVersion` as changed in 1d152a.

Since `parseVersion` explicitly requires a `v` prefix, it should be safe
to just add this back in the human output.
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