This change enables using fetch-subunit-output role along with
the fetch-output role. By default the role still synchronizes
artifacts back to the executor.
Change-Id: I50a50856b96a9112c1b4be82ca04b612fd29d939
Change new variable zuul_additional_subunit_dirs to
fetch_subunit_output_additional_dirs to name it after the role.
Change-Id: I7ac0b5d343701409a8741439334fe27f5374a258
In addition to the main subunit file from zuul_work_dir,
collect the subunit files from the elements (directories)
of the zuul_additional_subunit_dirs list.
The default behavior is unchanged.
While the documentation of this role states that zuul_work_dir
contains an absolute path, this is not always true.
So make sure to not make any assumption about zuul_work_dir
in order to not fail spectacularly as it happened with the
previous patch[0].
Add also some tests for the role: both the basic case
and with an additional test directory.
[0] https://review.opendev.org/673885
Change-Id: Iabf2e0cf6d86e36a174778367186bbd39a65c3dd
In addition to the main subunit file from zuul_work_dir,
collect the subunit files from the elements (directories)
of the zuul_additional_subunit_dirs list.
The default behavior is unchanged.
Change-Id: Ic3bc76aa104951bcc0c186347196067342e28751
The role allows override of the zuul_work_dir variable, however in the
step where it retrieves the test results is the only step where it
prepends the directory with a fixed path. This breaks the step if the
zuul_work_dir given is a fully qualified path.
This patch defaults the zuul_work_dir to a fully qualified directory and
standarizes the usage zuul_work_dir across the role.
Change-Id: I62bc7571f625586a0d1b660f896e0c7cbaeae6e1
This reverts commit 9f4efe98e10c9d553dc8a15bd7b84584e1118684.
Compress testr results.html before fetching it
The synchronize command has testr_results.html.gz in the file list, but
we don't ever gzip it so it fails on fetch.
Make the "Check for testr directory" task actually check for
.testrepository and not for .stestr a second time.
The compression bug broke only for people using stestr. For people
using testr, the path bug meant they don't get html reports.
Change-Id: I0cdfc66ee8b046affeb0b071fef38c21cb7a4948
We don't need two roles for testr and stestr. The roles have logic to
check to see if .testrepository or .stestr dirs exist, and the logic
after the subunit extraction is the same. Combine them into a single
role which should do the right thing.
Also, document it.
Change-Id: I833320cf9a932d8e119645eb798ce0c93d854321
