Browse Source

Fix netchecker update side effect (#1644)

* Fix netchecker update side effect

kubectl apply should only be used on resources created
with kubectl apply. To workaround this, we should apply
the old manifest before upgrading it.

* Update 030_check-network.yml
pull/1627/head
Matthew Mosesohn 7 years ago
committed by GitHub
parent
commit
649388188b
2 changed files with 18 additions and 1 deletions
  1. 17
      roles/kubernetes-apps/ansible/tasks/netchecker.yml
  2. 2
      tests/testcases/030_check-network.yml

17
roles/kubernetes-apps/ansible/tasks/netchecker.yml

@ -1,4 +1,21 @@
---
- name: Kubernetes Apps | Check if netchecker-server manifest already exists
stat:
path: "{{ kube_config_dir }}/netchecker-server-deployment.yml.j2"
register: netchecker_server_manifest
tags: ['facts', 'upgrade']
- name: Kubernetes Apps | Apply netchecker-server manifest to update annotations
kube:
name: "netchecker-server"
namespace: "{{ netcheck_namespace }}"
kubectl: "{{bin_dir}}/kubectl"
resource: "deploy"
state: latest
when: inventory_hostname == groups['kube-master'][0] and netchecker_server_manifest.stat.exists
tags: upgrade
- name: Kubernetes Apps | Lay Down Netchecker Template
template:
src: "{{item.file}}"

2
tests/testcases/030_check-network.yml

@ -16,7 +16,7 @@
shell: "{{bin_dir}}/kubectl get pods --all-namespaces -owide"
register: get_pods
- debug: msg="{{get_pods.stdout}}"
- debug: msg="{{get_pods.stdout.split('\n')}}"
- name: Get pod names
shell: "{{bin_dir}}/kubectl get pods -o json"

Loading…
Cancel
Save