-
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
✨ Add ClusterClass changes e2e test #5664
✨ Add ClusterClass changes e2e test #5664
Conversation
57aae42
to
50fa707
Compare
/test pull-cluster-api-e2e-full-main |
50fa707
to
4d5b8bf
Compare
4d5b8bf
to
b01738d
Compare
/test pull-cluster-api-e2e-full-main |
b01738d
to
e08348f
Compare
/test pull-cluster-api-e2e-full-main |
/retest |
e08348f
to
bb596a3
Compare
/test pull-cluster-api-e2e-full-main |
bb596a3
to
5e5beed
Compare
webhooks/cluster.go
Outdated
), | ||
) | ||
} | ||
// FIXME(before merge): can be removed when https://github.com/kubernetes-sigs/cluster-api/pull/5644 has been merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will be rolled back before merge after #5644 has been merged
/test pull-cluster-api-e2e-full-main |
5e5beed
to
5ddbbea
Compare
/test pull-cluster-api-e2e-full-main |
/test pull-cluster-api-test-mink8s-main |
5ddbbea
to
ef63d7f
Compare
/hold |
/test pull-cluster-api-e2e-full-main |
Tests fail because main is broken right now |
ef63d7f
to
d497c07
Compare
Fix merged, should work now: |
d497c07
to
ba8758a
Compare
/hold cancel |
/test pull-cluster-api-e2e-full-main |
ba8758a
to
cf4fd5d
Compare
/test pull-cluster-api-e2e-full-main Rebased on top of main after the upgrade PR has been merged. |
/test pull-cluster-api-e2e-full-main |
1 similar comment
/test pull-cluster-api-e2e-full-main |
cf4fd5d
to
af3797d
Compare
/test pull-cluster-api-e2e-full-main @fabriziopandini Thx for the review, should be fixed/answered PTAL. |
Signed-off-by: Stefan Büringer buringerst@vmware.com
af3797d
to
baa09c3
Compare
/test pull-cluster-api-e2e-full-main @fabriziopandini Should be fixed, I hope I found every occurrence. |
/test pull-cluster-api-e2e-full-main |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabriziopandini The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What this PR does / why we need it:
This PR adds an e2e test which verifies that changes to the ClusterClass and ClusterClass rebases are rolled out correctly.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #