-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
test/e2e_kubeadm: fix matching UnversionedKubeletConfigMap defaults #108127
test/e2e_kubeadm: fix matching UnversionedKubeletConfigMap defaults #108127
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: neolit123 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 |
/triage accepted |
/hold need to revist as something is not right |
The kubeadm defaults in features.go differ between versions. e2e_kubeadm tests cannot import the kubeadm features.go, or easily detect the versioned of the kubeadm binary used to create the cluster. Check for the existence of both versioned and unversioned objects independent of the value of the FG. Once the FG goes GA only the unversioned objects should be checked. Without this change kubeadm e2e skew tests will fail where kubeadm is at 1.24 (has the FG defaulted to true), the FG is not explicitly set by the user and the k8s version is at 1.23.
e4f932b
to
2fba7c8
Compare
kubeletConfigConfigMapResource = &authv1.ResourceAttributes{ | ||
Namespace: kubeSystemNamespace, | ||
Name: "", | ||
Resource: "configmaps", | ||
Verb: "get", | ||
} |
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.
this object was unused
if err != nil { | ||
cm, err = f.ClientSet.CoreV1(). | ||
ConfigMaps(kubeSystemNamespace). | ||
Get(context.TODO(), kubeletConfigConfigMapNameVersioned, metav1.GetOptions{}) |
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.
just remove lines like this after the feature gate is removed.
/lgtm
hadn't thought about the kubeadm n-1 kubelet skew when reviewing last PR.
/unhold
/retest |
1 similar comment
/retest |
…08127-origin-release-1.23 Automated cherry pick of #108127: test/e2e_kubeadm: fix matching UnversionedKubeletConfigMap
/kind failing-test
What this PR does / why we need it:
The kubeadm defaults in features.go differ between versions.
e2e_kubeadm tests cannot import the kubeadm features.go,
or easily detect the versioned of the kubeadm binary used
to create the cluster.
Check for the existence of both versioned and unversioned
objects independent of the value of the FG. Once the FG
goes GA only the unversioned objects should be checked.
Without this change kubeadm e2e skew tests will fail where kubeadm
is at 1.24 (has the FG defaulted to true), the FG is not
explicitly set by the user and the k8s version is at 1.23.
Which issue(s) this PR fixes:
xref kubernetes/kubeadm#1582 (comment)
https://k8s-testgrid.appspot.com/sig-cluster-lifecycle-kubeadm#kubeadm-kinder-latest-on-1-23
Special notes for your reviewer:
need to backport this to 1.23. technically we only need to fix the 1.23 branch, but having this in master first is accurate.
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: