Skip to content

Commit 06bfb9d

Browse files
danpawlikdasm
authored andcommitted
Fix setting fact for cifmw_test_operator_concurrency
Some part of previous commit [1] was not needed and it breaks overwriting the values (it always was uing 8 value for concurrency). This commit partially reverts commit proposed in PR [1]. [1] #2938 Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
1 parent d471c0c commit 06bfb9d

2 files changed

Lines changed: 3 additions & 6 deletions

File tree

roles/test_operator/defaults/main.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ cifmw_test_operator_controller_namespace: openstack-operators
2929
cifmw_test_operator_bundle: ""
3030
cifmw_test_operator_timeout: 3600
3131
cifmw_test_operator_logs_image: quay.io/quay/busybox
32+
cifmw_test_operator_concurrency: 8
3233
cifmw_test_operator_cleanup: false
3334
cifmw_test_operator_dry_run: false
3435
cifmw_test_operator_default_groups:
@@ -139,7 +140,7 @@ cifmw_test_operator_tempest_config:
139140
{{ stage_vars_dict.cifmw_test_operator_tempest_exclude_list | default('') }}
140141
expectedFailuresList: |
141142
{{ stage_vars_dict.cifmw_test_operator_tempest_expected_failures_list | default('') }}
142-
concurrency: "{{ stage_vars_dict.cifmw_test_operator_concurrency | default(8) }}"
143+
concurrency: "{{ cifmw_test_operator_concurrency }}"
143144
externalPlugin: "{{ stage_vars_dict.cifmw_test_operator_tempest_external_plugin | default([]) }}"
144145
extraRPMs: "{{ stage_vars_dict.cifmw_test_operator_tempest_extra_rpms | default([]) }}"
145146
extraImages: "{{ stage_vars_dict.cifmw_test_operator_tempest_extra_images | default([]) }}"

roles/test_operator/tasks/stages.yml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,7 @@
3535
start_with: cifmw_test_operator_{{ _stage_vars.type }}
3636
when: item.key.startswith(start_with)
3737
ansible.builtin.set_fact:
38-
stage_vars_dict: "{{ stage_vars_dict | combine({item.key: _stage_test_vars[item.key] | default(lookup('vars', item.key, default=omit)) }) }}"
39-
40-
- name: Overwrite concurrency in global_vars
41-
ansible.builtin.set_fact:
42-
stage_vars_dict: "{{ stage_vars_dict | combine({'cifmw_test_operator_concurrency': _stage_test_vars.cifmw_test_operator_concurrency | default(lookup('vars', 'concurrency', default=omit)) }) }}"
38+
stage_vars_dict: "{{ stage_vars_dict | combine({item.key: _stage_test_vars[item.key] | default(lookup('vars', item.key, default=omit))} ) }}"
4339

4440
- name: Override specific type config
4541
vars:

0 commit comments

Comments
 (0)