amazon-vpc-cni-k8s icon indicating copy to clipboard operation
amazon-vpc-cni-k8s copied to clipboard

Any plans on dropping dockershim?

Open njuettner opened this issue 3 years ago • 11 comments

Hey folks 👋🏻 ,

we're currently investigating CAPI/CAPA and we'd like to know if there's a plan to drop dockershim from aws-cni? We'd like to drop docker completely and use containerd. I saw there was already an issue created ~1 year old but I'd like to know if anything has changed so far.

DaemonSet Volume Mount:

        - mountPath: /var/run/dockershim.sock
          name: dockershim

Ref.: https://github.com/aws/amazon-vpc-cni-k8s/issues/971

njuettner avatar Mar 04 '21 09:03 njuettner

Hi @njuettner,

We are planning on removing the dockershim mount in release v1.8.

This is the PR that removes the mount: #1352.

couralex6 avatar Mar 04 '21 17:03 couralex6

We have announced support to containerd as an option in EKS1.21 release. Please refer to this document.

haouc avatar Nov 24 '21 17:11 haouc

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see. If you need more assistance, please open a new issue that references this one. If you wish to keep having a conversation with other community members under this issue feel free to do so.

github-actions[bot] avatar Nov 24 '21 17:11 github-actions[bot]

@haouc that is really nice you supporting containerd for EKS but what if someone don't use EKS, you can also use CAPA without any EKS. The PR @couralex6 mentioned is still not merged, so I'm a bit confused why you're closing the issue.

Maybe I'm missing something, but I'd appreciate if someone could shed some light 🙂

njuettner avatar Nov 24 '21 17:11 njuettner

@njuettner You are right. My bad. I will reopen this issue. We are still working to make a clean path for customers to migrate to dockershim independent AWS VPC CNI.

We haven't concluded the release yet. I will provide updates here.

haouc avatar Nov 24 '21 18:11 haouc

@haouc any updates here 🙂?

njuettner avatar Jan 21 '22 10:01 njuettner

Hey @njuettner thanks for checking on this. Yes, we are working on this currently.

haouc avatar Jan 21 '22 23:01 haouc

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days

github-actions[bot] avatar Jun 18 '22 00:06 github-actions[bot]

(noise comment to reset the stale counter)

anguslees avatar Jun 20 '22 13:06 anguslees

Hey folks, Do we have any way to migrate to contained from dockershim without facing any downtime? we are managing our k8s cluster using kops.

ankitjain28may avatar Aug 17 '22 18:08 ankitjain28may

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days

github-actions[bot] avatar Oct 17 '22 00:10 github-actions[bot]

v1.12 release is out and dockershim is removed from manifest. Closing this issue.

jayanthvn avatar Oct 28 '22 16:10 jayanthvn

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see. If you need more assistance, please open a new issue that references this one. If you wish to keep having a conversation with other community members under this issue feel free to do so.

github-actions[bot] avatar Oct 28 '22 16:10 github-actions[bot]