Skip to content

[Task Manager] Change info message "ran out Available Workers" to debug#62083

Merged
pmuellr merged 1 commit intoelastic:masterfrom
pmuellr:tm/no-busy-message
Apr 1, 2020
Merged

[Task Manager] Change info message "ran out Available Workers" to debug#62083
pmuellr merged 1 commit intoelastic:masterfrom
pmuellr:tm/no-busy-message

Conversation

@pmuellr
Copy link
Copy Markdown
Contributor

@pmuellr pmuellr commented Mar 31, 2020

resolves #54920

The scenario this message warns about is actually something that we should
expect to happen in a busy system - and we back off as we should. But
there's no reason it needs to be info-logged, especially as it seems
somewhat frightening.

Changing to debug, so in case we do need this for diagnostic purposes, we
can still get it via logging config.

resolves elastic#54920

The scenario this message warns about is actually something that we should
expect to happen in a busy system - and we back off as we should.  But
there's no reason it needs to be info-logged, especially as it seems
somewhat frightening.

Changing to debug, so in case we do need this for diagnostic purposes, we
can still get it via logging config.
@pmuellr pmuellr added Feature:Alerting v8.0.0 v7.7.0 Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v7.8.0 labels Mar 31, 2020
@pmuellr pmuellr requested a review from a team as a code owner March 31, 2020 22:00
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

@pmuellr pmuellr added the release_note:skip Skip the PR/issue when compiling release notes label Mar 31, 2020
Copy link
Copy Markdown
Contributor

@YulNaumenko YulNaumenko left a comment

Choose a reason for hiding this comment

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

LGTM

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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

Copy link
Copy Markdown
Contributor

@mikecote mikecote left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
Contributor

@bmcconaghy bmcconaghy left a comment

Choose a reason for hiding this comment

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

LGTM

@pmuellr pmuellr merged commit 01880ac into elastic:master Apr 1, 2020
pmuellr added a commit to pmuellr/kibana that referenced this pull request Apr 1, 2020
…ug (elastic#62083)

resolves elastic#54920

The scenario this message warns about is actually something that we should
expect to happen in a busy system - and we back off as we should.  But
there's no reason it needs to be info-logged, especially as it seems
somewhat frightening.

Changing to debug, so in case we do need this for diagnostic purposes, we
can still get it via logging config.
pmuellr added a commit to pmuellr/kibana that referenced this pull request Apr 1, 2020
…ug (elastic#62083)

resolves elastic#54920

The scenario this message warns about is actually something that we should
expect to happen in a busy system - and we back off as we should.  But
there's no reason it needs to be info-logged, especially as it seems
somewhat frightening.

Changing to debug, so in case we do need this for diagnostic purposes, we
can still get it via logging config.
gmmorris added a commit to gmmorris/kibana that referenced this pull request Apr 1, 2020
* master: (44 commits)
  [Alerting] add alerting privileges for uptime and metrics (elastic#61113)
  Update percy agent to latest version (elastic#62089)
  [APM] Update central configuration text (elastic#61556)
  [Fleet] Ouput api key do not need metricbeat* access (elastic#60319)
  Document new `xpack.security.authc.*` settings and related 8.0.0 breaking changes. (elastic#61443)
  Migrate test plugins ⇒ NP (kbn_tp_sample_panel_action) (elastic#60749)
  [Alerting] Add "Start trial" button for connectors (elastic#61774)
  [ML] Transforms: Fix handling of default and advanced search on step summary view. (elastic#61799)
  [Task Manager] Change info message "ran out Available Workers" to debug (elastic#62083)
  [Maps] Highlight selected layer in TOC (elastic#61510)
  ensure pageIndex is set correclty in analytics list (elastic#62041)
  [ML] Functional API tests - fix mml request bodies (elastic#62116)
  Fix validation for index threshold when selecting an index (elastic#61615)
  [SIEM][Detection Engine] Adds release notes link and updates one UI section
  [backport] Bump to 5.1.2 (elastic#62117)
  [APM] .apm-agent-configuration is not created if Kibana is started while ES is not ready (elastic#61610)
  [Fleet] Enrollment list page (elastic#61346)
  [ML] Fix maximum default enabled columns for data grid. (elastic#62005)
  [Home][Tutorial] Add Oracle data UI (elastic#61595)
  [APM] Ensure telemetry data matches SO/telemetry mapping (elastic#61957)
  ...
pmuellr added a commit that referenced this pull request Apr 1, 2020
…ug (#62083) (#62161)

resolves #54920

The scenario this message warns about is actually something that we should
expect to happen in a busy system - and we back off as we should.  But
there's no reason it needs to be info-logged, especially as it seems
somewhat frightening.

Changing to debug, so in case we do need this for diagnostic purposes, we
can still get it via logging config.
pmuellr added a commit that referenced this pull request Apr 1, 2020
…ug (#62083) (#62160)

resolves #54920

The scenario this message warns about is actually something that we should
expect to happen in a busy system - and we back off as we should.  But
there's no reason it needs to be info-logged, especially as it seems
somewhat frightening.

Changing to debug, so in case we do need this for diagnostic purposes, we
can still get it via logging config.
@mikecote mikecote added release_note:fix and removed release_note:skip Skip the PR/issue when compiling release notes labels Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backported Feature:Alerting release_note:fix Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v7.7.0 v7.8.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve task manager warning log messaging when all workers are used

6 participants