#25 Set default artifacts directory from environment variable TEST_ARTIFACTS
Merged by merlinm. Opened by merlinm.
Unknown source master

Download 25.patch

The standard test roles have been revised so they will use the artifacts directory provided by the environment variable TEST_ARTIFACTS (if specified), otherwise they will use the ./artifacts directory. Note: The artifacts variables had to be removed from the _role_/vars/main.yml files and added to the _role_/defaults/main.yml files to lower their precedence so the value could be set from somewhere other than a command line extra vars (-e) value.

Looks good to me.

I think it is useful for debugging, testing and making this easier to
work with.

However the canonical way for a testing system to invoke a playbook is
to set both the relevent environment variables and the ansible
variables. The spec states this clearly:

https://fedoraproject.org/wiki/Changes/InvokingTests

Pull-Request has been merged by merlinm

Metadata