Skip to content

[7.x] [Task Manager] Log at different levels based on the state (#101751)#102402

Merged
chrisronline merged 1 commit intoelastic:7.xfrom
chrisronline:backport/7.x/pr-101751
Jun 16, 2021
Merged

[7.x] [Task Manager] Log at different levels based on the state (#101751)#102402
chrisronline merged 1 commit intoelastic:7.xfrom
chrisronline:backport/7.x/pr-101751

Conversation

@chrisronline
Copy link
Contributor

Backports the following commits to 7.x:

…1751)

* Log at different levels based on the state

* Fix types and add tests

* Remove unnecessary code

* Add more descriptive message

* Partially fix failing tests

* Move into separate function

* Get rid of customStatus in favor of moving the logging logic to a separate, mockable function

* Remove debug logging

* Do not log as an error if the stats are empty

* PR feedback

* Add docker whitelist

* alpha order

* English is hard

* Removing extra newline

* PR feedback around ignoring capacity estimation

* Move json utils
@chrisronline chrisronline added the backport This PR is a backport of another PR label Jun 16, 2021
@chrisronline chrisronline enabled auto-merge (squash) June 16, 2021 19:21
@spalger
Copy link
Contributor

spalger commented Jun 16, 2021

jenkins, test this

(restarting due to jenkins upgrade)

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@chrisronline chrisronline merged commit b16513a into elastic:7.x Jun 16, 2021
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