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

Improve provisioning exceptions with step ID and hide exception details #515

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

dbwiddis
Copy link
Member

@dbwiddis dbwiddis commented Feb 9, 2024

Description

Handles exceptions during the provisioning workflow better:

  • includes the workflow step ID that caused the exception
  • Includes only the exception class type but not the exception message details

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.

Copy link

codecov bot commented Feb 9, 2024

Codecov Report

Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (33ff2f3) 72.01% compared to head (1ec10df) 72.03%.

Files Patch % Lines
...rk/transport/ProvisionWorkflowTransportAction.java 0.00% 9 Missing ⚠️
...framework/indices/FlowFrameworkIndicesHandler.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #515      +/-   ##
============================================
+ Coverage     72.01%   72.03%   +0.02%     
  Complexity      624      624              
============================================
  Files            78       78              
  Lines          3162     3161       -1     
  Branches        239      239              
============================================
  Hits           2277     2277              
+ Misses          780      779       -1     
  Partials        105      105              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@owaiskazi19 owaiskazi19 left a comment

Choose a reason for hiding this comment

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

Looks fine for the quick fix

@dbwiddis dbwiddis merged commit 861616a into opensearch-project:main Feb 9, 2024
32 of 39 checks passed
@dbwiddis dbwiddis deleted the provision-exceptions branch February 9, 2024 23:53
opensearch-trigger-bot bot pushed a commit that referenced this pull request Feb 9, 2024
…ls (#515)

Signed-off-by: Daniel Widdis <[email protected]>
(cherry picked from commit 861616a)
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 Feb 9, 2024
…ls (#515)

Signed-off-by: Daniel Widdis <[email protected]>
(cherry picked from commit 861616a)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
dbwiddis pushed a commit that referenced this pull request Feb 10, 2024
…exception details (#518)

Improve provisioning exceptions with step ID and hide exception details (#515)


(cherry picked from commit 861616a)

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>
joshpalis added a commit that referenced this pull request Feb 10, 2024
… exception details (#519)

Improve provisioning exceptions with step ID and hide exception details (#515)


(cherry picked from commit 861616a)

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>
Co-authored-by: Joshua Palis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants