Skip to content
This repository was archived by the owner on Jun 29, 2022. It is now read-only.

Commit 44420ef

Browse files
committed
apiserver: Use private key for service-account-key-file
For the apiserver flag `--service-account-key-file` we use to provide public service-account key. Now this commit provides the private service-account key, the same as the one provided to `--service-account-signing-key-file`. Since there is no need of the public key this commit also removes the whole pipeline that embedded the public key into the kube-apiserver secret. Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
1 parent ce3635d commit 44420ef

File tree

6 files changed

+2
-6
lines changed

6 files changed

+2
-6
lines changed

assets/charts/control-plane/kube-apiserver/templates/_helpers.tpl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939
- --kubelet-client-key=/etc/kubernetes/secrets/apiserver.key
4040
- --kubelet-preferred-address-types=InternalIP,ExternalIP,Hostname
4141
- --secure-port=6443
42-
- --service-account-key-file=/etc/kubernetes/secrets/service-account.pub
42+
- --service-account-key-file=/etc/kubernetes/secrets/service-account.key
4343
- --service-account-signing-key-file=/etc/kubernetes/secrets/service-account.key
4444
- --service-account-issuer=https://kubernetes.default.svc
4545
- --service-cluster-ip-range={{ .Values.apiserver.serviceCIDR }}

assets/charts/control-plane/kube-apiserver/templates/kube-apiserver-secret.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
{{- define "secrets" }}
22
apiserver.key: "{{ .Values.apiserver.apiserverKey }}"
33
apiserver.crt: "{{ .Values.apiserver.apiserverCert }}"
4-
service-account.pub: "{{ .Values.apiserver.serviceAccountPub }}"
54
service-account.key: "{{ .Values.apiserver.serviceAccountPrivate }}"
65
ca.crt: "{{ .Values.apiserver.caCert }}"
76
etcd-client-ca.crt: "{{ .Values.apiserver.etcdClientCaCert }}"

assets/charts/control-plane/kube-apiserver/values.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
apiserver:
22
apiserverKey:
33
apiserverCert:
4-
serviceAccountPub:
54
serviceAccountPrivate:
65
caCert:
76
etcdClientCaCert:

assets/terraform-modules/bootkube/assets.tf

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@ resource "local_file" "kube-apiserver" {
3939
ca_cert = base64encode(tls_self_signed_cert.kube-ca.cert_pem)
4040
apiserver_key = base64encode(tls_private_key.apiserver.private_key_pem)
4141
apiserver_cert = base64encode(tls_locally_signed_cert.apiserver.cert_pem)
42-
serviceaccount_pub = base64encode(tls_private_key.service-account.public_key_pem)
4342
serviceaccount_private = base64encode(tls_private_key.service-account.private_key_pem)
4443
etcd_ca_cert = base64encode(tls_self_signed_cert.etcd-ca.cert_pem)
4544
etcd_client_cert = base64encode(tls_locally_signed_cert.client.cert_pem)

assets/terraform-modules/bootkube/resources/bootstrap-manifests/bootstrap-apiserver.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ spec:
3838
- --kubelet-client-key=/etc/kubernetes/secrets/apiserver.key
3939
- --kubelet-preferred-address-types=InternalIP,ExternalIP,Hostname
4040
- --secure-port=6443
41-
- --service-account-key-file=/etc/kubernetes/secrets/service-account.pub
41+
- --service-account-key-file=/etc/kubernetes/secrets/service-account.key
4242
- --service-account-signing-key-file=/etc/kubernetes/secrets/service-account.key
4343
- --service-account-issuer=https://kubernetes.default.svc
4444
- --service-cluster-ip-range=${service_cidr}

assets/terraform-modules/bootkube/resources/charts/kube-apiserver.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
apiserver:
22
apiserverKey: ${apiserver_key}
33
apiserverCert: ${apiserver_cert}
4-
serviceAccountPub: ${serviceaccount_pub}
54
serviceAccountPrivate: ${serviceaccount_private}
65
caCert: ${ca_cert}
76
etcdClientCaCert: ${etcd_ca_cert}

0 commit comments

Comments
 (0)