Skip to content

Commit 42f33cb

Browse files
committed
Refactor DW VM module defaults and omit logic
Signed-off-by: Webster Mudge <wmudge@cloudera.com>
1 parent 9a4567a commit 42f33cb

File tree

2 files changed

+8
-12
lines changed

2 files changed

+8
-12
lines changed

roles/runtime/tasks/initialize_base.yml

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -190,13 +190,9 @@
190190
use_default_dbc: "{{ __dw_config.0.use_default_dbc }}"
191191
type: "{{ __dw_config.1.type | default(run__dw_default_vw_type) }}"
192192
template: "{{ __dw_config.1.template | default(run__dw_default_template_type) }}"
193-
autoscaling_min_nodes: "{{ __dw_config.1.autoscaling.min_nodes | default(omit) }}"
194-
autoscaling_max_nodes: "{{ __dw_config.1.autoscaling.max_nodes | default(omit) }}"
195-
common_configs: "{{ __dw_config.1.configs.common_configs | default(omit) }}"
196-
application_configs: "{{ __dw_config.1.configs.application_configs | default(omit) }}"
197-
ldap_groups: "{{ __dw_config.1.configs.ldap_groups | default(omit) }}"
198-
enable_sso: "{{ __dw_config.1.configs.enable_sso | default(omit) }}"
199193
tags: "{{ __dw_config.1.tags | default({}) | combine(run__dw_tags) }}"
194+
autoscaling: "{{ __dw_config.1.autoscaling | default({}) }}"
195+
configs: "{{ __dw_config.1.configs | default({}) }}"
200196
loop: "{{ run__dw_dbc_configs | subelements('virtual_warehouses')}}"
201197
loop_control:
202198
loop_var: __dw_config

roles/runtime/tasks/setup_aws.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -121,13 +121,13 @@
121121
type: "{{ __dw_vw_config.type }}"
122122
name: "{{ __dw_vw_config.name }}"
123123
template: "{{ __dw_vw_config.template }}"
124-
autoscaling_min_nodes: "{{ __dw_vw_config.autoscaling_min_nodes }}"
125-
autoscaling_max_nodes: "{{ __dw_vw_config.autoscaling_max_nodes }}"
126-
common_configs: "{{ __dw_vw_config.common_configs }}"
127-
application_configs: "{{ __dw_vw_config.application_configs }}"
128-
ldap_groups: "{{ __dw_vw_config.ldap_groups }}"
129-
enable_sso: "{{ __dw_vw_config.enable_sso | bool }}"
130124
tags: "{{ __dw_vw_config.tags }}"
125+
autoscaling_min_nodes: "{{ __dw_vw_config.autoscaling.min_nodes | default(omit) }}"
126+
autoscaling_max_nodes: "{{ __dw_vw_config.autoscaling.max_nodes | default(omit) }}"
127+
common_configs: "{{ __dw_vw_config.configs.common_configs | default(omit) }}"
128+
application_configs: "{{ __dw_vw_config.configs.application_configs | default(omit) }}"
129+
ldap_groups: "{{ __dw_vw_config.configs.ldap_groups | default(omit) }}"
130+
enable_sso: "{{ __dw_vw_config.configs.enable_sso | default(omit) }}"
131131
wait: yes
132132
async: 3600 # 1 hour timeout
133133
poll: 0

0 commit comments

Comments
 (0)