Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TaskManager] log health on interval with background_tasks only role #158890

Merged
merged 3 commits into from
Jun 6, 2023

Conversation

pmuellr
Copy link
Member

@pmuellr pmuellr commented Jun 2, 2023

resolves #158870

Summary

For Kibana servers that only have node role background_tasks, log the task manager health report to the Kibana logs on an interval, currently every hour.

resolves elastic#158870

For Kibana servers that only have node role `background_tasks`, log
the task manager health report to the Kibana logs on an interval,
currently every hour.
@pmuellr pmuellr added enhancement New value added to drive a business result Feature:Task Manager Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Jun 2, 2023
@pmuellr pmuellr marked this pull request as ready for review June 2, 2023 19:44
@pmuellr pmuellr requested a review from a team as a code owner June 2, 2023 19:44
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@pmuellr pmuellr added the release_note:skip Skip the PR/issue when compiling release notes label Jun 2, 2023
Copy link
Contributor

@ymao1 ymao1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Verified this logs for background nodes only

@pmuellr
Copy link
Member Author

pmuellr commented Jun 5, 2023

@elasticmachine merge upstream

@pmuellr
Copy link
Member Author

pmuellr commented Jun 6, 2023

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
taskManager 40 41 +1
Unknown metric groups

API count

id before after diff
taskManager 82 83 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
securitySolution 413 417 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
securitySolution 497 501 +4
total +6

History

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

@pmuellr pmuellr merged commit 837ef26 into elastic:main Jun 6, 2023
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.8 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 158890

Questions ?

Please refer to the Backport tool documentation

@pmuellr pmuellr added v8.8.2 backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) and removed v8.8.1 labels Jun 6, 2023
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.8 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 158890

Questions ?

Please refer to the Backport tool documentation

@pmuellr
Copy link
Member Author

pmuellr commented Jun 6, 2023

💚 All backports created successfully

Status Branch Result
8.8

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

pmuellr added a commit to pmuellr/kibana that referenced this pull request Jun 6, 2023
…lastic#158890)

resolves elastic#158870

## Summary

For Kibana servers that only have node role `background_tasks`, log the
task manager health report to the Kibana logs on an interval, currently
every hour.

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit 837ef26)

# Conflicts:
#	x-pack/plugins/task_manager/server/plugin.ts
pmuellr added a commit that referenced this pull request Jun 6, 2023
… role (#158890) (#159142)

# Backport

This will backport the following commits from `main` to `8.8`:
- [[TaskManager] log health on interval with background_tasks only role
(#158890)](#158890)

<!--- Backport version: 8.9.7 -->

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

<!--BACKPORT [{"author":{"name":"Patrick
Mueller","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-06-06T12:42:40Z","message":"[TaskManager]
log health on interval with background_tasks only role
(#158890)\n\nresolves
https://github.com/elastic/kibana/issues/158870\r\n\r\n##
Summary\r\n\r\nFor Kibana servers that only have node role
`background_tasks`, log the\r\ntask manager health report to the Kibana
logs on an interval, currently\r\nevery hour.\r\n\r\nCo-authored-by:
Kibana Machine
<[email protected]>","sha":"837ef26fb0cced40214b25f0f1f22a8a0d610fb2","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["enhancement","release_note:skip","Feature:Task
Manager","Team:ResponseOps","backport:prev-minor","v8.9.0","v8.8.2"],"number":158890,"url":"https://github.com/elastic/kibana/pull/158890","mergeCommit":{"message":"[TaskManager]
log health on interval with background_tasks only role
(#158890)\n\nresolves
https://github.com/elastic/kibana/issues/158870\r\n\r\n##
Summary\r\n\r\nFor Kibana servers that only have node role
`background_tasks`, log the\r\ntask manager health report to the Kibana
logs on an interval, currently\r\nevery hour.\r\n\r\nCo-authored-by:
Kibana Machine
<[email protected]>","sha":"837ef26fb0cced40214b25f0f1f22a8a0d610fb2"}},"sourceBranch":"main","suggestedTargetBranches":["8.8"],"targetPullRequestStates":[{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/158890","number":158890,"mergeCommit":{"message":"[TaskManager]
log health on interval with background_tasks only role
(#158890)\n\nresolves
https://github.com/elastic/kibana/issues/158870\r\n\r\n##
Summary\r\n\r\nFor Kibana servers that only have node role
`background_tasks`, log the\r\ntask manager health report to the Kibana
logs on an interval, currently\r\nevery hour.\r\n\r\nCo-authored-by:
Kibana Machine
<[email protected]>","sha":"837ef26fb0cced40214b25f0f1f22a8a0d610fb2"}},{"branch":"8.8","label":"v8.8.2","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
@mistic mistic removed the v8.8.1 label Jun 8, 2023
@mistic
Copy link
Member

mistic commented Jun 8, 2023

This PR backport didn't make it into the latest BC for 8.8.1. Updating the labels.

@pmuellr
Copy link
Member Author

pmuellr commented Jun 8, 2023

This PR backport didn't make it into the latest BC for 8.8.1. Updating the labels.

Just for reference, we didn't intend to put it in 8.8.1 (though would have been nice!) - I did have a label on it for 8.8.1, before 8.8.1 got pulled in, that I changed to 8.8.2, then kibanamachine added 8.8.1 after the backport. I assume this was a timing race between the merge and the FF, which I'm guessing happened the same day. It's been a while, but learned my lesson on merging backports on a FF day!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) enhancement New value added to drive a business result Feature:Task Manager release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.8.2 v8.9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TaskManager] log kibana health report every hour for nodes with background_tasks only role
6 participants