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

[DOCS] Refactor ingest pipeline docs #70253

Merged
merged 5 commits into from
Mar 15, 2021
Merged

[DOCS] Refactor ingest pipeline docs #70253

merged 5 commits into from
Mar 15, 2021

Conversation

jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Mar 10, 2021

@jrodewig jrodewig added :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >docs General docs changes v7.12.1 v7.13.0 v8.0.0 v7.11.3 labels Mar 10, 2021
@jrodewig jrodewig marked this pull request as ready for review March 10, 2021 18:58
@elasticmachine elasticmachine added Team:Data Management Meta label for data/management team Team:Docs Meta label for docs team labels Mar 10, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

@elasticmachine
Copy link
Collaborator

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

@jrodewig jrodewig requested review from martijnvg and a team March 10, 2021 18:58
Comment on lines +1 to +2
[[common-log-format-example]]
== Example: Parse logs in the Common Log Format
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note for reviewers:
I included this tutorial as it was part of the ingest node docs in Kibana. I plan to remove those Kibana docs when this is merged. However, I'm not 100% sure we need a long-form tutorial. Let me know your thoughts. I can easily remove it if wanted.

Copy link
Contributor

@debadair debadair left a comment

Choose a reason for hiding this comment

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

This is a nice improvement. I think the long-form example is fine to keep around.

jrodewig pushed a commit that referenced this pull request Jul 23, 2021
These docs were accidentally removed in #70253.
jrodewig added a commit that referenced this pull request Jul 23, 2021
These docs were accidentally removed in #70253.

Co-authored-by: Dan Hermann <[email protected]>
jrodewig added a commit that referenced this pull request Jul 23, 2021
These docs were accidentally removed in #70253.

Co-authored-by: Dan Hermann <[email protected]>
jrodewig added a commit that referenced this pull request Jul 23, 2021
These docs were accidentally removed in #70253.

Co-authored-by: Dan Hermann <[email protected]>
jrodewig added a commit that referenced this pull request Jul 23, 2021
These docs were accidentally removed in #70253.

Co-authored-by: Dan Hermann <[email protected]>
jrodewig added a commit that referenced this pull request Jul 23, 2021
These docs were accidentally removed in #70253.

Co-authored-by: Dan Hermann <[email protected]>
ywangd pushed a commit to ywangd/elasticsearch that referenced this pull request Jul 30, 2021
These docs were accidentally removed in elastic#70253.
jrodewig added a commit to elastic/kibana that referenced this pull request Oct 5, 2021
While Elasticsearch ingest pipelines require a node with the `ingest`
role, we don't need to 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. This also better aligns the Kibana UI with the
Elasticsearch docs.

The PR also makes some related changes to the Kibana docs.

Relates to elastic/elasticsearch#70253.
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 <[email protected]>
jrodewig added a commit to elastic/kibana that referenced this pull request Oct 5, 2021
While Elasticsearch ingest pipelines require a node with the `ingest`
role, we don't need to 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. This also better aligns the Kibana UI with the
Elasticsearch docs.

The PR also makes some related changes to the Kibana docs.

Relates to elastic/elasticsearch#70253.
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 <[email protected]>
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 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
: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.11.3 v7.12.1 v7.13.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants