Skip to content

[8.x] Improve error logs for task manager poller (#197635)#197804

Merged
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-197635
Oct 25, 2024
Merged

[8.x] Improve error logs for task manager poller (#197635)#197804
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-197635

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

I noticed some scenarios we see error logs from the task poller like
`Failed to poll for work: undefined` making me think `err.message` is
empty in some situations. I'm modifying the code to handle string
situations if ever they occur by performing `err.message || err` and to
also include a stack trace when strings are passed-in.

---------

Co-authored-by: Patrick Mueller <patrick.mueller@elastic.co>
(cherry picked from commit 81b63c6)
@kibanamachine kibanamachine requested a review from a team as a code owner October 25, 2024 11:17
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Oct 25, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) October 25, 2024 11:17
@kibanamachine kibanamachine merged commit a12a70a into elastic:8.x Oct 25, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @mikecote

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