Skip to content

Allow deletion of custom log type if custom rule index is missing (#767)#814

Merged
eirsep merged 1 commit intoopensearch-project:2.xfrom
goyamegh:2.x
Jan 19, 2024
Merged

Allow deletion of custom log type if custom rule index is missing (#767)#814
eirsep merged 1 commit intoopensearch-project:2.xfrom
goyamegh:2.x

Conversation

@goyamegh
Copy link
Copy Markdown
Collaborator


Description

Backport #767

Issues Resolved

#700

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

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.

…ensearch-project#767)

* Allow deletion of custom log type if custom rule index is missing

Signed-off-by: Megha Goyal <goyamegh@amazon.com>

* Allow custom log type name to be updated when custom rule index is missing

Signed-off-by: Megha Goyal <goyamegh@amazon.com>

* Adding changes for detector index missing

Signed-off-by: Megha Goyal <goyamegh@amazon.com>

* Update log type when detector index is missing

Signed-off-by: Megha Goyal <goyamegh@amazon.com>

---------

Signed-off-by: Megha Goyal <goyamegh@amazon.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 18, 2024

Codecov Report

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

Comparison is base (8990aed) 24.83% compared to head (cb21b80) 24.88%.
Report is 1 commits behind head on 2.x.

Files Patch % Lines
...s/transport/TransportIndexCustomLogTypeAction.java 0.00% 54 Missing ⚠️
.../transport/TransportDeleteCustomLogTypeAction.java 0.00% 34 Missing ⚠️
...opensearch/securityanalytics/util/RuleIndices.java 0.00% 12 Missing ⚠️
...analytics/transport/TransportDeleteRuleAction.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                2.x     #814      +/-   ##
============================================
+ Coverage     24.83%   24.88%   +0.05%     
  Complexity     1024     1024              
============================================
  Files           277      277              
  Lines         12683    12656      -27     
  Branches       1389     1390       +1     
============================================
  Hits           3150     3150              
+ Misses         9267     9240      -27     
  Partials        266      266              

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

@eirsep eirsep merged commit 541b99b into opensearch-project:2.x Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants