Skip to content
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

[1.30] Remove deprecated kublet flag pod-infra-container-image #7408

Closed
dereknola opened this issue May 2, 2023 · 8 comments
Closed

[1.30] Remove deprecated kublet flag pod-infra-container-image #7408

dereknola opened this issue May 2, 2023 · 8 comments
Assignees
Milestone

Comments

@dereknola
Copy link
Member

dereknola commented May 2, 2023

Is your feature request related to a problem? Please describe.

image

Running K3s with --pause-image=:"rancher/mirrored-pause:3.5" produces logs with:

Flag --pod-infra-container-image has been deprecated, will be removed in a future release. Image garbage collector will     get sandbox image information from CRI.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

kubernetes/kubernetes#106893

@dereknola
Copy link
Member Author

The upstream issue is still tracking as incomplete.

@caroline-suse-rancher caroline-suse-rancher added this to the Backlog milestone Oct 11, 2023
@dereknola dereknola changed the title Remove deprecated kublet flag pod-infra-container-image [1.29] Remove deprecated kublet flag pod-infra-container-image Oct 24, 2023
@dereknola
Copy link
Member Author

Update: Upstream is on track to no-op the argument in 1.29, with remove in 1.30+

@dereknola dereknola modified the milestones: Backlog, v1.29.0+k3s1 Dec 13, 2023
@dereknola dereknola changed the title [1.29] Remove deprecated kublet flag pod-infra-container-image [1.30] Remove deprecated kublet flag pod-infra-container-image Dec 20, 2023
@dereknola
Copy link
Member Author

Upstream bumping it out to "future release" during 1.27 development. kubernetes/kubernetes@a1def4b

Later it was supposed to be 1.29, but cri-dockerd was still not ready, so now its 1.30

@afbjorklund
Copy link

The flag doesn't do anything in 1.29, so the kubelet now tries to garbage collect the pause image for docker...

But most of the time, there is some pod or other still using it - so that won't make much of a difference in practice.

@brandond
Copy link
Member

The pause image is supposed to get pinned via CRI if the runtime supports it. I'm not sure that cri-dockerd does yet though.

@afbjorklund
Copy link

I made a quick hack for it, but it got stuck in CI due to the missing API (since it only did k8s 1.22 at the time)

@brandond
Copy link
Member

Thanks! If upstream doesn't merge that, we might pull it into our fork.

@rancher-max
Copy link
Contributor

Validated using v1.30.0-rc2+k3s1 and v1.30.0-rc2+rke2r1

This log warning is no longer present. See only deprecation warning remaining:

$ journalctl -eu k3s | grep -i deprecated
May 08 19:14:42 max1 k3s[4500]: Flag --containerd has been deprecated, This is a cadvisor flag that was mistakenly registered with the Kubelet. Due to legacy concerns, it will follow the standard CLI deprecation timeline before being removed.

config.yaml used:

write-kubeconfig-mode: 644
selinux: true
cluster-init: true
pause-image: rancher/mirrored-pause:3.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Closed
Archived in project
Development

No branches or pull requests

6 participants