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

Why not a kube-proxy chart? #154

Closed
clayvan opened this issue Jun 3, 2020 · 8 comments
Closed

Why not a kube-proxy chart? #154

clayvan opened this issue Jun 3, 2020 · 8 comments

Comments

@clayvan
Copy link

clayvan commented Jun 3, 2020

EKS is one of the few managed Kubernetes platforms that doesn't provide kube-proxy as a part of the control plane upgrade, as we still have to manage/upgrade kube-proxy during EKS upgrades.

I was surprised to find 0 public helm charts for kube-proxy, and I think it's because it's pretty specific to EKS.

In the meantime I'll create my own. If you think it's a good idea, I'll PR it up to this repo, but was more curious as to why no-one else has one.

@max-rocket-internet
Copy link
Contributor

I don't know of a specific reason why there's no chart but my impression is that other than updating the image, there's not much else to for an upgrade.

@mogren
Copy link
Contributor

mogren commented Jun 10, 2020

@clayvan the task to manage addons is tracked here aws/containers-roadmap#252

@TBBle
Copy link

TBBle commented Jun 15, 2020

I'm pretty sure kube-proxy doesn't have a chart because everyone is using the Daemonset deployed by kubeadm. I haven't checked my EKS cluster, but I'd be surprised if there was anything different in its deployment except using an ECR-hosted image.

It'd be a pretty simple Helm chart to create, but it's technically not eks-specific, and since helm/charts ("stable") is being deprecated it's not obvious where it should live. Basically, who wants to maintain it?

@sc250024
Copy link

sc250024 commented Jul 14, 2020

I'd totally be for this, coupled with an adoption strategy via Helm. I opened the issue (aws/containers-roadmap#923) because of this exact problem.

@stevehipwell
Copy link

What would be peoples thoughts be on a PR for an EKS specific chart for kube-proxy to be added to this repo? Our use case would be to remove the default config at cluster creation and manage it via Helm thereafter, all in Terraform. The EKS specifics would be around the image selection based on cluster version and region inputs as well as taints etc for Fargate.

@sc250024
Copy link

What would be peoples thoughts be on a PR for an EKS specific chart for kube-proxy to be added to this repo? Our use case would be to remove the default config at cluster creation and manage it via Helm thereafter, all in Terraform. The EKS specifics would be around the image selection based on cluster version and region inputs as well as taints etc for Fargate.

Yes! This is especially the case since the above issue (aws/containers-roadmap#923) has apparently been considered, and will be released at some point.

@stevehipwell
Copy link

@sc250024 I've not seen any meaningful movement on aws/containers-roadmap#923 which is why I'm looking at adding the Helm chart myself. Having to use shell scripts to manipulate kube-proxy and core-dns doesn't go well with infrastructure automation.

@bryantbiggs
Copy link
Member

thank you for the issue - however, there are no plans to add a kube-proxy chart at this time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants