#22 drop requirement of setting subjects and artifacts ansible variables when executing the test playbook
Merged 5 years ago by psss. Opened 5 years ago by bgoncalv.
fedora-ci/ bgoncalv/docs update-sti  into  master

@@ -178,9 +178,6 @@ 

  . MUST execute the playbook with the following operating system environment variables:

  .. `TEST_SUBJECTS`: The _test subjects_ string as described above

  .. `TEST_ARTIFACTS`: The full path of an empty folder for _test artifacts_

- . MUST execute the playbook with the following Ansible variables.

- .. `subjects`: The _test subjects_ string as described above

- .. `artifacts`: The full path of an empty folder for _test artifacts_

  . SHOULD execute the playbook with all Ansible tags best represent the intended _test context_.

  .. The choice of _test context_ tags is related to the _test subject_ being tested

  . MUST execute Ansible with inventory set to the full path of the file or directory `tests/inventory` if it exists.

@@ -58,7 +58,7 @@ 

  

  You can find output artifacts of the tests in an `artifacts/` or specify a specific directory like this:

  

-  ansible-playbook -e artifacts=/tmp/output tests.yml

+  TEST_ARTIFACTS=/tmp/output ansible-playbook tests.yml

  

  You can filter which kinds of tests are run by providing a `--tags` argument.

  To only run tests that are suited for classic systems installed by `yum` or `dnf` you can use a command like:

Fix related to https://pagure.io/standard-test-roles/issue/307

Before changing how the pipeline executes the playbook the STI specification needs to be updated.

rebased onto 2179c7b669228cc0a4e1348859da0dbb6f6127ee

5 years ago

rebased onto 31fd0f2

5 years ago

Metadata Update from @psss:
- Request assigned

5 years ago

Makes sense, especially because these variables have never been supported by the Standard Test Roles. Going to merge.

Pull-Request has been merged by psss

5 years ago