Skip to content

Commit affcd2a

Browse files
lpiwowaropenshift-merge-bot[bot]
authored andcommitted
Unset resources section in workflows
There is a bug in Tempest which prevents us to set a sane upper limit to the amount of memory Tempest can consume [1]. Until the fix reaches the upstream build of the openstack-tempest-all image we should not be setting any resource limits to the tempest pods. This commit is ensuring that we are unsetting the default values set on the side of the test-operator in the workflow section of Tempest CR. [1] https://bugs.launchpad.net/tempest/+bug/2088074
1 parent 53adb63 commit affcd2a

File tree

1 file changed

+25
-0
lines changed

1 file changed

+25
-0
lines changed

roles/test_operator/tasks/tempest-tests.yml

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -173,3 +173,28 @@
173173
test_operator_cr |
174174
combine({'spec': {'workflow': overriden_workflow}}, recursive=true)
175175
}}
176+
177+
- name: Make sure resources are not set for worklfow step
178+
when:
179+
- not cifmw_test_operator_dry_run | bool
180+
- stage_vars_dict.cifmw_test_operator_tempest_workflow | list | length > 0
181+
block:
182+
- name: Remove resources until fix for bug 2088074 reaches openstack-tempest-all image
183+
when: item.resources is undefined
184+
vars:
185+
_no_resources_workflow_step: >-
186+
{{
187+
item |
188+
combine({"resources": {"requests": {}, "limits": {}}}, recursive=true)
189+
}}
190+
ansible.builtin.set_fact:
191+
no_resources_workflow: "{{ no_resources_workflow | default([]) + [_no_resources_workflow_step] }}"
192+
loop: "{{ stage_vars_dict.cifmw_test_operator_tempest_workflow | list }}"
193+
194+
- name: Override the Tempest CR workflow
195+
ansible.builtin.set_fact:
196+
test_operator_cr: >-
197+
{{
198+
test_operator_cr |
199+
combine({'spec': {'workflow': no_resources_workflow}}, recursive=true)
200+
}}

0 commit comments

Comments
 (0)