Skip to content

Commit 7dee4ce

Browse files
committed
Ensure all role variables are passed correctly
1 parent 283cf40 commit 7dee4ce

2 files changed

Lines changed: 6 additions & 3 deletions

File tree

roles/zuulnodepool/defaults/main.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,4 @@ ci_roles:
99
- name: ansible-role-zuul
1010
repo: https://github.com/openstack/ansible-role-zuul
1111
nodepool_install_method: git
12-
zuul_file_main_yaml_src: files/main.yaml
13-
zuul_file_zuul_conf_src: templates/zuul.conf
14-
nodepool_file_nodepool_yaml_src: files/nodepool.yaml
1512
nodepool_file_nodepool_launcher_service_config_src: files/nodepool-enable-webapp.conf

roles/zuulnodepool/tasks/main.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,9 @@
1616
vars:
1717
nodepool_task_manager:
1818
- pre
19+
nodepool_install_method: "{{ nodepool_install_method }}"
20+
nodepool_file_nodepool_yaml_src: "{{ nodepool_file_nodepool_yaml_src }}"
21+
nodepool_file_nodepool_launcher_service_config_src: "{{ nodepool_file_nodepool_launcher_service_config_src}}"
1922

2023
- name: Allow passwordless sudo for nodepool
2124
lineinfile:
@@ -43,6 +46,9 @@
4346
- include_role:
4447
name: "{{item.name}}"
4548
with_items: "{{ ci_roles }}"
49+
vars:
50+
nodepool_install_method: "{{ nodepool_install_method }}"
51+
nodepool_file_nodepool_launcher_service_config_src: "{{ nodepool_file_nodepool_launcher_service_config_src}}"
4652

4753
- name: Create zuul.conf
4854
template:

0 commit comments

Comments
 (0)