Skip to content

Commit

Permalink
avoid variable conflict
Browse files Browse the repository at this point in the history
  • Loading branch information
Tcharl committed Aug 18, 2023
1 parent 5b14cb1 commit 5744d20
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 15 deletions.
Binary file modified .tox/.tmp/package/56/tcharl_freeipa_server-0.1.0.tar.gz
Binary file not shown.
18 changes: 9 additions & 9 deletions tasks/facts.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@
ansible.builtin.setup:
gather_subset:
- network
register: current_host_facts
register: freeipa_server_current_host_facts
delegate_facts: true

- name: Facts | debug network facts
ansible.builtin.debug:
var: current_host_facts
var: freeipa_server_current_host_facts
verbosity: 2

- name: Facts | debug preferred_nic
Expand All @@ -19,20 +19,20 @@

- name: Facts | debug preferred_nic
ansible.builtin.debug:
msg: "{{ preferred_nic | default(current_host_facts.ansible_facts.ansible_default_ipv4.interface) }}"
register: idm_preferred_nic
msg: "{{ preferred_nic | default(freeipa_server_current_host_facts.ansible_facts.ansible_default_ipv4.interface) }}"
register: freeipa_server_idm_nic

- name: Facts | compute ip if preferred_nic is not set
ansible.builtin.debug:
msg: "{{ current_host_facts.ansible_facts['ansible_' + idm_preferred_nic.msg | replace('-', '_')].ipv4.address if idm_preferred_nic.msg is defined else current_host_facts.ansible_facts.ansible_default_ipv4.address | default(current_host_facts.ansible_facts.ansible_all_ipv4_addresses | first) }}"
register: securehost_current_host_ip
msg: "{{ freeipa_server_current_host_facts.ansible_facts['ansible_' + freeipa_server_idm_nic.msg | replace('-', '_')].ipv4.address if freeipa_server_idm_nic.msg is defined else freeipa_server_current_host_facts.ansible_facts.ansible_default_ipv4.address | default(freeipa_server_current_host_facts.ansible_facts.ansible_all_ipv4_addresses | first) }}"
register: freeipa_server_current_host_ip

- name: Facts | retrieve current hostname
ansible.builtin.command: "hostname"
changed_when: False
register: securehost_current_hostname
register: freeipa_server_current_hostname

- name: Facts | compute server hostname
ansible.builtin.debug:
msg: "{{ hostname if hostname is defined and hostname.endswith(company_domain) else hostname + '.' + company_domain if hostname is defined else securehost_current_hostname.stdout if securehost_current_hostname.stdout.endswith(company_domain) else idm_server_default_domain_prefix + '.' + company_domain }}"
register: securehost_freeipa_server_host
msg: "{{ hostname if hostname is defined and hostname.endswith(company_domain) else hostname + '.' + company_domain if hostname is defined else freeipa_server_current_hostname.stdout if freeipa_server_current_hostname.stdout.endswith(company_domain) else idm_server_default_domain_prefix + '.' + company_domain }}"
register: freeipa_server_freeipa_server_host
12 changes: 6 additions & 6 deletions tasks/freeipa-server.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@
ansible.builtin.include_role:
name: tcharl.ansible_hostname
vars:
hostname: "{{ securehost_freeipa_server_host.msg }}"
hostname: "{{ freeipa_server_freeipa_server_host.msg }}"
hostname_reboot: false
ansible_become: true
preferred_nic: "{{ idm_preferred_nic.msg }}"
preferred_nic: "{{ freeipa_server_idm_nic.msg }}"
when:
- securehost_freeipa_server_host.msg == idm_server_default_domain_prefix + '.' + company_domain
- freeipa_server_freeipa_server_host.msg == idm_server_default_domain_prefix + '.' + company_domain

- name: Freeipa-server | opens freeipa services firewall
ansible.builtin.include_role:
Expand All @@ -18,7 +18,7 @@
firewalld_zones:
- name: "{{ firewalld_zone }}"
nics:
- "{{ idm_preferred_nic.msg }}"
- "{{ freeipa_server_idm_nic.msg }}"
enabled_services:
- service: https
#- service: freeipa-ldap
Expand All @@ -32,8 +32,8 @@
vars:
ipaserver_firewalld_zone: "{{ firewalld_zone }}"
ipaserver_auto_reverse: True
ipaserver_hostname: "{{ securehost_freeipa_server_host.msg }}"
ipaserver_ip_addresses: "{{ [securehost_current_host_ip.msg] }}"
ipaserver_hostname: "{{ freeipa_server_freeipa_server_host.msg }}"
ipaserver_ip_addresses: "{{ [freeipa_server_current_host_ip.msg] }}"
ipaserver_domain: "{{ company_domain }}"
ipaserver_realm: "{{ company_domain | upper }}"
allow_zone_overlap: "{{ allow_dns_zone_overlap | bool }}"
Expand Down

0 comments on commit 5744d20

Please sign in to comment.