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] Fix debug log for missing ISM config index #850

Merged
merged 1 commit into from
Jul 28, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 99df725 from #846

* Does not sweep if cluster doesn't contain an ISM Config Index

Signed-off-by: Ronnak Saxena <ronsax@amazon.com>

* fixed the warn message

Signed-off-by: Ronnak Saxena <ronsax@amazon.com>

* changed log from warn to debug

Signed-off-by: Ronnak Saxena <ronsax@amazon.com>

---------

Signed-off-by: Ronnak Saxena <ronsax@amazon.com>
(cherry picked from commit 99df725)
@codecov
Copy link

codecov bot commented Jul 10, 2023

Codecov Report

Merging #850 (99824a9) into 2.x (ccd01b1) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x     #850      +/-   ##
============================================
+ Coverage     75.94%   75.98%   +0.04%     
+ Complexity     2907     2905       -2     
============================================
  Files           366      366              
  Lines         16558    16561       +3     
  Branches       2397     2398       +1     
============================================
+ Hits          12575    12584       +9     
+ Misses         2597     2588       -9     
- Partials       1386     1389       +3     
Impacted Files Coverage Δ
...nt/indexstatemanagement/ManagedIndexCoordinator.kt 68.67% <100.00%> (-1.47%) ⬇️

... and 10 files with indirect coverage changes

@AWSHurneyt AWSHurneyt merged commit a02ea78 into 2.x Jul 28, 2023
13 of 14 checks passed
@github-actions github-actions bot deleted the backport/backport-846-to-2.x branch July 28, 2023 14:01
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