Browse Source

Merge pull request #691 from adidenko/calico-old-cni-fix

Fix possible problems with legacy calicoctl
pull/700/head
Bogdan Dobrelya 8 years ago
committed by GitHub
parent
commit
710d5ae48e
2 changed files with 7 additions and 5 deletions
  1. 10
      roles/network_plugin/calico/tasks/main.yml
  2. 2
      roles/network_plugin/calico/templates/cni-calico.conf.j2

10
roles/network_plugin/calico/tasks/main.yml

@ -1,4 +1,9 @@
---
- name: Calico | Check calicoctl version
run_once: true
set_fact:
legacy_calicoctl: "{{ calicoctl_image_tag | version_compare('v1.0.0', '<') }}"
- name: Calico | Write Calico cni config
template:
src: "cni-calico.conf.j2"
@ -71,11 +76,6 @@
delegate_to: "{{groups['etcd'][0]}}"
run_once: true
- name: Calico | Check calicoctl version
run_once: true
set_fact:
legacy_calicoctl: "{{ calicoctl_image_tag | version_compare('v1.0.0', '<') }}"
- name: Calico | Configure calico network pool
shell: >
echo '{

2
roles/network_plugin/calico/templates/cni-calico.conf.j2

@ -1,6 +1,8 @@
{
"name": "calico-k8s-network",
{% if not legacy_calicoctl %}
"hostname": "{{ inventory_hostname }}",
{% endif %}
"type": "calico",
"etcd_endpoints": "{{ etcd_access_endpoint }}",
"etcd_cert_file": "{{ etcd_cert_dir }}/node.pem",

Loading…
Cancel
Save