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

[DOC] Add ingest processors documentation #4299

Merged
merged 286 commits into from
Aug 29, 2023
Merged

[DOC] Add ingest processors documentation #4299

merged 286 commits into from
Aug 29, 2023

Conversation

vagimeli
Copy link
Collaborator

@vagimeli vagimeli commented Jun 9, 2023

Description

Add ingest processors documentation

Issues Resolved

#4193

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@vagimeli

This comment was marked as resolved.

@vagimeli vagimeli self-assigned this Jun 23, 2023
@vagimeli vagimeli linked an issue Jun 23, 2023 that may be closed by this pull request
4 tasks
heemin32

This comment was marked as resolved.

kolchfa-aws

This comment was marked as resolved.

_api-reference/ingest-apis/create-update-ingest.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/ingest-pipelines.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/ingest-pipelines.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/ingest-pipelines.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/ingest-pipelines.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/pipeline-failures.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/pipeline-failures.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/processors-reference/append.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/processors-reference/append.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/processors-reference/append.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/create-ingest.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/create-ingest.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/create-ingest.md Show resolved Hide resolved
_api-reference/ingest-apis/ingest-processors.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/pipeline-failures.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/pipeline-failures.md Outdated Show resolved Hide resolved
_api-reference/ingest-apis/processors/convert.md Outdated Show resolved Hide resolved
@vagimeli vagimeli linked an issue Aug 22, 2023 that may be closed by this pull request
45 tasks
vagimeli and others added 3 commits August 24, 2023 08:34
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
* Update searchable_snapshot.md

This is more in-line with what is defined in the example Opensearch docker-compose
I tried the current approach and it straight-up did not work in it's current form.

Signed-off-by: adaisley <90253063+adaisley@users.noreply.github.com>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

---------

Signed-off-by: adaisley <90253063+adaisley@users.noreply.github.com>
Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
vagimeli and others added 15 commits August 28, 2023 13:44
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Heemin Kim <heemin@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
@vagimeli

This comment was marked as resolved.

@vagimeli
Copy link
Collaborator Author

@natebower I've accepted your suggested changes and your comments. Where you left comments, I rewrote as you suggested.

@heemin32 I've addressed your comments. I accepted your suggested changes and suggestions.

@kolchfa-aws I've addressed your comments. I accepted your changes or rewrote as suggested.

Moving forward, I'll create separate documentation PRs for each processor. Thank you for your patience and thorough multiple reviews.

@vagimeli vagimeli added the backport 2.9 PR: Backport label for 2.9 label Aug 28, 2023
@vagimeli

This comment was marked as resolved.

Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Heemin Kim <heemin@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
@vagimeli vagimeli merged commit 71935cf into main Aug 29, 2023
3 checks passed
@vagimeli vagimeli deleted the processors-4193 branch August 29, 2023 20:07
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 29, 2023
Created new documentation to close content gaps

Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
(cherry picked from commit 71935cf)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
vagimeli pushed a commit that referenced this pull request Aug 29, 2023
Created new documentation to close content gaps


(cherry picked from commit 71935cf)

Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
harshavamsi pushed a commit to harshavamsi/documentation-website that referenced this pull request Oct 31, 2023
Created new documentation to close content gaps

Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
vagimeli added a commit that referenced this pull request Dec 21, 2023
Created new documentation to close content gaps

Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Editorial review PR: Editorial review in progress backport 2.9 PR: Backport label for 2.9 Content gap
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[META DOC] Ingest Processors Content Gaps [DOC] Add ingest processors
7 participants