Merge ~andersson123/qa-jenkins-jobs:add-timestamp-wrapper into qa-jenkins-jobs:master

Proposed by Tim Andersson
Status: Merged
Merged at revision: 579838816e31c7792c534e909e2229c142ed8290
Proposed branch: ~andersson123/qa-jenkins-jobs:add-timestamp-wrapper
Merge into: qa-jenkins-jobs:master
Diff against target: 70 lines (+11/-1)
4 files modified
jobs/auto-upgrade-testing/jobs.yaml (+3/-1)
jobs/autopkgtest-cloud/jobs.yaml (+4/-0)
jobs/iso-testing/jobs.yaml (+2/-0)
jobs/releaseqa/jobs.yaml (+2/-0)
Reviewer Review Type Date Requested Status
Paride Legovini Approve
Review via email: mp+458009@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Paride Legovini (paride) wrote (last edit ):

The change itself is (almost) good, but the idea is to have timestamps (and timeouts) on all the job, not only on the upgrade testing ones.

review: Needs Fixing
Revision history for this message
Paride Legovini (paride) :
review: Needs Fixing
Revision history for this message
Tim Andersson (andersson123) wrote :

Gotcha, I'll amend this

Revision history for this message
Tim Andersson (andersson123) wrote :

Amended now, please re-review

Revision history for this message
Paride Legovini (paride) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/jobs/auto-upgrade-testing/jobs.yaml b/jobs/auto-upgrade-testing/jobs.yaml
index b4e310e..dc01e02 100644
--- a/jobs/auto-upgrade-testing/jobs.yaml
+++ b/jobs/auto-upgrade-testing/jobs.yaml
@@ -40,6 +40,8 @@
40 gitproject: auto-upgrade-testing-specifications40 gitproject: auto-upgrade-testing-specifications
41 giturl: https://git.launchpad.net/auto-upgrade-testing-specifications41 giturl: https://git.launchpad.net/auto-upgrade-testing-specifications
42 gitvarprefix: AUTS42 gitvarprefix: AUTS
43 wrappers:
44 - timestamps
43 builders:45 builders:
44 - clear-artifacts:46 - clear-artifacts:
45 - shell: |47 - shell: |
@@ -63,7 +65,7 @@
63 cd auto-upgrade-testing65 cd auto-upgrade-testing
6466
65 set +e67 set +e
66 sudo --preserve-env=AUTOPKGTEST_APT_SOURCES_FILE python3 -m upgrade_testing.command_line -c $PROFILE --provision --results-dir "$WORKSPACE/{profile}/" --adt-args=--timeout-factor=10 2>&1 | ts -s68 sudo --preserve-env=AUTOPKGTEST_APT_SOURCES_FILE python3 -m upgrade_testing.command_line -c $PROFILE --provision --results-dir "$WORKSPACE/{profile}/" --adt-args=--timeout-factor=10
67 EXITSTATUS=$?69 EXITSTATUS=$?
68 sudo chown -R $USER:$USER "$WORKSPACE"70 sudo chown -R $USER:$USER "$WORKSPACE"
69 cp $WORKSPACE/{profile}/*/artifacts/upgrade_run/junit.xml $WORKSPACE/junit.xml71 cp $WORKSPACE/{profile}/*/artifacts/upgrade_run/junit.xml $WORKSPACE/junit.xml
diff --git a/jobs/autopkgtest-cloud/jobs.yaml b/jobs/autopkgtest-cloud/jobs.yaml
index 1a2f7a1..2043162 100644
--- a/jobs/autopkgtest-cloud/jobs.yaml
+++ b/jobs/autopkgtest-cloud/jobs.yaml
@@ -24,6 +24,8 @@
24 node: 'venonat'24 node: 'venonat'
25 triggers:25 triggers:
26 - timed: '@daily'26 - timed: '@daily'
27 wrappers:
28 - timestamps
27 builders:29 builders:
28 - clear-artifacts:30 - clear-artifacts:
29 - shell: |31 - shell: |
@@ -59,6 +61,8 @@
59 - url: 'https://git.launchpad.net/autopkgtest-cloud/patch/?id=HEAD'61 - url: 'https://git.launchpad.net/autopkgtest-cloud/patch/?id=HEAD'
60 check-content:62 check-content:
61 - simple: true63 - simple: true
64 wrappers:
65 - timestamps
62 builders:66 builders:
63 - clear-artifacts:67 - clear-artifacts:
64 - shell: |68 - shell: |
diff --git a/jobs/iso-testing/jobs.yaml b/jobs/iso-testing/jobs.yaml
index a1c21a0..fc466d9 100644
--- a/jobs/iso-testing/jobs.yaml
+++ b/jobs/iso-testing/jobs.yaml
@@ -160,6 +160,8 @@
160 It can be run again to update to the latest versions of dependencies160 It can be run again to update to the latest versions of dependencies
161 Defined in lp:qa-jenkins-jobs161 Defined in lp:qa-jenkins-jobs
162 node: 'iso-testing'162 node: 'iso-testing'
163 wrappers:
164 - timestamps
163 builders:165 builders:
164 - shell: |166 - shell: |
165 sudo add-apt-repository -y ppa:canonical-platform-qa-jenkins/canonical-platform-qa-jenkins-prod167 sudo add-apt-repository -y ppa:canonical-platform-qa-jenkins/canonical-platform-qa-jenkins-prod
diff --git a/jobs/releaseqa/jobs.yaml b/jobs/releaseqa/jobs.yaml
index 46cc475..80bdba8 100644
--- a/jobs/releaseqa/jobs.yaml
+++ b/jobs/releaseqa/jobs.yaml
@@ -25,6 +25,8 @@
25 - workspace-cleanup25 - workspace-cleanup
26 triggers:26 triggers:
27 - timed: '@weekly'27 - timed: '@weekly'
28 wrappers:
29 - timestamps
28 builders:30 builders:
29 - shell: |31 - shell: |
30 #!/bin/bash32 #!/bin/bash

Subscribers

People subscribed via source and target branches

to all changes: