Skip to content

Commit a454116

Browse files
authored
Use separate buildkite annotation context for failed build scans (#101740) (#101747)
1 parent 7a85091 commit a454116

File tree

1 file changed

+24
-13
lines changed

1 file changed

+24
-13
lines changed

build-tools-internal/src/main/groovy/elasticsearch.build-scan.gradle

Lines changed: 24 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -131,19 +131,30 @@ buildScan {
131131
link 'Source', "https://github.com/${repository}/tree/${BuildParams.gitRevision}"
132132
}
133133

134-
buildScanPublished { scan ->
135-
// Attach build scan link as build metadata
136-
// See: https://buildkite.com/docs/pipelines/build-meta-data
137-
new ProcessBuilder('buildkite-agent', 'meta-data', 'set', "build-scan-${System.getenv('BUILDKITE_JOB_ID')}", "${scan.buildScanUri}")
138-
.start()
139-
.waitFor()
140-
141-
// Add a build annotation
142-
// See: https://buildkite.com/docs/agent/v3/cli-annotate
143-
def body = """<div class="mb3"><span class="p1 border rounded">${System.getenv('BUILDKITE_LABEL')}</span> :gradle: build ran: <a href="${scan.buildScanUri}"><code>gradle ${gradle.startParameter.taskNames.join(' ')}</code></a></div>"""
144-
new ProcessBuilder('buildkite-agent', 'annotate', '--context', 'gradle-build-scans', '--append', '--style', 'info', body)
145-
.start()
146-
.waitFor()
134+
buildFinished { result ->
135+
buildScanPublished { scan ->
136+
// Attach build scan link as build metadata
137+
// See: https://buildkite.com/docs/pipelines/build-meta-data
138+
new ProcessBuilder('buildkite-agent', 'meta-data', 'set', "build-scan-${System.getenv('BUILDKITE_JOB_ID')}", "${scan.buildScanUri}")
139+
.start()
140+
.waitFor()
141+
142+
// Add a build annotation
143+
// See: https://buildkite.com/docs/agent/v3/cli-annotate
144+
def body = """<div class="mb3"><span class="p1 border rounded">${System.getenv('BUILDKITE_LABEL')}</span> :gradle: ${result.failure ? 'failed' : 'successful'} build: <a href="${scan.buildScanUri}"><code>gradle ${gradle.startParameter.taskNames.join(' ')}</code></a></div>"""
145+
new ProcessBuilder(
146+
'buildkite-agent',
147+
'annotate',
148+
'--context',
149+
result.failure ? 'gradle-build-scans-failed' : 'gradle-build-scans',
150+
'--append',
151+
'--style',
152+
result.failure ? 'error' : 'info',
153+
body
154+
)
155+
.start()
156+
.waitFor()
157+
}
147158
}
148159
} else {
149160
tag 'LOCAL'

0 commit comments

Comments
 (0)