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.9] [Backport 2.x] Implemented support for configuring a cluster metrics monitor to call cat/indices, and cat/shards. #992 #1011

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 84e8b00 from #1009

…monitor to call cat/indices, and cat/shards. #992 (#1009)

* Implemented support for configuring a cluster metrics monitor to call cat/indices, and cat/shards. (#992)

* Implemented support for configuring a cluster metrics monitor to call cat/indices, and cat/shards.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Fixed ktlint errors.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Refactored executeTransportAction to use suspendUntil() instead of get() to receive responses.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

---------

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Resolved merge conflicts.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Fixed ktlint errors.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Refactored API calls from suspendUntil() to get().

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

---------

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
(cherry picked from commit 84e8b00)
@codecov
Copy link

codecov bot commented Jul 12, 2023

Codecov Report

Merging #1011 (0971bfc) into 2.9 (57b2ebf) will decrease coverage by 1.58%.
The diff coverage is 56.67%.

@@             Coverage Diff              @@
##                2.9    #1011      +/-   ##
============================================
- Coverage     74.14%   72.57%   -1.58%     
  Complexity      119      119              
============================================
  Files           158      160       +2     
  Lines          9460    10424     +964     
  Branches       1364     1569     +205     
============================================
+ Hits           7014     7565     +551     
- Misses         1756     1967     +211     
- Partials        690      892     +202     
Impacted Files Coverage Δ
.../clusterMetricsMonitorHelpers/CatIndicesHelpers.kt 55.23% <55.23%> (ø)
...l/clusterMetricsMonitorHelpers/CatShardsHelpers.kt 57.00% <57.00%> (ø)
...lpers/SupportedClusterMetricsSettingsExtensions.kt 54.16% <71.87%> (ø)
...ain/kotlin/org/opensearch/alerting/InputService.kt 89.88% <100.00%> (-0.44%) ⬇️
...erting/settings/SupportedClusterMetricsSettings.kt 20.00% <100.00%> (+3.72%) ⬆️
.../kotlin/org/opensearch/alerting/util/IndexUtils.kt 77.04% <100.00%> (+0.38%) ⬆️

... and 4 files with indirect coverage changes

@AWSHurneyt AWSHurneyt merged commit 354a795 into 2.9 Jul 12, 2023
10 of 16 checks passed
@github-actions github-actions bot deleted the backport/backport-1009-to-2.9 branch July 12, 2023 07:16
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