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.x] DocLevel Monitor - generate findings when 0 triggers #895

Closed
wants to merge 2 commits into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 4ba7d42 from #856

Signed-off-by: Petar Dzepina <[email protected]>
(cherry picked from commit 4ba7d42)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team May 2, 2023 08:15
@codecov-commenter
Copy link

codecov-commenter commented May 2, 2023

Codecov Report

Merging #895 (9175634) into 2.x (493139e) will increase coverage by 0.04%.
Report is 31 commits behind head on 2.x.
The diff coverage is 85.18%.

❗ Current head 9175634 differs from pull request most recent head f19be3b. Consider uploading reports for the commit f19be3b to get more accurate results

@@             Coverage Diff              @@
##                2.x     #895      +/-   ##
============================================
+ Coverage     76.64%   76.68%   +0.04%     
  Complexity      116      116              
============================================
  Files           125      125              
  Lines          7061     7078      +17     
  Branches       1034     1036       +2     
============================================
+ Hits           5412     5428      +16     
- Misses         1123     1125       +2     
+ Partials        526      525       -1     
Files Changed Coverage Δ
.../opensearch/alerting/DocumentLevelMonitorRunner.kt 83.01% <85.18%> (-0.38%) ⬇️

... and 3 files with indirect coverage changes

@lezzago lezzago self-requested a review August 2, 2023 21:33
@lezzago
Copy link
Member

lezzago commented Aug 2, 2023

This was merged as part of #907

@lezzago lezzago closed this Aug 2, 2023
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.

4 participants