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

[Backport 2.7] revert to deleting monitor metadata after deleting doc level queries to fix delete monitor regression. #937

Merged
merged 1 commit into from
May 25, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 5e2312d from #931

…to fix regression. Metadata is required for correct query index clean up (#931)

Signed-off-by: Surya Sashank Nistala <[email protected]>
(cherry picked from commit 5e2312d)
@codecov
Copy link

codecov bot commented May 25, 2023

Codecov Report

Merging #937 (637dde0) into 2.7 (edc00ae) will increase coverage by 0.03%.
The diff coverage is 66.25%.

@@             Coverage Diff              @@
##                2.7     #937      +/-   ##
============================================
+ Coverage     76.66%   76.69%   +0.03%     
  Complexity      116      116              
============================================
  Files           125      125              
  Lines          7221     7253      +32     
  Branches       1049     1049              
============================================
+ Hits           5536     5563      +27     
- Misses         1155     1158       +3     
- Partials        530      532       +2     
Impacted Files Coverage Δ
...alerting/transport/TransportDeleteMonitorAction.kt 68.57% <62.74%> (+0.48%) ⬆️
.../alerting/transport/TransportIndexMonitorAction.kt 56.47% <72.41%> (+1.61%) ⬆️

... and 3 files with indirect coverage changes

@eirsep
Copy link
Member

eirsep commented May 25, 2023

CI failing due to fix pending review in #934.

@eirsep eirsep merged commit 19869b0 into 2.7 May 25, 2023
@github-actions github-actions bot deleted the backport/backport-931-to-2.7 branch May 25, 2023 21:24
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.

2 participants