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

Ability to add sidecar containers to static pod manifests #1490

Closed
dsexton opened this issue Apr 5, 2019 · 3 comments
Closed

Ability to add sidecar containers to static pod manifests #1490

dsexton opened this issue Apr 5, 2019 · 3 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.
Milestone

Comments

@dsexton
Copy link

dsexton commented Apr 5, 2019

Is this a BUG REPORT or FEATURE REQUEST?

FEATURE REQUEST

It would be helpful if kubeadm had the ability add additional containers to the generated static pod manifests. For example, adding the aws-encryption-provider to the apiserver pod.

Example:

apiVersion: kubeadm.k8s.io/v1beta1
kind: ClusterConfiguration
apiServer:
  extraArgs:
    cloud-provider: "aws"
  extraContainers:
    - name: aws-encryption-provider
      image: k8s.gcr.io/aws-encryption-provider:v0.1.0
      resources:
        limits:
          memory: "200Mi"
        requests:
          memory: "100Mi"

You could have a fixed one-off just for encryption provider support, however having this be generic could provide a solution to other use cases down the road.

Misc References

/cc @chuckha @sethpollack

@neolit123 neolit123 added the kind/feature Categorizes issue or PR as related to a new feature. label Apr 8, 2019
@neolit123
Copy link
Member

  • the configuration format is already locked for 1.15 (v1beta2) feature wise.
  • it's already possible to do this using kubeadm phases:
# write your own CP component manifest files in /etc/kubernetes
kubeadm init --skip-phases=control-plane
  • such a cluster with side cars to the CP components might be unsupported in upgrades.

@kubernetes/sig-cluster-lifecycle

@neolit123 neolit123 added the priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. label Apr 11, 2019
@fabriziopandini
Copy link
Member

IMO this issue should be merged with discussion on #1379
/release Next

@fabriziopandini fabriziopandini modified the milestones: v1.15, Next Apr 11, 2019
@timothysc
Copy link
Member

Closing and deferring to the other related issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.
Projects
None yet
Development

No branches or pull requests

4 participants