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.

Sign up for GitHub

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

Jump to bottom

Fix example code for temporary search pipelines #5555

Merged
merged 1 commit into from
Nov 9, 2023
Merged

Fix example code for temporary search pipelines #5555

merged 1 commit into from
Nov 9, 2023

Conversation

upwards-gravity
Copy link
Contributor

Description

Fixes the example code for temporary search pipelines.

Issues Resolved

The temporary pipeline has to be referenced with search_pipeline instead of pipeline in the JSON.

Otherwise, you get the error {"error":{"root_cause":[{"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16}],"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16},"status":400}.

Tested with Docker image opensearchproject/opensearch:2.11.0.

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.

@upwards-gravity
Corrected example for temporary pipeline
a54ccb0 
The temporary pipeline has to be referenced with `search_pipeline` instead of `pipeline` in the JSON.

Otherwise, you get the error `{"error":{"root_cause":[{"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16}],"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16},"status":400}`.

Tested with Docker image `opensearchproject/opensearch:2.11.0`.

Signed-off-by: Tim Vossen <[email protected]>
kolchfa-aws
kolchfa-aws approved these changes Nov 9, 2023
View reviewed changes
Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

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

Thanks, @upwards-gravity!

@kolchfa-aws kolchfa-aws self-assigned this Nov 9, 2023
@kolchfa-aws kolchfa-aws added backport 2.9 PR: Backport label for 2.9 backport 2.10 PR: Backport label for 2.10 backport 2.11 PR: Backport label for 2.11 labels Nov 9, 2023
@kolchfa-aws kolchfa-aws merged commit d8b6f66 into opensearch-project:main Nov 9, 2023
5 checks passed
@upwards-gravity upwards-gravity deleted the patch-1 branch November 9, 2023 15:05
@opensearch-trigger-bot opensearch-trigger-bot bot mentioned this pull request Nov 9, 2023
Merged
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 9, 2023
@github-actions
Corrected example for temporary pipeline (#5555)
87b85d5 
The temporary pipeline has to be referenced with `search_pipeline` instead of `pipeline` in the JSON.

Otherwise, you get the error `{"error":{"root_cause":[{"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16}],"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16},"status":400}`.

Tested with Docker image `opensearchproject/opensearch:2.11.0`.

Signed-off-by: Tim Vossen <[email protected]>
(cherry picked from commit d8b6f66)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot mentioned this pull request Nov 9, 2023
Merged
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 9, 2023
@github-actions
Corrected example for temporary pipeline (#5555)
84d94fb 
The temporary pipeline has to be referenced with `search_pipeline` instead of `pipeline` in the JSON.

Otherwise, you get the error `{"error":{"root_cause":[{"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16}],"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16},"status":400}`.

Tested with Docker image `opensearchproject/opensearch:2.11.0`.

Signed-off-by: Tim Vossen <[email protected]>
(cherry picked from commit d8b6f66)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 9, 2023
@github-actions
Corrected example for temporary pipeline (#5555)
7c020b5 
The temporary pipeline has to be referenced with `search_pipeline` instead of `pipeline` in the JSON.

Otherwise, you get the error `{"error":{"root_cause":[{"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16}],"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16},"status":400}`.

Tested with Docker image `opensearchproject/opensearch:2.11.0`.

Signed-off-by: Tim Vossen <[email protected]>
(cherry picked from commit d8b6f66)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot mentioned this pull request Nov 9, 2023
Merged
@kolchfa-aws kolchfa-aws mentioned this pull request Nov 9, 2023
Closed
1 task
kolchfa-aws pushed a commit that referenced this pull request Nov 9, 2023
@opensearch-trigger-bot @github-actions
Corrected example for temporary pipeline (#5555) (#5558)
539b7fc 
The temporary pipeline has to be referenced with `search_pipeline` instead of `pipeline` in the JSON.

Otherwise, you get the error `{"error":{"root_cause":[{"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16}],"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16},"status":400}`.

Tested with Docker image `opensearchproject/opensearch:2.11.0`.


(cherry picked from commit d8b6f66)

Signed-off-by: Tim Vossen <[email protected]>
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>
kolchfa-aws pushed a commit that referenced this pull request Nov 9, 2023
@opensearch-trigger-bot @github-actions
Corrected example for temporary pipeline (#5555) (#5557)
e1b03f0 
The temporary pipeline has to be referenced with `search_pipeline` instead of `pipeline` in the JSON.

Otherwise, you get the error `{"error":{"root_cause":[{"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16}],"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16},"status":400}`.

Tested with Docker image `opensearchproject/opensearch:2.11.0`.


(cherry picked from commit d8b6f66)

Signed-off-by: Tim Vossen <[email protected]>
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>
kolchfa-aws pushed a commit that referenced this pull request Nov 9, 2023
@opensearch-trigger-bot @github-actions
Corrected example for temporary pipeline (#5555) (#5556)
e4a9e2e 
The temporary pipeline has to be referenced with `search_pipeline` instead of `pipeline` in the JSON.

Otherwise, you get the error `{"error":{"root_cause":[{"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16}],"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16},"status":400}`.

Tested with Docker image `opensearchproject/opensearch:2.11.0`.


(cherry picked from commit d8b6f66)

Signed-off-by: Tim Vossen <[email protected]>
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>
vagimeli pushed a commit that referenced this pull request Dec 21, 2023
@upwards-gravity @vagimeli
Corrected example for temporary pipeline (#5555)
e7fa50d 
The temporary pipeline has to be referenced with `search_pipeline` instead of `pipeline` in the JSON.

Otherwise, you get the error `{"error":{"root_cause":[{"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16}],"type":"parsing_exception","reason":"Unknown key for a START_OBJECT in [pipeline].","line":8,"col":16},"status":400}`.

Tested with Docker image `opensearchproject/opensearch:2.11.0`.

Signed-off-by: Tim Vossen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kolchfa-aws kolchfa-aws kolchfa-aws approved these changes

@hdhalter hdhalter Awaiting requested review from hdhalter

@Naarcha-AWS Naarcha-AWS Awaiting requested review from Naarcha-AWS Naarcha-AWS is a code owner

@vagimeli vagimeli Awaiting requested review from vagimeli

@ananzh ananzh Awaiting requested review from ananzh

@seanneumann seanneumann Awaiting requested review from seanneumann

@AMoo-Miki AMoo-Miki Awaiting requested review from AMoo-Miki AMoo-Miki is a code owner

@natebower natebower Awaiting requested review from natebower natebower is a code owner

Assignees

@kolchfa-aws kolchfa-aws

Labels
backport 2.9 PR: Backport label for 2.9 backport 2.10 PR: Backport label for 2.10 backport 2.11 PR: Backport label for 2.11
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@upwards-gravity @kolchfa-aws