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
Install-WorkerServices should not set pod-infra-container-image as it is marked for deprecation #108144
Comments
@SergeyKanzhelev: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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. |
@SergeyKanzhelev this has one effect currently. preventing the specified image from being garbage collected. (So we tell folks to specify the pause image in the containerd config toml and here - need to do both) |
Rephrased. Thank you! |
/assign |
@dims and @SergeyKanzhelev Could you please inform about where |
/remove-sig cloud-provider |
I'd say function |
@SergeyKanzhelev Please inform that am I understanding correct -
to function Am I correct, please inform if I am wrong in the comments of the PR. Also I am not sure about how we will use this change inside the function |
Yeah, it's here https://github.com/kubernetes/kubernetes/blob/master/cluster/gce/windows/k8s-node-setup.psm1#L1435 Tests covering this are the [sig-windows] suite ones, that creates Windows pools. For example, jobs like this have containerd configured to be used (KUBE_WINDOWS_CONTAINER_RUNTIME=containerd): https://k8s-testgrid.appspot.com/sig-windows-master-release#gce-windows-2019-containerd-master |
@ibabou I am not understanding the conversation about the tests, actually I have not worked much in this domain. Could you please elaborate on what we are trying to achieve now and what steps we should take to achieve that. Please make a comment in the PR made. Apologies for asking such a vague question. |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
@SergeyKanzhelev / @ibabou should we close this issue in favor of the one linked above? |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
/remove-lifecycle rotten either one of these two issues should be tracking this. |
Yes
No, in the |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /close not-planned |
@k8s-triage-robot: Closing this issue, marking it as "Not Planned". 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 happened?
This code:
kubernetes/cluster/gce/windows/k8s-node-setup.psm1
Line 1051 in 6644566
should not set
pod-infra-container-image
as it was just marked for deprecation and has to set thesandbox_image
instead on ContianerdWhat did you expect to happen?
code not setting this flag
How can we reproduce it (as minimally and precisely as possible)?
code issue
Anything else we need to know?
/sig windows
/sig cloud-provider
Kubernetes version
master
Cloud provider
any
OS version
Install tools
Container runtime (CRI) and and version (if applicable)
Related plugins (CNI, CSI, ...) and versions (if applicable)
The text was updated successfully, but these errors were encountered: