-
Notifications
You must be signed in to change notification settings - Fork 423
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
Pin k8s e2e version to v1.25.11 #3762
Pin k8s e2e version to v1.25.11 #3762
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #3762 +/- ##
=======================================
Coverage 54.12% 54.12%
=======================================
Files 187 187
Lines 18909 18909
=======================================
Hits 10234 10234
Misses 8126 8126
Partials 549 549 ☔ View full report in Codecov by Sentry. |
/test pull-cluster-api-provider-azure-e2e-optional |
This looks to be mitigating the issue (e2e test passed on first try), started the optional test to run the dual stack test as well /assign @nojnhuh |
/hold for optional test |
/cherry-pick release-1.10 |
@CecileRobertMichon: once the present PR merges, I will cherry-pick it on top of release-1.10 in a new PR and assign it to you. 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. |
AKS autoscaler flake /test pull-cluster-api-provider-azure-e2e-aks |
@mboersma @nojnhuh thoughts on overriding the aks test here? this is fixing #3754 (or at least getting us unblocked for now) but it can't merge because the aks test is broken in main (tracked by #3763) which is unrelated to this PR (the k8s version change doesn't affect AKS) and @nojnhuh is looking into it separately. |
+1 to overriding AKS test here |
@CecileRobertMichon: Overrode contexts on behalf of CecileRobertMichon: pull-cluster-api-provider-azure-e2e-aks 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. |
/cherry-pick release-1.10 since this seems to be affecting both |
@CecileRobertMichon: once the present PR merges, I will cherry-pick it on top of release-1.10 in a new PR and assign it to you. 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. |
/hold cancel |
/lgtm |
LGTM label has been added. Git tree hash: 0f0ff07cc2ed08b67ee115d7a5b524f2ca9ca044
|
/override pull-cluster-api-provider-azure-e2e-aks |
@CecileRobertMichon: Overrode contexts on behalf of CecileRobertMichon: pull-cluster-api-provider-azure-e2e-aks 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. |
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.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mboersma 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 |
@CecileRobertMichon: new pull request created: #3764 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. |
@CecileRobertMichon: new pull request created: #3765 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. |
What type of PR is this?
/kind failing-test
What this PR does / why we need it: Trying to see if this fixes #3754
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):mitigates #3754
Special notes for your reviewer:
TODOs:
Release note: