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

Update telemetry device docs #1247

Merged
merged 2 commits into from
Apr 21, 2021

Conversation

dliappis
Copy link
Contributor

Telemetry device docs are a little out of date wrt which ones to attach
when setting up the cluster vs at runtime.

Given that the new cluster management commands are experimental we take
the opportunity to briefly clarify which category of telemetry devices
needs to be specified via the start subcommand.

In a future PR we could enhance telemetry devices to have an additional
(internal) property, that is presented via the list subcommand, to
explicitly classify their usage level i.e. setup vs runtime.

Telemetry device docs are a little out of date wrt which ones to attach
when setting up the cluster vs at runtime.

Given that the new cluster management commands are experimental we take
the opportunity to briefly clarify which category of telemetry devices
needs to be specified via the start subcommand.

In a future PR we could enhance telemetry devices to have an additional
(internal) property, that is presented via the `list` subcommand, to
explicitly classify their usage level i.e. setup vs runtime.
@dliappis dliappis added enhancement Improves the status quo :Docs Changes to the documentation labels Apr 21, 2021
@dliappis dliappis added this to the 2.2.0 milestone Apr 21, 2021
@dliappis dliappis self-assigned this Apr 21, 2021
Copy link
Member

@danielmitterdorfer danielmitterdorfer left a comment

Choose a reason for hiding this comment

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

There is one small syntax error but apart from that LGTM. No need for another review cycle after addressing.


.. note::

If you are using the experimental `cluster management commands <cluster_management>`_, setup level telemetry devices (and their parameters) should only be specified via the ``start`` subcommand and not via the `race`` subcommand. For more details check ``esrally start --help``.
Copy link
Member

Choose a reason for hiding this comment

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

Nit: You need to enclose the race subcommand in two backticks but there is only one at the beginning.

@dliappis dliappis merged commit 95630ad into elastic:master Apr 21, 2021
@dliappis dliappis deleted the clarify-telemetry-devices branch April 21, 2021 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Docs Changes to the documentation enhancement Improves the status quo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants