Merge ~paride/qa-jenkins-jobs:jjb2 into qa-jenkins-jobs:master

Proposed by Paride Legovini
Status: Merged
Approved by: Jean-Baptiste Lallement
Approved revision: 5b390fa81074e9d03c5937a15fa4f78315af775c
Merged at revision: 5b390fa81074e9d03c5937a15fa4f78315af775c
Proposed branch: ~paride/qa-jenkins-jobs:jjb2
Merge into: qa-jenkins-jobs:master
Diff against target: 118 lines (+38/-35)
4 files modified
jobs/auto-upgrade-testing/jobs.yaml (+7/-6)
jobs/ubiquity/jobs.yaml (+16/-15)
jobs/ubuntu-system-tests/jobs.yaml (+14/-13)
scripts/deploy-jobs.sh (+1/-1)
Reviewer Review Type Date Requested Status
Jean-Baptiste Lallement Approve
Review via email: mp+374928@code.launchpad.net

Commit message

Make the jobs and deploy script compatible with JJB in Bionic

Changes:
 - deploy-jobs.sh: call jenkins-jobs directly
 - make the jobs definitions compatible with JJB 2.x

To post a comment you must log in.
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

LGTM. THanks for the debugging session and fixing this.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/jobs/auto-upgrade-testing/jobs.yaml b/jobs/auto-upgrade-testing/jobs.yaml
2index d457701..d75375e 100755
3--- a/jobs/auto-upgrade-testing/jobs.yaml
4+++ b/jobs/auto-upgrade-testing/jobs.yaml
5@@ -78,11 +78,12 @@
6 publishers:
7 - postbuildscript:
8 builders:
9- - shell: 'sudo chown -R $USER:$USER "$WORKSPACE/{profile}"'
10- - grafana-tests-upload:
11- results_type: 'xunit'
12- results_file: 'junit.xml'
13- info_file: 'global_info.json'
14+ - build-steps:
15+ - shell: 'sudo chown -R $USER:$USER "$WORKSPACE/{profile}"'
16+ - grafana-tests-upload:
17+ results_type: 'xunit'
18+ results_file: 'junit.xml'
19+ info_file: 'global_info.json'
20 script-only-if-succeeded: False
21 script-only-if-failed: False
22 - archive:
23@@ -97,4 +98,4 @@
24 It can be run again to update to the latest versions of dependencies
25 Defined in lp:qa-jenkins-jobs
26 builders:
27- shell: pip3 install --user junitparser
28+ - shell: pip3 install --user junitparser
29diff --git a/jobs/ubiquity/jobs.yaml b/jobs/ubiquity/jobs.yaml
30index ce24ce9..48417c0 100755
31--- a/jobs/ubiquity/jobs.yaml
32+++ b/jobs/ubiquity/jobs.yaml
33@@ -117,21 +117,22 @@
34 - archive-artifacts
35 - postbuildscript:
36 builders:
37- - generate-junit-files:
38- source_dir: 'results/var/local/autopilot/junit'
39- test: '{test}'
40- - grafana-info:
41- device: 'vm'
42- model: ''
43- os: '{flavor}'
44- release: '{release}'
45- arch: '{arch}'
46- class: 'ubiquity'
47- conf: ''
48- - grafana-tests-upload:
49- results_type: 'xunit'
50- results_file: 'junit.xml'
51- info_file: 'global_info.json'
52+ - build-steps:
53+ - generate-junit-files:
54+ source_dir: 'results/var/local/autopilot/junit'
55+ test: '{test}'
56+ - grafana-info:
57+ device: 'vm'
58+ model: ''
59+ os: '{flavor}'
60+ release: '{release}'
61+ arch: '{arch}'
62+ class: 'ubiquity'
63+ conf: ''
64+ - grafana-tests-upload:
65+ results_type: 'xunit'
66+ results_file: 'junit.xml'
67+ info_file: 'global_info.json'
68 script-only-if-succeeded: False
69 script-only-if-failed: False
70 - junit:
71diff --git a/jobs/ubuntu-system-tests/jobs.yaml b/jobs/ubuntu-system-tests/jobs.yaml
72index cea8ce1..726fff4 100755
73--- a/jobs/ubuntu-system-tests/jobs.yaml
74+++ b/jobs/ubuntu-system-tests/jobs.yaml
75@@ -116,19 +116,20 @@
76 allow-empty: 'true'
77 - postbuildscript:
78 builders:
79- - subunit-to-junit-results-builder
80- - grafana-info:
81- device: 'vm'
82- model: '{model}'
83- os: '{flavor}'
84- release: '{release}'
85- arch: '{arch}'
86- class: 'system-tests'
87- conf: ''
88- - grafana-tests-upload:
89- results_type: 'pyunit'
90- results_file: 'results/*.pyunit'
91- info_file: 'global_info.json'
92+ - build-steps:
93+ - subunit-to-junit-results-builder
94+ - grafana-info:
95+ device: 'vm'
96+ model: '{model}'
97+ os: '{flavor}'
98+ release: '{release}'
99+ arch: '{arch}'
100+ class: 'system-tests'
101+ conf: ''
102+ - grafana-tests-upload:
103+ results_type: 'pyunit'
104+ results_file: 'results/*.pyunit'
105+ info_file: 'global_info.json'
106 script-only-if-succeeded: False
107 script-only-if-failed: False
108 - junit:
109diff --git a/scripts/deploy-jobs.sh b/scripts/deploy-jobs.sh
110index 3cd540e..6271741 100755
111--- a/scripts/deploy-jobs.sh
112+++ b/scripts/deploy-jobs.sh
113@@ -11,4 +11,4 @@ case $1 in
114 esac
115
116 make -C "$JOB_PATH"
117-python3 -m jenkins_jobs.cmd $ARGS -r "$JOB_PATH"
118+jenkins-jobs $ARGS -r "$JOB_PATH"

Subscribers

People subscribed via source and target branches

to all changes: