-
Notifications
You must be signed in to change notification settings - Fork 31
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
Fix process node zero timeout and handle not found exceptions #411
Conversation
Signed-off-by: Daniel Widdis <[email protected]>
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #411 +/- ##
============================================
- Coverage 71.95% 71.83% -0.13%
Complexity 613 613
============================================
Files 79 79
Lines 3081 3082 +1
Branches 238 237 -1
============================================
- Hits 2217 2214 -3
- Misses 760 763 +3
- Partials 104 105 +1 ☔ View full report in Codecov by Sentry. |
Thanks for making this fix! Been trying to look at it for a while with no luck finding root cause. I was wondering if you understood why this was happening only on multi node consistently |
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 for the fix @dbwiddis
src/main/java/org/opensearch/flowframework/transport/DeprovisionWorkflowTransportAction.java
Show resolved
Hide resolved
Signed-off-by: Daniel Widdis <[email protected]> (cherry picked from commit c6e6ed2) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…eptions (#412) Fix process node zero timeout and handle not found exceptions (#411) (cherry picked from commit c6e6ed2) Signed-off-by: Daniel Widdis <[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>
Description
Fixes a bug in deprovisioning where the
ProcessNode
had no timeout and blocked threads.Also handles exceptionally completed futures with an
OpenSearchStatusException
withNOT_FOUND
status as successful deprovisions.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.