openstack
/
tripleo-quickstart-extras
Code Issues Proposed changes

Removed obseletes params from validate-tempest role 

* --isolated, and --failing options does not work
  with ostestr as these options are used with testr. Since ostestr
  is used as a universal test runner for running tempest across the
  board. If the above options got enabled by mistake it will break
  the tempest run. So removing it.

*  --load-list is replaced by whitelist-files and is taken care
  by this review: https://review.openstack.org/#/c/485252/

Change-Id: I9745fdcf592c4dc5bf3cbaf11e9e6b11e209ea77
This commit is contained in:
Chandan Kumar 2017-07-25 14:21:09 +05:30
parent a0350bafac
commit 517ce10604
3 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
roles/validate-tempest/README.md
View File

@ -22,10 +22,8 @@ Role Variables
* `tempest_config`: false/true - whether to prepare the script which configures and runs tempest or not
* `skip_file_src`: path to skip tests file, set it empty if running without skip list: `skip_file_src=''`
* `tempest_workers`: int - how many parallel workers to run (default is number of cores)
* `tempest_isolated`: false/true - if to run every test id in separate test runner (default is false)
* `tempest_tests_file`: path to file - path to file with tests to run
* `tempest_until_failure`: false/true - default is false, repeat the run again and again until failure occurs
* `tempest_failing`: false/true - default is false, run only tests known to be failing
* `tempest_exit_on_failure`: true/false - whether to exit from role with tempest exit code (default: true)
* `tempestmail_config`: config.yaml - name of config file for tempestmail script
* `tempest_track_resources`: true/false - whether to save the state of resources after tempest run (default: true)

2
roles/validate-tempest/defaults/main.yml
View File

@ -25,9 +25,7 @@ post_tempest: true
tempest_format: packages # venv or packages
skip_file_src: "skip_file.j2"
skip_file: skip_file
tempest_isolated: false
tempest_until_failure: false
tempest_failing: false
tempest_exit_on_failure: true
# Set tempest_version for different release
# For forked-tempest, we have branched name in synced with releases.

2
roles/validate-tempest/templates/run-tempest.sh.j2
View File

@ -34,10 +34,8 @@ $TEMPESTCLI cleanup --init-saved-state
( $OSTESTR '({{ test_regex }})' --no-pretty --subunit {% if skip_file_src != '' %}--blacklist_file={{ working_dir }}/{{ skip_file }}{% endif %}
{% if tempest_workers is defined %} --concurrency {{ tempest_workers }} {% endif %}
{% if tempest_isolated|bool %} --isolated {% endif %}
{% if tempest_tests_file is defined %} --load-list={{ tempest_tests_file }} {% endif %}
{% if tempest_until_failure|bool %} --until-failure {% endif %}
{% if tempest_failing|bool %} --failing {% endif %}
) | tee >( $S2JXML ) | $SUBUNIT
## Check which all tenants would be modified in the tempest run