CNTRLPLANE-371: Update to Kubernetes v1.33.2 #2314
Conversation
also debug inotify limits before/after setting
…erd-node-e2e CI jobs Signed-off-by: Davanum Srinivas <[email protected]>
Signed-off-by: Davanum Srinivas <[email protected]>
…-kube-up kube-up.sh: set inotify limits
…n-for-kubelet-metrics-url Use established pattern for kubelet metrics url
…ory_swap_limit_bytes-in-ci-cos-cgroupv1-containerd-node-e2e-ci-jobs Fix container_spec_memory_swap_limit_bytes in ci-cos-cgroupv1-containerd-node-e2e CI jobs
Signed-off-by: Davanum Srinivas <[email protected]>
…to-reflect-additional-ci-job-environments Adjust metrics ranges to reflect additional CI job environments
Surface Pressure Stall Information (PSI) metrics
When the kube-apiserver has --anonymous-auth=false, the regular http.Client.Get() that WaitForAllControlPlaneComponents does will not work. Always use the discovery client when checking the health status of the kube-apiserver. Do a minor rework of struct fields and unit tests. Replace nil client in cmd/phases/join/waitcontrolplane.go.
…ance Add support for HPA configurable tolerance
Remove endpointSlicesHaveSameZoneHints check. We are testing that connections end up at the right endpoints. We don't need to validate _why_ they go to the right endpoints, which is already tested by other tests anyway. (Also, validating the hints becomes more complicated in the same-node case, where there may or may not also be same-zone hints depending on cluster configuration.) Remove DeferCleanup calls; we don't need to delete anything manually because namespaced resources will automatically be deleted when the test case's namespace is deleted. Remove a setting of pod.NodeName that was redundant with e2epod.SetNodeSelection().
(No other changes to the code)
Split the logic of creating the clients and the servers apart from the logic of checking which clients connect to which servers. Add some extra complexity to support additional use cases (like multiple endpoints on the same node).
(Identical to the PreferClose test.)
The existing TrafficDistribution test didn't really distinguish "same zone" from "same node". Add another test that makes sure there are at least 2 nodes in each zone so it can do that. (Keep the original test as well to avoid losing coverage in CI systems with single-schedulable-node-per-zone clusters.)
bertinatto
changed the title
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bertinatto, kevinrizza 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 |
|
/remove-label backports/unvalidated-commits |
openshift-ci
bot
removed
do-not-merge/hold
|
/retest-required |
1 similar comment
|
/retest-required |
|
/override ci/prow/e2e-aws-ovn-serial The last 2 jobs failed because of a known issue: https://redhat-internal.slack.com/archives/CDCP2LA9L/p1752619539151619?thread_ts=1752614736.467729&cid=CDCP2LA9L |
|
@bertinatto: Overrode contexts on behalf of bertinatto: ci/prow/e2e-aws-ovn-serial 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. |
|
@bertinatto: Overrode contexts on behalf of bertinatto: ci/prow/e2e-aws-ovn-serial 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. |
openshift-merge-bot
bot
merged commit b94367c
into
openshift:master
|
@bertinatto: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
Got it thanks! |
|
[ART PR BUILD NOTIFIER] Distgit: openshift-enterprise-pod |
|
[ART PR BUILD NOTIFIER] Distgit: ose-installer-kube-apiserver-artifacts |
|
[ART PR BUILD NOTIFIER] Distgit: openshift-enterprise-hyperkube |
|
[ART PR BUILD NOTIFIER] Distgit: kube-proxy |
|
[ART PR BUILD NOTIFIER] Distgit: openshift-enterprise-pod |
|
[ART PR BUILD NOTIFIER] Distgit: openshift-enterprise-hyperkube |
|
[ART PR BUILD NOTIFIER] Distgit: ose-installer-kube-apiserver-artifacts |
|
[ART PR BUILD NOTIFIER] Distgit: kube-proxy |
No description provided.