Skip to content
This repository has been archived by the owner on Jul 30, 2021. It is now read-only.

Certificate management using cert-manager #14

Closed
moshloop opened this issue Jul 3, 2019 · 5 comments
Closed

Certificate management using cert-manager #14

moshloop opened this issue Jul 3, 2019 · 5 comments
Milestone

Comments

@moshloop
Copy link
Contributor

moshloop commented Jul 3, 2019

While perhaps not for a 0.1 release (or even for this provider?), we should look at implementing certificate management as a first-class citizen.

Storing certs in the spec is bad, storing certs in secrets is better, using a CA to manage certs is best.

Ideally, there is a full PKI hierarchy with a single root CA with intermediary CA's per Cluster/Node.

This would have the following characteristics:

  1. Kubeconfig's could be generated that are valid for a) all clusters or b) a single cluster
  2. Cross-talk between clusters would be prevented
  3. Certificate generation could be moved to node boot phase (provides nodes can communicate with the management cluster) which has 2 major benefits:
    a) Keys are never transmitted over the wire or stored outside the node using them following PKI best practices
    b) user-data size would be reduced, and arbitrary certificates could be created.

See kubernetes-sigs/cluster-api-provider-aws#847 and kubernetes/kubeadm#1631

@ashish-amarnath
Copy link

/assign

@chuckha
Copy link
Contributor

chuckha commented Jul 12, 2019

/milestone Next

@k8s-ci-robot k8s-ci-robot added this to the Next milestone Jul 12, 2019
@texascloud
Copy link

I like this suggestion, but I don't think this should be tied to this bootstrap provider. To decide where it should belong, questions like "what other types of bootstrap providers may be created in the future/are being started now? do those providers have a need for cert management too?"

@ashish-amarnath ashish-amarnath removed their assignment Sep 26, 2019
@chuckha
Copy link
Contributor

chuckha commented Oct 10, 2019

we are starting to see some CAPI cert-manager adoption, but I don't think this is the right repository for it. Let's revisit if there is more evidence that this is the right place for cert-manager. Alternatively it would be really cool if you could open a new issue that shows how cert-manager fits into this repo and we can discuss there.

/close

@k8s-ci-robot
Copy link
Contributor

@chuckha: Closing this issue.

In response to this:

we are starting to see some CAPI cert-manager adoption, but I don't think this is the right repository for it. Let's revisit if there is more evidence that this is the right place for cert-manager. Alternatively it would be really cool if you could open a new issue that shows how cert-manager fits into this repo and we can discuss there.

/close

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.

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

No branches or pull requests

5 participants