CCCHH/ansible-infra
CCCHH/ansible-infra
2
2
Fork 1
Code Issues 10 Pull requests 9 Wiki Activity Actions

WIP: new z9 ccchh router #98

Draft
bitwhisker wants to merge 23 commits from new_ccchh_router into main
pull from: new_ccchh_router
merge into: CCCHH:main
Conversation 2 Commits 23 Files changed 52 +1 -1
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit dc29dd8b13 - Show all commits

kea_dhcp(role): fix include wars check

Prev Next
bitwhisker 2026-06-21 20:01:55 +02:00
commit dc29dd8b13
Signed by: bitwhisker
SSH key fingerprint: SHA256:KybIk/tusSKao6eLGY+ILlFa1rCrzwx66/acBAcKUqE

2
roles/kea_dhcp/tasks/kea.yaml
View file

@ -1,6 +1,6 @@
--- ---
- name: Include config vars - name: Include config vars
when: kea_dhcp__include_vars is not None when: kea_dhcp__include_vars != None
bitwhisker marked this conversation as resolved Outdated
june commented 2026-05-24 18:30:07 +02:00
Outdated
Copy link

I would rather just stick to only having tags at the playbook level. I don't think this kind granular control for the role is really needed as it the config files also shouldn't trigger a reload, if not changed.

I would rather just stick to only having tags at the playbook level. I don't think this kind granular control for the role is really needed as it the config files also shouldn't trigger a reload, if not changed.
bitwhisker commented 2026-05-24 21:51:03 +02:00
Outdated
Copy link

this was just copied from the fux noc ansible, but I can remove it

this was just copied from the fux noc ansible, but I can remove it
👍 1
ansible.builtin.include_vars: ansible.builtin.include_vars:
file: "{{ kea_dhcp__include_vars }}" file: "{{ kea_dhcp__include_vars }}"