Navigation Menu

Skip to content

Commit

Permalink
Revert "Fix #4237: update kube cert path (#4354)" (#4369)
Browse files Browse the repository at this point in the history
This reverts commit ea7a6f1.

This change modified the certs dir for Kubernetes, but did not move the directories for existing clusters.
  • Loading branch information
mattymo authored and k8s-ci-robot committed Mar 20, 2019
1 parent e640233 commit ec08303
Show file tree
Hide file tree
Showing 9 changed files with 11 additions and 15 deletions.
8 changes: 4 additions & 4 deletions contrib/terraform/openstack/README.md
Expand Up @@ -412,13 +412,13 @@ sudo route add -net [internal-subnet]/24 gw [router-ip]
```
3. List Kubernetes certificates & keys:
```
ssh [os-user]@[master-ip] sudo ls /etc/kubernetes/pki/
ssh [os-user]@[master-ip] sudo ls /etc/kubernetes/ssl/
```
4. Get `admin`'s certificates and keys:
```
ssh [os-user]@[master-ip] sudo cat /etc/kubernetes/pki/admin-kube-master-k8s-master-1-key.pem > admin-key.pem
ssh [os-user]@[master-ip] sudo cat /etc/kubernetes/pki/admin-kube-master-k8s-master-1.pem > admin.pem
ssh [os-user]@[master-ip] sudo cat /etc/kubernetes/pki/ca.pem > ca.pem
ssh [os-user]@[master-ip] sudo cat /etc/kubernetes/ssl/admin-kube-master-1-key.pem > admin-key.pem
ssh [os-user]@[master-ip] sudo cat /etc/kubernetes/ssl/admin-kube-master-1.pem > admin.pem
ssh [os-user]@[master-ip] sudo cat /etc/kubernetes/ssl/ca.pem > ca.pem
```
5. Configure kubectl:
```ShellSession
Expand Down
2 changes: 1 addition & 1 deletion contrib/vault/roles/vault/defaults/main.yml
Expand Up @@ -114,7 +114,7 @@ vault_client_headers:
Content-Type: "application/json"

etcd_cert_dir: /etc/ssl/etcd/ssl
kube_cert_dir: /etc/kubernetes/pki
kube_cert_dir: /etc/kubernetes/ssl

vault_pki_mounts:
userpass:
Expand Down
4 changes: 2 additions & 2 deletions contrib/vault/vault.md
Expand Up @@ -76,8 +76,8 @@ generated elsewhere, you'll need to copy the certificate and key to the hosts in
* ``/etc/ssl/etcd/ssl/ca.pem``
* ``/etc/ssl/etcd/ssl/ca-key.pem``
* kubernetes:
* ``/etc/kubernetes/pki/ca.pem``
* ``/etc/kubernetes/pki/ca-key.pem``
* ``/etc/kubernetes/ssl/ca.pem``
* ``/etc/kubernetes/ssl/ca-key.pem``

Additional Notes:

Expand Down
4 changes: 1 addition & 3 deletions inventory/sample/group_vars/k8s-cluster/k8s-cluster.yml
Expand Up @@ -8,9 +8,7 @@ kube_script_dir: "{{ bin_dir }}/kubernetes-scripts"
kube_manifest_dir: "{{ kube_config_dir }}/manifests"

# This is where all the cert scripts and certs will be located
# For old version of k8s next line should be used instead
# kube_cert_dir: "{{ kube_config_dir }}/ssl"
kube_cert_dir: "{{ kube_config_dir }}/pki"
kube_cert_dir: "{{ kube_config_dir }}/ssl"

# This is where all of the bearer tokens will be stored
kube_token_dir: "{{ kube_config_dir }}/tokens"
Expand Down
1 change: 0 additions & 1 deletion roles/kubernetes/client/defaults/main.yml
Expand Up @@ -4,5 +4,4 @@ kubectl_localhost: false
artifacts_dir: "{{ inventory_dir }}/artifacts"

kube_config_dir: "/etc/kubernetes"
kube_cert_dir: "{{ kube_config_dir }}/pki"
kube_apiserver_port: "6443"
2 changes: 1 addition & 1 deletion roles/kubernetes/client/tasks/main.yml
Expand Up @@ -49,7 +49,7 @@
kubeconfig user
--client-name kubernetes-admin
--org system:masters
--cert-dir {{ kube_cert_dir }}
--cert-dir {{ kube_config_dir }}/ssl
--apiserver-advertise-address {{ external_apiserver_address }}
--apiserver-bind-port {{ external_apiserver_port }}
run_once: yes
Expand Down
2 changes: 1 addition & 1 deletion roles/kubernetes/master/tasks/kubeadm-setup.yml
Expand Up @@ -71,7 +71,7 @@
tags: facts

- name: kubeadm | Copy etcd cert dir under k8s cert dir
command: "cp -TR {{ etcd_cert_dir }} {{ kube_cert_dir }}/etcd"
command: "cp -TR {{ etcd_cert_dir }} {{ kube_config_dir }}/ssl/etcd"
changed_when: false

- name: Create audit-policy directory
Expand Down
1 change: 0 additions & 1 deletion roles/kubernetes/preinstall/defaults/main.yml
Expand Up @@ -25,7 +25,6 @@ disable_ipv6_dns: false

kube_cert_group: kube-cert
kube_config_dir: /etc/kubernetes
kube_cert_dir: "{{ kube_config_dir }}/pki"

# Container Linux by CoreOS cloud init config file to define /etc/resolv.conf content
# for hostnet pods and infra needs
Expand Down
2 changes: 1 addition & 1 deletion roles/kubespray-defaults/defaults/main.yaml
Expand Up @@ -93,7 +93,7 @@ kube_script_dir: "{{ bin_dir }}/kubernetes-scripts"
kube_manifest_dir: "{{ kube_config_dir }}/manifests"

# This is where all the cert scripts and certs will be located
kube_cert_dir: "{{ kube_config_dir }}/pki"
kube_cert_dir: "{{ kube_config_dir }}/ssl"

# This is where all of the bearer tokens will be stored
kube_token_dir: "{{ kube_config_dir }}/tokens"
Expand Down

0 comments on commit ec08303

Please sign in to comment.