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

Add missing documentation of --pipelines flag of setup command #7033

Merged

Conversation

kvch
Copy link
Contributor

@kvch kvch commented May 7, 2018

No description provided.

@@ -481,6 +481,9 @@ ifeval::["{beatname_lc}"=="filebeat"]
Specifies a comma-separated list of modules. Use this flag to avoid errors when
there are no modules defined in the +{beatname_lc}.yml+ file.

*`--pipelines`*::
Sets up Ingest pipelines of configured filesets only.
Copy link
Contributor

Choose a reason for hiding this comment

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

ingest should be lower case. I'd change this to say: "Sets up ingest pipelines for configured filesets." (I don't think "only" is needed here.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Thanks for the review.

@kvch kvch force-pushed the fix/filebeat/add-missing-docs-of-setup-cmd branch from 5597a2c to 6d296c1 Compare May 8, 2018 07:03
@ruflin ruflin merged commit c8fb0e2 into elastic:master May 14, 2018
stevea78 pushed a commit to stevea78/beats that referenced this pull request May 20, 2018
stevea78 pushed a commit to stevea78/beats that referenced this pull request May 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants