-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
clusterctl config repositories point to invalid URLs #2830
Comments
Seems the correct URL should be https://github.com/kubernetes-sigs/cluster-api-provider-vsphere/releases/latest/download/infrastructure-components.yaml missing the /assign @fabriziopandini @wfernandes |
@vincepri yes, sorry, I forgot to notate what the correct URL should be. Was thinking it also should reference the correct version that is associated with the currently installed |
Sounds like a valid ask to me |
WRT to the suggestion of making the url corresponding to the real url I agree this can be less confusing for the users. However, we should make sure that the download/ subpath is ignored otherwise the api calls will fail.
The idea behind Nevertheless, the idea of pinning versions IMO is not applicable to the list of out the box providers, because as of today this will require a coordination/synchronization between 6 different projects to define the correct version to be associated with the currently installed clusterctl binary, and a considerable churn as each of the 6 projects move forward with different timelines. (TL;DR; the correct version is hard to define) Please note that, if necessary, each user can pin the provider definition to a specific version by replacing "latest" with a version tag using the clusterctl config file as described here https://cluster-api.sigs.k8s.io/clusterctl/configuration.html#provider-repositories, but TBH i think that in this case, the CLI syntax for pinning version is much more simple and explicit (eg. -infrastructure aws:v0.5.1) |
Maybe we should remove the full URL from the output and just point to the base repository instead? |
we can also output the version separately |
How the URL should be interpreted to determine what is the base repository vs what is latest/version depends on the type of repository (Github vs FileSystem). Instantiating those repository objects might be overkilling for the current command output, but if instead, we decide to redesign the output and show available versions this could make sense (there is also an issue for this #2171). |
/area clusterctl |
Maybe we can present the URLs in the output of I believe our documentation already refers to Again, the internal defaults would be the original URL and wouldn't change, but the CLI output would just trim out the yaml filename in the "presentation layer". WDYT? |
I can work on this issue regarding the solution I proposed above. What do y'all think? |
Hey @wfernandes apologies for the delayed response. I think your solution is acceptable. It would certainly fix the confusion surrounding broken links and inaccessible URLs. Probably the best approach considering what @fabriziopandini has pointed out. Other thoughts??? |
Cool...I can get started on this and I'm open to feedback and suggestions @fabriziopandini if you have any input on this. |
/assign |
@wfernandes +1 to the idea |
/lifecycle active |
What steps did you take and what happened:
Run the following:
Pick any of the repositories and you will get a 'Not Found' message:
What did you expect to happen:
The repositories will reference valid YAML repositories AND the specific version of clusterctl should be used in the repository URL (not latest).
Anything else you would like to add:
N/A
Environment:
kubectl version
): N/A/etc/os-release
): MacOS/kind bug
The text was updated successfully, but these errors were encountered: