-
Notifications
You must be signed in to change notification settings - Fork 261
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
v0.4.0 (v1alpha4) #745
Comments
/assign |
@jichenjc @hidekazuna I'm still fighting a bit with OpenLab but I already had green conformance test and only had a minor issue left regarding cluster delete. But now it fails, imho because of random reboots of OpenLab servers. But I think fixing those issues (and probably just retrying a lot) won't lead to a lot of changes on the PR. |
I would keep this issue for now to track further work regarding v1alpha4. Afaik the plan is to release cluster-api v1alpha4 in a few weeks. |
Current plan is to release CAPI v1alpha4 in June. I keep this issue for an overview over ongoing implementations for v1alpha4. Currently, there is nothing to do. |
As written in Slack. We will probably release next week. |
Since CAPI v0.4.0 has been released, does not include kubernetes-sigs/cluster-api#4514, I think we can release v0.4.0 does not care kubernetes-sigs/cluster-api#4514. |
is it mandatory? If it's an enhancement we can always have 0.4.1 + ? |
If we got the code ready I would really prefer merging it before. Afaik it's absolutely intended to be binding for v1alpha4. Otherwise imho they would have to wait probably until v1alpha5/v1beta1 with a change like this. And I would much rather imnplement this in CAPO v0.4.0 then have a breaking change in v0.4.1 |
not realize it's a breaking change ,if it is, then yes, definitely need wait |
@jichenjc I think it's a breaking change because we change fields in the InfraCluster CRD. It's somewhat mitigated between v0.3.0 and v0.4.0 with the conversion. But I don't want to implement a conversion webhook for v0.4.0 to v0.4.1 (and I don't know if it's possible :)). |
We are now beta mode then we wait for the following PRs? |
@hidekazuna I will push the last commits of my PR in a few seconds and remove the WIP label. Then it should be ready to merge the next days. |
I created an issue regarding the readme. |
We've decided not to wait for #934 and #921. https://kubernetes.slack.com/archives/CFKJB65G9/p1626244978144500 |
Further, the last PRs are merged
So we are good to release |
All done /close |
@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/test-infra repository. |
/kind feature
v1alpha4 implementation is on the main branch now. This issue will provide an overview of open tasks until the v1alpha4 release:
Postponed to v0.5+:
The text was updated successfully, but these errors were encountered: