Bug #10356
Adapt the test job artifacts archiving script to their new location
100%
Description
In the Feature #10151 related branch, the screenshots, video captures and pcap files of a test suite run will be stored in a sub-directory of TMP_DIR.
In the Jenkins environment, the post_test_cleanup
script copy them back in the workspace so that Jenkins can archive them. It uses a find "${TEMP_DIR}" -maxdepth 1 [...]
to do so, which won’t work anymore.
So once Feature #10151 is merged, this find will have to be modified accordingly, likely by removing the -maxdepth 1
.
Subtasks
History
#1 Updated by bertagaz 2015-10-11 02:39:22
- blocked by
Feature #10151: Organize automated test suite artifacts in per-run directories added
#2 Updated by bertagaz 2015-10-11 02:41:21
- Parent task set to
Feature #10153
#3 Updated by bertagaz 2015-10-14 02:39:44
- blocks deleted (
)Feature #10151: Organize automated test suite artifacts in per-run directories
#4 Updated by bertagaz 2015-10-14 02:45:09
- Status changed from Confirmed to In Progress
- Assignee changed from bertagaz to intrigeri
- % Done changed from 0 to 80
- QA Check set to Ready for QA
I’ve already implemented this, in commit c49eb48
of the puppet-tails
repo.
Since then test #5 of the stable branch (without Feature #10151) and test #15 of the experimental branch (with Feature #10151) have happened and they still correctly store the artifacts.
So I guess we can close this ticket.
(Note that the tailstester.json and debug.log files in this test run were not archived, but that’s a different problem introduced by a buggy change in the puppet-tails
repo in commit 29ea4c6
and resolved in 88efaf6
).
#5 Updated by intrigeri 2015-10-14 05:53:58
- Assignee changed from intrigeri to bertagaz
- QA Check changed from Ready for QA to Info Needed
> I’ve already implemented this, in commit c49eb48
of the puppet-tails
repo.
:)
> (Note that the tailstester.json and debug.log files in this test run were not archived, but that’s a different problem introduced by a buggy change in the puppet-tails
repo in commit 29ea4c6
and resolved in 88efaf6
).
I see that the recent test_Tails_ISO_experimental
had no artifacts stored, and I don’t know if this has anything to do with this ticket. Is that the same problem that you say was fixed already? If not, please ticket.
#6 Updated by bertagaz 2015-10-14 13:47:33
- Assignee changed from bertagaz to intrigeri
- QA Check changed from Info Needed to Ready for QA
intrigeri wrote:
> I see that the recent test_Tails_ISO_experimental
had no artifacts stored, and I don’t know if this has anything to do with this ticket. Is that the same problem that you say was fixed already? If not, please ticket.
No, that was because of the Feature #10151 branch merge into experimental side affect, which failed the tests at the beginning, so no artifacts. Since then it has been fixed as shown in https://jenkins.tails.boum.org/view/Ale/job/test_Tails_ISO_experimental/20/
#7 Updated by intrigeri 2015-10-15 02:33:07
- Status changed from In Progress to Resolved
- Assignee deleted (
intrigeri) - % Done changed from 80 to 100
- QA Check changed from Ready for QA to Pass
Thanks for the explanations + pointers. So we’re done here :)