Skip to content

[8.15] [ResponseOps][alerting] add rule info to logging in alertsClient (#190857)#191094

Merged
kibanamachine merged 1 commit intoelastic:8.15from
kibanamachine:backport/8.15/pr-190857
Aug 22, 2024
Merged

[8.15] [ResponseOps][alerting] add rule info to logging in alertsClient (#190857)#191094
kibanamachine merged 1 commit intoelastic:8.15from
kibanamachine:backport/8.15/pr-190857

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.15:

Questions ?

Please refer to the Backport tool documentation

…stic#190857)

## Summary

While investigating some issues with the alertsClient, I realized that
we weren't writing out any rule information for the logged messages.
This made debugging quite difficult, as I wanted to see the rule, so had
to search through the alerts indices for the specified alert to get it's
rule id, rule type, etc.

As an example, see elastic#190376

This PR adds that kind of rule info to the logged messages in
alertsClient, as well as the typical sort of tags we write out (rule id,
rule type, module).

(cherry picked from commit 07717a4)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Aug 22, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) August 22, 2024 14:34
@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @pmuellr

@kibanamachine kibanamachine merged commit 5ce3574 into elastic:8.15 Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants