-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Reset isPipelineResolved to false to resolve the system ingest pipeline again. #18911
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
Reset isPipelineResolved to false to resolve the system ingest pipeline again. #18911
Conversation
fbde32d to
8d40807
Compare
|
❌ Gradle check result for eb1ee06: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
eb1ee06 to
a779b1d
Compare
|
❌ Gradle check result for a779b1d: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
server/src/main/java/org/opensearch/ingest/SystemIngestPipelineCache.java
Outdated
Show resolved
Hide resolved
a779b1d to
9e49930
Compare
|
❌ Gradle check result for 1f284ad: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
❌ Gradle check result for 1f284ad: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
@bzhangam can you run |
…ne again. Signed-off-by: Bo Zhang <bzhangam@amazon.com>
1f284ad to
8119208
Compare
fixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thanks @bzhangam
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #18911 +/- ##
============================================
+ Coverage 72.74% 72.83% +0.09%
- Complexity 68840 68868 +28
============================================
Files 5601 5601
Lines 316394 316395 +1
Branches 45872 45872
============================================
+ Hits 230155 230450 +295
+ Misses 67599 67230 -369
- Partials 18640 18715 +75 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…ne again. (opensearch-project#18911) Signed-off-by: Bo Zhang <bzhangam@amazon.com>
Description
Reset isPipelineResolved to false to resolve the system ingest pipeline again. If we don't reset it as false we simply skip resolving the pipeline.
Related Issues
Resolves #18909
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.