-
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
Github bot created bad release 1.7.3 #10749
Comments
This issue is currently awaiting triage. If CAPI contributors determine this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Could you please refer which clusterctl version you are using? This seems to be this bug: Which should have been solved in: which be part of clusterctl versions >= v1.7.0 :-) A similar fix was done for the e2e test module which is part of v1.7.3. |
Just followed the release process document, i will discuss this issue in the release call |
Note: v1.7.3 is not yet released, which is why binaries/yamls are not visible yet. The github action did actually work, but the release is not published yet. |
Yeah, everything is fine with the release. It's just in progress. The point here is just about if clusterctl works correctly or not. |
Thank you. Saw that the previous releases were done from a PR and the binary/yamls appeared in the release page after 15-20 minutes (almost 5h passed already now). I can close this issue if the release has not finished being released. The term |
I can close the issue if the "release" was properly created by the Github bot process and there are no pending issues on why it takes so much time. |
The reason is that cutting a release is much more than what the GitHub bot is doing. Please see: https://github.com/kubernetes-sigs/cluster-api-provider-vsphere/blob/main/docs/release/release-tasks.md What Christian is talking about is that clusterctl >= v1.7.0 should be able to handle this. It should just use v1.7.2. Are you using clusterctl >= v1.7.0? |
So to clarify:
|
This is what clusterctl v1.7.0 is doing on my env
|
Thank you for the explanation @sbueringer. It took me a while to test things out, as I had to wait around 4 minutes for clusterctl to run (as it had to timeout or maybe retry the http requests):
|
As I was using an old version of |
Thx for the feedback! I'll re-open. I think we should look into the 4 min delay. I saw something similar on my machine. Just wasn't sure if it's specific to my env /reopen (I'll leave the point about having an issue etc. up to the release team) |
@sbueringer: Reopened this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
(cc/fyi @chrischdi regarding the delay) |
Just checked again: The following PR: has an additional fix which improves on the timeout. This was backported to v1.7 and gets released in clusterctl v1.7.3 . |
Tested this with the partially-released CAPV v1.10.1 and with clusterctl v1.7.3 it was falling back in a reasonable timeframe to v1.10.0. I'll close this issue then My take on release process changes. We already have a lot of work, especially for the release team (see: https://github.com/kubernetes-sigs/cluster-api/blob/main/docs/release/release-tasks.md). I think it's not worth the effort to add additional work by having to open an issue for every release that we do. I'll prefer having to answer/close a few issues if folks are using old clusterctl versions & if the release process takes a bit longer. That's a lot less work in the long run. |
@sbueringer: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Thank you for the info and help, much appreciated! |
What steps did you take and what happened?
https://github.com/kubernetes-sigs/cluster-api/releases/tag/v1.7.3 does not contain any binary / yamls.
To give more context, this situation breaks the
cluster init
command, as by default, the command tries to find the latest release, and at this moment, it finds the 1.7.3 release on Github, and then tries to use files that are not in the release download links (as they currently don t exist).What did you expect to happen?
https://github.com/kubernetes-sigs/cluster-api/releases/tag/v1.7.3 to contain all binary / yamls like https://github.com/kubernetes-sigs/cluster-api/releases/tag/v1.7.2 does.
Cluster API version
None.
Kubernetes version
No response
Anything else you would like to add?
No response
Label(s) to be applied
/kind bug
One or more /area label. See https://github.com/kubernetes-sigs/cluster-api/labels?q=area for the list of labels.
The text was updated successfully, but these errors were encountered: