-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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
Autoscaling APIs not experimental #67202
Autoscaling APIs not experimental #67202
Conversation
We no longer regard the autoscaling APIs experimental though they are only intended for use by ESS/ECE/ECK. This commit updates the docs to reflect this and adds a minimal set of documentation for the feature.
d1d5761
to
3fbf3b2
Compare
Pinging @elastic/es-distributed (Team:Distributed) |
Pinging @elastic/es-docs (Team:Docs) |
Autoscaling APIs are no longer considered experimental and no longer hidden behind a feature flags. Updated the API specification accordingly. Relates elastic#67202 and elastic#65973
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good overall. I left several suggestions, but feel free to ignore those if wanted.
Two larger pieces of non-blocking feedback:
- I'd add a sentence or two about what deciders are and how autoscaling uses them to the top-level page. There isn't a conceptual intro to deciders in the docs currently.
- I'd create a separate
Parameters
orOptions
section for configuration options in each decider reference page.
docs/reference/autoscaling/deciders/reactive-storage-decider.asciidoc
Outdated
Show resolved
Hide resolved
docs/reference/autoscaling/deciders/reactive-storage-decider.asciidoc
Outdated
Show resolved
Hide resolved
docs/reference/autoscaling/apis/get-autoscaling-capacity.asciidoc
Outdated
Show resolved
Hide resolved
docs/reference/autoscaling/apis/get-autoscaling-capacity.asciidoc
Outdated
Show resolved
Hide resolved
docs/reference/autoscaling/apis/get-autoscaling-capacity.asciidoc
Outdated
Show resolved
Hide resolved
docs/reference/autoscaling/apis/get-autoscaling-capacity.asciidoc
Outdated
Show resolved
Hide resolved
single configuration option `forecast_window` is available, defaulting to 30 | ||
minutes. | ||
|
||
The estimation of expected additional data is based on past indexing that | ||
occurred within the `forecast_window`, i.e., by default the last 30 minutes. | ||
Only indexing into data streams contributes to the estimate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'd typically put this in a Parameters
or Options
section with a def for forecast_window
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did this in d0429bd
Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
…sciidoc Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
…sciidoc Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
…asciidoc Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
…asciidoc Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
…sciidoc Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
…gandersen/elasticsearch into docs_autoscaling_not_experimental
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a few formatting nits, but this looks great to me. Thanks @henningandersen!
docs/reference/autoscaling/deciders/machine-learning-decider.asciidoc
Outdated
Show resolved
Hide resolved
docs/reference/autoscaling/deciders/machine-learning-decider.asciidoc
Outdated
Show resolved
Hide resolved
docs/reference/autoscaling/deciders/machine-learning-decider.asciidoc
Outdated
Show resolved
Hide resolved
docs/reference/autoscaling/deciders/machine-learning-decider.asciidoc
Outdated
Show resolved
Hide resolved
Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
docs/reference/autoscaling/autoscaling-designed-for-note.asciidoc
Outdated
Show resolved
Hide resolved
Co-authored-by: Jason Tedor <jason@tedor.me>
We no longer regard the autoscaling APIs experimental though they are only intended for use by ESS/ECE/ECK. This commit updates the docs to reflect this and adds a minimal set of documentation for the feature. Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
We no longer regard the autoscaling APIs experimental though they are only intended for use by ESS/ECE/ECK. This commit updates the docs to reflect this and adds a minimal set of documentation for the feature. Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
We no longer regard the autoscaling APIs experimental though they are
only intended for use by ESS/ECE/ECK. This commit updates the docs
to reflect this and adds a minimal set of documentation for the
feature.