Skip to content

[DOCS] Updates ML decider docs by mentioning CPU as scaling criterion#92018

Merged
szabosteve merged 2 commits intoelastic:mainfrom
szabosteve:update.ml.decider.docs
Nov 30, 2022
Merged

[DOCS] Updates ML decider docs by mentioning CPU as scaling criterion#92018
szabosteve merged 2 commits intoelastic:mainfrom
szabosteve:update.ml.decider.docs

Conversation

@szabosteve
Copy link
Copy Markdown
Contributor

@szabosteve szabosteve commented Nov 30, 2022 •
edited
Loading

Overview

This PR:

  • updates the ML decider docs with the CPU scaling criterion,
  • fixes a typo in the NOTE at the top of the page where "jobs" should be "nodes".

Related issue: https://github.com/elastic/cloud/issues/107710

Preview

ML decider

@szabosteve szabosteve changed the title [DOCS] Updates ML decider docs by mentioning CPU as scaling criterion. [DOCS] Updates ML decider docs by mentioning CPU as scaling criterion Nov 30, 2022
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 30, 2022

Documentation preview:

@szabosteve szabosteve added >docs General docs changes :ml Machine learning Team:Docs Meta label for docs team v8.6.0 v8.7.0 labels Nov 30, 2022
@szabosteve szabosteve marked this pull request as ready for review November 30, 2022 12:12
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

elasticsearchmachine commented Nov 30, 2022

Pinging @elastic/ml-core (Team:ML)

@elasticsearchmachine elasticsearchmachine added the Team:ML Meta label for the ML team label Nov 30, 2022
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

elasticsearchmachine commented Nov 30, 2022

Pinging @elastic/es-docs (Team:Docs)

abdonpijpelink
abdonpijpelink approved these changes Nov 30, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@abdonpijpelink abdonpijpelink left a comment

Choose a reason for hiding this comment

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

LGTM. I left one small suggestion.

@szabosteve @abdonpijpelink
Update docs/reference/autoscaling/deciders/machine-learning-decider.a…
e3fcf41 
…sciidoc

Co-authored-by: Abdon Pijpelink <abdon.pijpelink@elastic.co>
droberts195
droberts195 approved these changes Nov 30, 2022
View reviewed changes
Copy link
Copy Markdown

@droberts195 droberts195 left a comment

Choose a reason for hiding this comment

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

LGTM

@szabosteve szabosteve merged commit 26bc894 into elastic:main Nov 30, 2022
@szabosteve szabosteve deleted the update.ml.decider.docs branch November 30, 2022 12:37
@szabosteve szabosteve added auto-backport Automatically create backport pull requests when merged and removed auto-backport Automatically create backport pull requests when merged labels Nov 30, 2022
@szabosteve szabosteve mentioned this pull request Nov 30, 2022
Merged
@szabosteve
Copy link
Copy Markdown
Contributor Author

szabosteve commented Nov 30, 2022

💚 All backports created successfully

Status Branch Result
✅ 8.6

Questions ?

Please refer to the Backport tool documentation

szabosteve added a commit to szabosteve/elasticsearch that referenced this pull request Nov 30, 2022
@szabosteve @abdonpijpelink
[DOCS] Updates ML decider docs by mentioning CPU as scaling criterion (…
dc27034 
…elastic#92018)

Co-authored-by: Abdon Pijpelink <abdon.pijpelink@elastic.co>
(cherry picked from commit 26bc894)
szabosteve added a commit that referenced this pull request Nov 30, 2022
@szabosteve @abdonpijpelink
[DOCS] Updates ML decider docs by mentioning CPU as scaling criterion (…
0d7fe83 
…#92018) (#92020)

Co-authored-by: Abdon Pijpelink <abdon.pijpelink@elastic.co>
(cherry picked from commit 26bc894)
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Dec 2, 2022
@weizijun
Merge branch 'upstream/main' into rollup-support-cancel
e6e33de 
* upstream/main: (209 commits)
  Remove unused methods and classes from HLRC (elastic#92030)
  Clean up on exception while chunking XContent (elastic#92024)
  Add profiling plugin (elastic#91640)
  Remove unused methods and classes from HLRC (elastic#92012)
  Remove IndexerState from HLRC (elastic#92023)
  Ensure cached time elapses in ClusterServiceIT (elastic#91986)
  Chunked encoding for RestGetIndicesAction (elastic#92016)
  Simplify shardsWithState (elastic#91991)
  [DOCS] Updates ML decider docs by mentioning CPU as scaling criterion (elastic#92018)
  Add chunking to ClusterState.Custom impls (elastic#91963)
  Speedup time_series agg by caching current tsid ordinal, parent bucket ordinal and buck ordinal (elastic#91784)
  Drop the ingest listener call count tracking (elastic#92003)
  [DOCS] fixes issue number 91889 - missing [discrete] header (elastic#91976)
  Fix PersistentTasksClusterServiceTests (elastic#92002)
  [docs] Update search-settings documentation to reflect the fact that the indices.query.bool.max_clause_count setting has been deprecated (elastic#91811)
  Clarify writability in Netty4HttpPipeliningHandler (elastic#91982)
  Load stable plugins as synthetic modules (elastic#91869)
  Handle any exception thrown while generating source for an IngestDocument (elastic#91981)
  fixing Apache HttpHost url on java-rest doc (elastic#91945)
  Implement repair functionality for aliases colliding with indices bug (elastic#91887)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@droberts195 droberts195 droberts195 approved these changes

@abdonpijpelink abdonpijpelink abdonpijpelink approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

>docs General docs changes :ml Machine learning Team:Docs Meta label for docs team Team:ML Meta label for the ML team v8.6.0 v8.7.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@szabosteve @elasticsearchmachine @droberts195 @abdonpijpelink