RomanNum3ral
/
k3s-ansible
mirror of https://github.com/timothystewart6/k3s-ansible.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix(metallb): guard cilium_bgp variable before evaluating 

- The 'Deploy metallb manifest' and 'Deploy metallb pool' conditionals evaluate
  'not cilium_bgp' directly, which raises an undefined-variable error when the
  k3s_server role runs without cilium_bgp in scope and no Cilium variables are set
- Guard with 'cilium_bgp is not defined' so the condition resolves cleanly when
  Cilium BGP is not configured
- Fixes #644
This commit is contained in:
Timothy Stewart 2026-08-02 13:51:53 -05:00
parent 6aea6e71b6
commit 7e1d62b866
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
roles/k3s_server/tasks/main.yml
View File

@ -27,7 +27,7 @@
- name: Deploy metallb manifest
ansible.builtin.include_tasks: metallb.yml
tags: metallb
when: kube_vip_lb_ip_range is not defined and (not cilium_bgp or cilium_iface is not defined)
when: kube_vip_lb_ip_range is not defined and (cilium_bgp is not defined or cilium_iface is not defined)
- name: Deploy kube-vip manifest
ansible.builtin.include_tasks: kube-vip.yml

2
roles/k3s_server_post/tasks/main.yml
View File

@ -12,7 +12,7 @@
- name: Deploy metallb pool
ansible.builtin.include_tasks: metallb.yml
tags: metallb
when: kube_vip_lb_ip_range is not defined and (not cilium_bgp or cilium_iface is not defined)
when: kube_vip_lb_ip_range is not defined and (cilium_bgp is not defined or cilium_iface is not defined)
- name: Remove tmp directory used for manifests
ansible.builtin.file: