Skip to content

Conversation

jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Oct 4, 2021

The Elasticsearch ingest pipeline docs cover pipelines for Fleet and
Elastic Agent. However, the docs don't cover Beats. This adds 'em.

Relates to elastic/beats#28239.

Preview

https://elasticsearch_78633.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/ingest.html#pipelines-for-beats

The Elasticsearch ingest pipeline docs cover ingest pipelines for Fleet and
Elastic Agent. However, the docs don't cover Beats. This adds those docs.

Relates to elastic/beats#28239.
@jrodewig jrodewig changed the title [DOCS] Add xrefs for Beats pipelines [DOCS] Add Beats config example for ingest pipelines Oct 4, 2021
@jrodewig jrodewig marked this pull request as ready for review October 4, 2021 18:49
@jrodewig jrodewig added :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >docs General docs changes v7.15.1 v7.16.0 labels Oct 4, 2021
@elasticmachine elasticmachine added Team:Docs Meta label for docs team Team:Data Management Meta label for data/management team labels Oct 4, 2021
@elasticmachine
Copy link
Collaborator

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

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

Copy link
Contributor

@dedemorton dedemorton left a comment

Choose a reason for hiding this comment

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

Very minor suggestion; otherwise LGTM. Thanks for doing this!

Co-authored-by: DeDe Morton <dede.morton@elastic.co>
@jrodewig jrodewig added auto-backport-and-merge auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Oct 5, 2021
@elasticsearchmachine elasticsearchmachine merged commit 5c7fac7 into elastic:master Oct 5, 2021
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

elasticsearchmachine pushed a commit that referenced this pull request Oct 5, 2021
* [DOCS] Add Beats config example for ingest pipelines

The Elasticsearch ingest pipeline docs cover ingest pipelines for Fleet and
Elastic Agent. However, the docs don't cover Beats. This adds those docs.

Relates to elastic/beats#28239.

* Update docs/reference/ingest.asciidoc

Co-authored-by: DeDe Morton <dede.morton@elastic.co>

Co-authored-by: DeDe Morton <dede.morton@elastic.co>

Co-authored-by: DeDe Morton <dede.morton@elastic.co>
elasticsearchmachine pushed a commit that referenced this pull request Oct 5, 2021
* [DOCS] Add Beats config example for ingest pipelines

The Elasticsearch ingest pipeline docs cover ingest pipelines for Fleet and
Elastic Agent. However, the docs don't cover Beats. This adds those docs.

Relates to elastic/beats#28239.

* Update docs/reference/ingest.asciidoc

Co-authored-by: DeDe Morton <dede.morton@elastic.co>

Co-authored-by: DeDe Morton <dede.morton@elastic.co>

Co-authored-by: DeDe Morton <dede.morton@elastic.co>
jrodewig added a commit to elastic/beats that referenced this pull request Oct 5, 2021
## What does this PR do?

Changes several 'ingest node' references in the docs to 'ingest pipeline.'

## Why is it important?

While a node with the `ingest` role is required to use Elasticsearch ingest pipelines, we no longer include 'ingest node' in the feature name.

There are no official plans, but the Elasticsearch team has discussed removing the `ingest` role in the future.

## Related issues

- elastic/elasticsearch#70253
- elastic/elasticsearch#78633

Co-authored-by: dedemorton <dede.morton@elastic.co>
jrodewig added a commit to elastic/beats that referenced this pull request Oct 6, 2021
Changes several 'ingest node' references in the docs to 'ingest pipeline.'

While a node with the `ingest` role is required to use Elasticsearch ingest pipelines, we no longer include 'ingest node' in the feature name.

There are no official plans, but the Elasticsearch team has discussed removing the `ingest` role in the future.

- elastic/elasticsearch#70253
- elastic/elasticsearch#78633

Co-authored-by: dedemorton <dede.morton@elastic.co>
@jrodewig jrodewig deleted the docs__add-beats-pipeline-xref branch October 12, 2021 12:30
Icedroid pushed a commit to Icedroid/beats that referenced this pull request Nov 1, 2021
## What does this PR do?

Changes several 'ingest node' references in the docs to 'ingest pipeline.'

## Why is it important?

While a node with the `ingest` role is required to use Elasticsearch ingest pipelines, we no longer include 'ingest node' in the feature name.

There are no official plans, but the Elasticsearch team has discussed removing the `ingest` role in the future.

## Related issues

- elastic/elasticsearch#70253
- elastic/elasticsearch#78633

Co-authored-by: dedemorton <dede.morton@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >docs General docs changes Team:Data Management Meta label for data/management team Team:Docs Meta label for docs team v7.15.1 v7.16.0 v8.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants