Handle the case when tempest fails
Before this change, the errors from tempest run are suppressed
in order to always execute cinderlib tests.
Use the new variable exported by run-tempest instead,
and fail properly if needed.
Depends-On: https://review.opendev.org/720213
Change-Id: Ic1fd6fcc9b9183f195c66f0022e913ae9734c97b
Luigi Toscano
6 years ago
| 18 | 18 | - setup-tempest-data-dir |
| 19 | 19 | - acl-devstack-files |
| 20 | 20 | - role: run-tempest |
| 21 | # ignore the errors, so that run-cinderlib-tests is always executed | |
| 21 | # ignore the errors for now, so that run-cinderlib-tests is always executed | |
| 22 | 22 | ignore_errors: yes |
| 23 | 23 | - role: change-devstack-data-owner |
| 24 | 24 | devstack_data_subdir_changed: cinder |
| 25 | 25 | devstack_data_subdir_owner: zuul |
| 26 | 26 | - role: run-cinderlib-tests |
| 27 | 27 | cinderlib_base_dir: "{{ ansible_user_dir }}/{{ zuul.projects['opendev.org/openstack/cinderlib'].src_dir }}" |
| 28 | post_tasks: | |
| 29 | - name: Fail if the first tempest run did not work | |
| 30 | fail: | |
| 31 | msg: "tempest run returned with an error" | |
| 32 | when: tempest_run_result is defined and tempest_run_result.rc != 0 |