Skip to content

Commit 322a6d4

Browse files
jlebonjbtrystram
authored andcommitted
jobs/bodhi-trigger: also report test successes
Sometimes, Bodhi testing is broken for a few days if rawhide is in a bad shape. Let's also have the test report successes so that it's easier to tell when upstream testing have been fixed. It makes it more obvious also when e.g. just a single run failed in a series of otherwise passing runs.
1 parent b38b8c7 commit 322a6d4

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

jobs/bodhi-trigger.Jenkinsfile

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -168,12 +168,15 @@ cosaPod(cpu: "0.1", kvm: false) {
168168

169169
stage("Report Completion") {
170170
def outcome
171+
def emoji
171172
// treat UNSTABLE as PASSED too; we often have expired snoozed tests
172173
// that'll warn and Greenwave/Bodhi treats NEEDS_INSPECTION outcomes
173174
// as blocking
174175
if (test.result == 'SUCCESS' || test.result == 'UNSTABLE') {
176+
emoji = "🟢"
175177
outcome = 'PASSED'
176178
} else {
179+
emoji = "🔴"
177180
outcome = 'FAILED'
178181
}
179182
def blueocean_url = "${JENKINS_URL}/blue/organizations/jenkins/test-override/detail/test-override/${test.number}"
@@ -188,10 +191,8 @@ cosaPod(cpu: "0.1", kvm: false) {
188191
--stream ${stream}
189192
""")
190193
}
191-
if (test.result != 'SUCCESS') {
192-
def bodhi_url="https://bodhi.fedoraproject.org/updates/${msg.update.updateid}"
193-
pipeutils.matrixSend("🔥 ${currentBuild.description} - [🌊](${blueocean_url}) [🪷](${bodhi_url})")
194-
}
194+
def bodhi_url="https://bodhi.fedoraproject.org/updates/${msg.update.updateid}"
195+
pipeutils.matrixSend("${emoji} ${currentBuild.description} - [🌊](${blueocean_url}) [🪷](${bodhi_url})")
195196
}
196197

197198
// propagate

0 commit comments

Comments
 (0)