Skip to content

Commit

Permalink
[8.14] [CI] Fix missing test failure annotations (#186446) (#186474)
Browse files Browse the repository at this point in the history
# Backport

This will backport the following commits from `main` to `8.14`:
- [[CI] Fix missing test failure annotations
(#186446)](#186446)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Alex
Szabo","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-06-19T14:55:18Z","message":"[CI]
Fix missing test failure annotations (#186446)\n\n## Summary\r\nrename
`SLACK_NOTIFICATIONS_ENABLED`
=>\r\n`ELASTIC_SLACK_NOTIFICATIONS_ENABLED` to follow up on
elastic-wide\r\nbuildkite changes.\r\nThis should re-enable test failure
listing on the slack errors we
post.","sha":"c2986d9a6f09fd7644695ad72f46e7acfec964a3","branchLabelMapping":{"^v8.15.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Operations","release_note:skip","skip-ci","backport:all-open","v8.15.0"],"title":"[CI]
Fix missing test failure
annotations","number":186446,"url":"https://github.com/elastic/kibana/pull/186446","mergeCommit":{"message":"[CI]
Fix missing test failure annotations (#186446)\n\n## Summary\r\nrename
`SLACK_NOTIFICATIONS_ENABLED`
=>\r\n`ELASTIC_SLACK_NOTIFICATIONS_ENABLED` to follow up on
elastic-wide\r\nbuildkite changes.\r\nThis should re-enable test failure
listing on the slack errors we
post.","sha":"c2986d9a6f09fd7644695ad72f46e7acfec964a3"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.15.0","branchLabelMappingKey":"^v8.15.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/186446","number":186446,"mergeCommit":{"message":"[CI]
Fix missing test failure annotations (#186446)\n\n## Summary\r\nrename
`SLACK_NOTIFICATIONS_ENABLED`
=>\r\n`ELASTIC_SLACK_NOTIFICATIONS_ENABLED` to follow up on
elastic-wide\r\nbuildkite changes.\r\nThis should re-enable test failure
listing on the slack errors we
post.","sha":"c2986d9a6f09fd7644695ad72f46e7acfec964a3"}}]}] BACKPORT-->

Co-authored-by: Alex Szabo <[email protected]>
  • Loading branch information
kibanamachine and delanni authored Jun 19, 2024
1 parent 660d82a commit db83390
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .buildkite/pipeline-utils/test-failures/annotate.ts
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,7 @@ export const annotateTestFailures = async () => {
);
}

if (process.env.SLACK_NOTIFICATIONS_ENABLED === 'true') {
if (process.env.ELASTIC_SLACK_NOTIFICATIONS_ENABLED === 'true') {
buildkite.setMetadata(
'slack:test_failures:body',
getSlackMessage(failures, failureHtmlArtifacts)
Expand Down

0 comments on commit db83390

Please sign in to comment.