Skip to content

Commit 3c4e246

Browse files
authored
Fix/template integer handling (#183)
* add clause to catch integer values in config
1 parent 68c2973 commit 3c4e246

File tree

8 files changed

+60
-21
lines changed

8 files changed

+60
-21
lines changed

molecule/default/collections.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
collections:
22
- name: community.mysql
33
- name: community.docker
4-
version: ">=1.9.0,<2.0.0"
54
- name: community.general
65
version: ">=2,<3"

molecule/default/converge.yml

Lines changed: 27 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,13 @@
33
- name: Converge
44
hosts: all
55
vars:
6+
icingaweb2_db:
7+
type: mysql
8+
name: icingaweb
9+
host: 127.0.0.1
10+
user: icingaweb
11+
password: icingaweb
12+
icingaweb2_database_import_schema: true
613
icingadb_database_import_schema: true
714
mysql_innodb_file_format: barracuda
815
mysql_innodb_large_prefix: 1
@@ -12,12 +19,17 @@
1219
- mariadb-server
1320
- python3-mysqldb
1421
mysql_users:
22+
- name: icingaweb
23+
host: "%"
24+
password: icingaweb
25+
priv: "icingaweb.*:ALL"
1526
- name: icingadb
1627
host: "%"
1728
password: icingadb
1829
priv: "icingadb.*:ALL"
1930
mysql_databases:
2031
- name: icingadb
32+
- name: icingaweb
2133
icinga2_confd: false
2234
icinga2_features:
2335
- name: perfdata
@@ -61,9 +73,18 @@
6173
- zones.d/main/services
6274
collections:
6375
- icinga.icinga
64-
roles:
65-
- repos
66-
- icinga2
67-
- geerlingguy.mysql
68-
- icingadb
69-
- icingadb_redis
76+
pre_tasks:
77+
- include_role:
78+
name: repos
79+
- include_role:
80+
name: geerlingguy.mysql
81+
- include_role:
82+
name: icinga2
83+
- include_role:
84+
name: icingadb
85+
- include_role:
86+
name: icingadb_redis
87+
88+
post_tasks:
89+
- include_role:
90+
name: icingaweb2

molecule/default/dependency.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,3 +2,5 @@ dependency:
22
name: galaxy
33
options:
44
role-file: requirements.yml
5+
env:
6+
ANSIBLE_ROLES_PATH: .ansible/roles

molecule/default/requirements.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1-
- src: geerlingguy.mysql
1+
roles:
2+
- geerlingguy.mysql

molecule/local-default-pgsql/converge.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,9 @@
6868
collections:
6969
- icinga.icinga
7070
roles:
71-
#- repos
72-
#- geerlingguy.postgresql
73-
#- icinga2
71+
- repos
72+
- geerlingguy.postgresql
73+
- icinga2
7474
- icingadb
75-
#- icingadb_redis
76-
#- icingaweb2
75+
- icingadb_redis
76+
- icingaweb2

molecule/local-default/converge.yml

Lines changed: 20 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
- name: Converge
22
hosts: all
33
become: true
4+
module_defaults:
5+
ansible.builtin.apt:
6+
cache_valid_time: 3600
47
vars:
58
icingaweb2_resources:
69
director_db:
@@ -28,7 +31,7 @@
2831
run_kickstart: true
2932
kickstart:
3033
config:
31-
endpoint: "{{ ansible_fqdn }}"
34+
endpoint: icinga-default
3235
host: 127.0.0.1
3336
username: root
3437
password: root
@@ -79,6 +82,7 @@
7982
icingaweb2_admin_username: admin
8083
icingaweb2_admin_password: admin
8184
icingaweb2_db_import_schema: true
85+
icingadb_database_type: mysql
8286
icingadb_database_import_schema: true
8387
mysql_innodb_file_format: barracuda
8488
mysql_innodb_large_prefix: 1
@@ -160,8 +164,18 @@
160164
- zones.d/main/services
161165
collections:
162166
- icinga.icinga
163-
roles:
164-
- repos
165-
- geerlingguy.mysql
166-
- icinga2
167-
- icingaweb2
167+
pre_tasks:
168+
- include_role:
169+
name: repos
170+
- include_role:
171+
name: geerlingguy.mysql
172+
- include_role:
173+
name: icinga2
174+
- include_role:
175+
name: icingadb
176+
- include_role:
177+
name: icingadb_redis
178+
179+
post_tasks:
180+
- include_role:
181+
name: icingaweb2

molecule/local-default/molecule.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ driver:
88
platforms:
99
- name: icinga-default
1010
box: ubuntu/jammy64
11-
memory: 512
11+
memory: 1024
1212
cpus: 1
1313
interfaces:
1414
- network_name: private_network

roles/icingaweb2/templates/ini_template.j2

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,9 @@
33

44
[{{ section }}]
55
{% for option, value in options.items() %}
6-
{% if value is iterable and (value is not string and value is not mapping) %}
6+
{% if value is number %}
7+
{{ option }} = "{{ value }}"
8+
{% elif value is iterable and (value is not string and value is not mapping) %}
79
{{ option }} = "{{ value | join(', ') }}"
810
{% elif value is string and "=" in value %}
911
{% else %}

0 commit comments

Comments
 (0)