Skip to content

Commit

Permalink
Fix etcd deployment type variable location (kubernetes-sigs#5587)
Browse files Browse the repository at this point in the history
On deployments types where etcd server is splitted from Kube Master, the deployment fails since it cannot find the variable.
  • Loading branch information
fabianotessarolo authored Feb 7, 2020
1 parent 422b25a commit 16fd2e5
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 3 deletions.
7 changes: 6 additions & 1 deletion docs/cri-o.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,14 @@ skip_downloads: false
## k8s-cluster.yml
```yaml
etcd_deployment_type: host
kubelet_deployment_type: host
container_manager: crio
```
## etcd.yml
```yaml
etcd_deployment_type: host
```
[CRI-O]: https://cri-o.io/
4 changes: 4 additions & 0 deletions inventory/sample/group_vars/etcd.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
---
## Etcd auto compaction retention for mvcc key value store in hour
# etcd_compaction_retention: 0

Expand All @@ -16,3 +17,6 @@
### ETCD: disable peer client cert authentication.
# This affects ETCD_PEER_CLIENT_CERT_AUTH variable
# etcd_peer_client_auth: true

## Settings for etcd deployment type
etcd_deployment_type: docker
3 changes: 1 addition & 2 deletions inventory/sample/group_vars/k8s-cluster/k8s-cluster.yml
Original file line number Diff line number Diff line change
Expand Up @@ -158,8 +158,7 @@ dns_domain: "{{ cluster_name }}"
## docker for docker, crio for cri-o and containerd for containerd.
container_manager: docker

## Settings for containerized control plane (etcd/kubelet/secrets)
etcd_deployment_type: docker
## Settings for containerized control plane (kubelet/secrets)
kubelet_deployment_type: host
helm_deployment_type: host

Expand Down

0 comments on commit 16fd2e5

Please sign in to comment.