Skipping destination migration if alerting index is not initialized #417

thalurur · 2022-04-20T20:07:00Z

Signed-off-by: Ravi Thaluru thalurur@users.noreply.github.com

Issue #, if available:
N/A

Description of changes:
Keep seeing following errors on cluster - skipping Destination migration if alert index is not initialized

[2022-04-20T20:04:42,752][ERROR][o.o.a.u.d.DestinationMigrationCoordinator] [dev-dsk-rthaluru-2b-5db44ff7.us-west-2.amazon.com] Failed to migrate destination data
org.opensearch.index.IndexNotFoundException: no such index [.opendistro-alerting-config]
        at org.opensearch.cluster.metadata.IndexNameExpressionResolver$WildcardExpressionResolver.indexNotFoundException(IndexNameExpressionResolver.java:1055) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.cluster.metadata.IndexNameExpressionResolver$WildcardExpressionResolver.innerResolve(IndexNameExpressionResolver.java:992) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.cluster.metadata.IndexNameExpressionResolver$WildcardExpressionResolver.resolve(IndexNameExpressionResolver.java:948) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.cluster.metadata.IndexNameExpressionResolver.concreteIndices(IndexNameExpressionResolver.java:252) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.cluster.metadata.IndexNameExpressionResolver.concreteIndices(IndexNameExpressionResolver.java:228) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.search.TransportSearchAction.resolveLocalIndices(TransportSearchAction.java:872) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.search.TransportSearchAction.executeSearch(TransportSearchAction.java:913) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.search.TransportSearchAction.executeLocalSearch(TransportSearchAction.java:757) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.search.TransportSearchAction.lambda$executeRequest$3(TransportSearchAction.java:398) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.ActionListener$1.onResponse(ActionListener.java:78) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.index.query.Rewriteable.rewriteAndFetch(Rewriteable.java:136) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.index.query.Rewriteable.rewriteAndFetch(Rewriteable.java:101) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.search.TransportSearchAction.executeRequest(TransportSearchAction.java:487) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.search.TransportSearchAction.doExecute(TransportSearchAction.java:277) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.search.TransportSearchAction.doExecute(TransportSearchAction.java:120) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.support.TransportAction$RequestFilterChain.proceed(TransportAction.java:194) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.indexmanagement.rollup.actionfilter.FieldCapsFilter.apply(FieldCapsFilter.kt:118) ~[?:?]
        at org.opensearch.action.support.TransportAction$RequestFilterChain.proceed(TransportAction.java:192) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.performanceanalyzer.action.PerformanceAnalyzerActionFilter.apply(PerformanceAnalyzerActionFilter.java:78) ~[?:?]
        at org.opensearch.action.support.TransportAction$RequestFilterChain.proceed(TransportAction.java:192) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.security.filter.SecurityFilter.apply0(SecurityFilter.java:272) ~[?:?]
        at org.opensearch.security.filter.SecurityFilter.apply(SecurityFilter.java:157) ~[?:?]
        at org.opensearch.action.support.TransportAction$RequestFilterChain.proceed(TransportAction.java:192) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.support.TransportAction.execute(TransportAction.java:169) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.action.support.TransportAction.execute(TransportAction.java:97) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.client.node.NodeClient.executeLocally(NodeClient.java:108) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.client.node.NodeClient.doExecute(NodeClient.java:95) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.client.support.AbstractClient.execute(AbstractClient.java:425) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.client.support.AbstractClient.search(AbstractClient.java:556) ~[opensearch-2.0.0-rc1.jar:2.0.0-rc1]
        at org.opensearch.alerting.util.destinationmigration.DestinationMigrationUtilService$Companion$retrieveDestinationsToMigrate$response$1.invoke(DestinationMigrationUtilService.kt:161) ~[opensearch-alerting-2.0.0.0-rc1.jar:2.0.0.0-rc1]
        at org.opensearch.alerting.util.destinationmigration.DestinationMigrationUtilService$Companion$retrieveDestinationsToMigrate$response$1.invoke(DestinationMigrationUtilService.kt:161) ~[opensearch-alerting-2.0.0.0-rc1.jar:2.0.0.0-rc1]
        at org.opensearch.alerting.opensearchapi.OpenSearchExtensionsKt.suspendUntil(OpenSearchExtensions.kt:189) ~[alerting-core-2.0.0.0-rc1.jar:?]
        at org.opensearch.alerting.util.destinationmigration.DestinationMigrationUtilService$Companion.retrieveDestinationsToMigrate(DestinationMigrationUtilService.kt:161) ~[opensearch-alerting-2.0.0.0-rc1.jar:2.0.0.0-rc1]
        at org.opensearch.alerting.util.destinationmigration.DestinationMigrationUtilService$Companion.migrateDestinations(DestinationMigrationUtilService.kt:67) ~[opensearch-alerting-2.0.0.0-rc1.jar:2.0.0.0-rc1]
        at org.opensearch.alerting.util.destinationmigration.DestinationMigrationCoordinator$initMigrateDestinations$scheduledJob$1$1.invokeSuspend(DestinationMigrationCoordinator.kt:89) [opensearch-alerting-2.0.0.0-rc1.jar:2.0.0.0-rc1]
        at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33) [kotlin-stdlib-1.6.10.jar:1.6.10-release-923(1.6.10)]
        at kotlinx.coroutines.DispatchedTask.run(Dispatched.kt:233) [kotlinx-coroutines-core-1.1.1.jar:?]
        at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:594) [kotlinx-coroutines-core-1.1.1.jar:?]
        at kotlinx.coroutines.scheduling.CoroutineScheduler.access$runSafely(CoroutineScheduler.kt:60) [kotlinx-coroutines-core-1.1.1.jar:?]
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:742) [kotlinx-coroutines-core-1.1.1.jar:?]

CheckList:
[X] 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.

codecov-commenter · 2022-04-20T20:18:26Z

Codecov Report

Merging #417 (d176183) into main (caf5855) will increase coverage by 0.20%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #417      +/-   ##
============================================
+ Coverage     76.66%   76.86%   +0.20%     
  Complexity      279      279              
============================================
  Files           183      183              
  Lines          8537     8542       +5     
  Branches       1237     1239       +2     
============================================
+ Hits           6545     6566      +21     
+ Misses         1375     1359      -16     
  Partials        617      617

Impacted Files	Coverage Δ
...n/kotlin/org/opensearch/alerting/AlertingPlugin.kt	`93.46% <100.00%> (ø)`
...nationmigration/DestinationMigrationCoordinator.kt	`63.82% <100.00%> (+4.30%)`	⬆️
...ing/model/destination/DestinationContextFactory.kt	`75.00% <0.00%> (-3.58%)`	⬇️
...destinationmigration/DestinationConversionUtils.kt	`68.88% <0.00%> (-1.12%)`	⬇️
.../kotlin/org/opensearch/alerting/core/JobSweeper.kt	`73.68% <0.00%> (-0.48%)`	⬇️
...ain/kotlin/org/opensearch/alerting/AlertService.kt	`78.40% <0.00%> (-0.47%)`	⬇️
...lin/org/opensearch/alerting/alerts/AlertIndices.kt	`73.65% <0.00%> (+3.90%)`	⬆️
.../alerting/transport/TransportIndexMonitorAction.kt	`66.66% <0.00%> (+4.65%)`	⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update caf5855...d176183. Read the comment docs.

Signed-off-by: Ravi Thaluru <thalurur@users.noreply.github.com>

alerting/src/main/kotlin/org/opensearch/alerting/alerts/AlertIndices.kt

Signed-off-by: Ravi Thaluru <thalurur@users.noreply.github.com>

…pensearch-project#417) Signed-off-by: Ravi Thaluru <thalurur@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

…pensearch-project#417) Signed-off-by: Ravi Thaluru <6005951+thalurur@users.noreply.github.com>

…pensearch-project#417) Signed-off-by: Ravi 6005951+thalurur@users.noreply.github.com

…pensearch-project#417) Signed-off-by: Ravi 6005951+thalurur@users.noreply.github.com Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* fix security test workflow (#407) Signed-off-by: Subhobrata Dey <sbcd90@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Integrate Document Level Alerting changes (#410) * Rebase to push doc level changes on latest main changes (#391) * Document level alerting dev (#272) Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * Add last run context to Monitor data model Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * add Update Monitor function Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * fix integ test Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * Implemented draft of Finding data model, a new Input type, and some basic unit tests. (#260) * Implemented draft of Finding data model, and some basic unit tests for it. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * POC for doc-level-alerting (#277) Signed-off-by: Sriram <59816283+skkosuri-amzn@users.noreply.github.com> * Add connection to triggers for doc level alerting (#316) Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * CRUD APIs integration Tests and validation"conflict resolved" (#362) Signed-off-by: charliezhangaws <zhanncha@amazon.com> * Segregate monitor runner logic for separation of concerns (#363) * Refactor monitor runner logic for separation of concerns and better testability. Signed-off-by: Saurabh Singh <getsaurabh02@gmail.com> * Add action and alert flow and findings schema and additional fixes (#381) Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * Finding Search API (#385) * Findings search API based on Annie's work Signed-off-by: Annie Lee <leeyun@amazon.com> * Fix Search API and add IT tests Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> Co-authored-by: Annie Lee <leeyun@amazon.com> * Fix integ tests and minor issues from doc level changes Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> Co-authored-by: Annie Lee <71157062+leeyun-amzn@users.noreply.github.com> Co-authored-by: Daniel Doubrovkine (dB.) <dblock@dblock.org> Co-authored-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Co-authored-by: Sriram <59816283+skkosuri-amzn@users.noreply.github.com> Co-authored-by: charliezhangaws <zhanncha@amazon.com> Co-authored-by: Saurabh Singh <getsaurabh02@gmail.com> Co-authored-by: Annie Lee <leeyun@amazon.com> * Add Trigger condition resolver which parses and evaluates the Trigger expression. (#405) Signed-off-by: Saurabh Singh <sisurab@amazon.com> * percolate query implementation in doc-level alerting (#399) Signed-off-by: Subhobrata Dey <sbcd90@gmail.com> * Finding Index rollover (#408) * Finding Index rollover Signed-off-by: jiahe zhang <zhanncha@amazon.com> * Apply fixes to make rollover work Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> Co-authored-by: jiahe zhang <zhanncha@amazon.com> Co-authored-by: Annie Lee <71157062+leeyun-amzn@users.noreply.github.com> Co-authored-by: Daniel Doubrovkine (dB.) <dblock@dblock.org> Co-authored-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Co-authored-by: Sriram <59816283+skkosuri-amzn@users.noreply.github.com> Co-authored-by: charliezhangaws <zhanncha@amazon.com> Co-authored-by: Saurabh Singh <getsaurabh02@gmail.com> Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: Saurabh Singh <sisurab@amazon.com> Co-authored-by: Subhobrata Dey <sbcd90@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Remove write Destination APIs (#412) * Remove write Destination API REST handlers Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Remove write Destination transport actions Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Remove write Destination action, request and response classes Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Create Alerting config index if it doesn't exist before legacy Destination indexing Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Remove write Destination related security tests Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Remove unused access roles and imports Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Deprecate the Master nomenclature in 2.0 (#415) Signed-off-by: Subhobrata Dey <sbcd90@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Remove Alerting's notification subproject (#413) * Remove notification subproject Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Remove publishing to maven in build script Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Skipping destination migration if alerting index is not initialized (#417) Signed-off-by: Ravi 6005951+thalurur@users.noreply.github.com Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Alias support for Document Level Monitors (#416) * Implemented support for defining doc level monitors using aliases. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Fix integ tests and cleaup alias logic Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> Co-authored-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Fixed a flaky test condition. (#375) Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Replace checked-in ZIP for bwc tests with a dynamic dependency (#411) * Replace checked-in ZIP with a dynamic dependency Signed-off-by: Subhobrata Dey <sbcd90@gmail.com> * Replace checked-in ZIP with a dynamic dependency Signed-off-by: Subhobrata Dey <sbcd90@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Update integTest gradle scripts to run via remote cluster independently (#418) Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Removed tests that were duplicated while resolving merge conflicts. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Co-authored-by: Subhobrata Dey <sbcd90@gmail.com> Co-authored-by: Ashish Agrawal <ashisagr@amazon.com> Co-authored-by: Annie Lee <71157062+leeyun-amzn@users.noreply.github.com> Co-authored-by: Daniel Doubrovkine (dB.) <dblock@dblock.org> Co-authored-by: Sriram <59816283+skkosuri-amzn@users.noreply.github.com> Co-authored-by: charliezhangaws <zhanncha@amazon.com> Co-authored-by: Saurabh Singh <getsaurabh02@gmail.com> Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: Saurabh Singh <sisurab@amazon.com> Co-authored-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> Co-authored-by: Ravi <6005951+thalurur@users.noreply.github.com>

* fix security test workflow (opensearch-project#407) Signed-off-by: Subhobrata Dey <sbcd90@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Integrate Document Level Alerting changes (opensearch-project#410) * Rebase to push doc level changes on latest main changes (opensearch-project#391) * Document level alerting dev (opensearch-project#272) Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * Add last run context to Monitor data model Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * add Update Monitor function Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * fix integ test Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * Implemented draft of Finding data model, a new Input type, and some basic unit tests. (opensearch-project#260) * Implemented draft of Finding data model, and some basic unit tests for it. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * POC for doc-level-alerting (opensearch-project#277) Signed-off-by: Sriram <59816283+skkosuri-amzn@users.noreply.github.com> * Add connection to triggers for doc level alerting (opensearch-project#316) Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * CRUD APIs integration Tests and validation"conflict resolved" (opensearch-project#362) Signed-off-by: charliezhangaws <zhanncha@amazon.com> * Segregate monitor runner logic for separation of concerns (opensearch-project#363) * Refactor monitor runner logic for separation of concerns and better testability. Signed-off-by: Saurabh Singh <getsaurabh02@gmail.com> * Add action and alert flow and findings schema and additional fixes (opensearch-project#381) Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> * Finding Search API (opensearch-project#385) * Findings search API based on Annie's work Signed-off-by: Annie Lee <leeyun@amazon.com> * Fix Search API and add IT tests Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> Co-authored-by: Annie Lee <leeyun@amazon.com> * Fix integ tests and minor issues from doc level changes Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> Co-authored-by: Annie Lee <71157062+leeyun-amzn@users.noreply.github.com> Co-authored-by: Daniel Doubrovkine (dB.) <dblock@dblock.org> Co-authored-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Co-authored-by: Sriram <59816283+skkosuri-amzn@users.noreply.github.com> Co-authored-by: charliezhangaws <zhanncha@amazon.com> Co-authored-by: Saurabh Singh <getsaurabh02@gmail.com> Co-authored-by: Annie Lee <leeyun@amazon.com> * Add Trigger condition resolver which parses and evaluates the Trigger expression. (opensearch-project#405) Signed-off-by: Saurabh Singh <sisurab@amazon.com> * percolate query implementation in doc-level alerting (opensearch-project#399) Signed-off-by: Subhobrata Dey <sbcd90@gmail.com> * Finding Index rollover (opensearch-project#408) * Finding Index rollover Signed-off-by: jiahe zhang <zhanncha@amazon.com> * Apply fixes to make rollover work Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> Co-authored-by: jiahe zhang <zhanncha@amazon.com> Co-authored-by: Annie Lee <71157062+leeyun-amzn@users.noreply.github.com> Co-authored-by: Daniel Doubrovkine (dB.) <dblock@dblock.org> Co-authored-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Co-authored-by: Sriram <59816283+skkosuri-amzn@users.noreply.github.com> Co-authored-by: charliezhangaws <zhanncha@amazon.com> Co-authored-by: Saurabh Singh <getsaurabh02@gmail.com> Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: Saurabh Singh <sisurab@amazon.com> Co-authored-by: Subhobrata Dey <sbcd90@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Remove write Destination APIs (opensearch-project#412) * Remove write Destination API REST handlers Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Remove write Destination transport actions Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Remove write Destination action, request and response classes Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Create Alerting config index if it doesn't exist before legacy Destination indexing Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Remove write Destination related security tests Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Remove unused access roles and imports Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Deprecate the Master nomenclature in 2.0 (opensearch-project#415) Signed-off-by: Subhobrata Dey <sbcd90@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Remove Alerting's notification subproject (opensearch-project#413) * Remove notification subproject Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Remove publishing to maven in build script Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Skipping destination migration if alerting index is not initialized (opensearch-project#417) Signed-off-by: Ravi 6005951+thalurur@users.noreply.github.com Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Alias support for Document Level Monitors (opensearch-project#416) * Implemented support for defining doc level monitors using aliases. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Fix integ tests and cleaup alias logic Signed-off-by: Ashish Agrawal <ashisagr@amazon.com> Co-authored-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Fixed a flaky test condition. (opensearch-project#375) Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Replace checked-in ZIP for bwc tests with a dynamic dependency (opensearch-project#411) * Replace checked-in ZIP with a dynamic dependency Signed-off-by: Subhobrata Dey <sbcd90@gmail.com> * Replace checked-in ZIP with a dynamic dependency Signed-off-by: Subhobrata Dey <sbcd90@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Update integTest gradle scripts to run via remote cluster independently (opensearch-project#418) Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Removed tests that were duplicated while resolving merge conflicts. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Co-authored-by: Subhobrata Dey <sbcd90@gmail.com> Co-authored-by: Ashish Agrawal <ashisagr@amazon.com> Co-authored-by: Annie Lee <71157062+leeyun-amzn@users.noreply.github.com> Co-authored-by: Daniel Doubrovkine (dB.) <dblock@dblock.org> Co-authored-by: Sriram <59816283+skkosuri-amzn@users.noreply.github.com> Co-authored-by: charliezhangaws <zhanncha@amazon.com> Co-authored-by: Saurabh Singh <getsaurabh02@gmail.com> Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: Saurabh Singh <sisurab@amazon.com> Co-authored-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> Co-authored-by: Ravi <6005951+thalurur@users.noreply.github.com> Signed-off-by: Angie Zhang <langelzh@amazon.com>

thalurur requested a review from a team April 20, 2022 20:07

thalurur marked this pull request as draft April 20, 2022 20:07

thalurur changed the title ~~Not doing destination migration if alerting index is not initialized~~ Skipping destination migration if alerting index is not initialized Apr 20, 2022

Not doing destination migration if alerting index is not initialized

5620074

Signed-off-by: Ravi Thaluru <thalurur@users.noreply.github.com>

thalurur force-pushed the main branch from d176183 to 5620074 Compare April 20, 2022 20:21

thalurur marked this pull request as ready for review April 20, 2022 20:51

lezzago requested changes Apr 20, 2022

View reviewed changes

alerting/src/main/kotlin/org/opensearch/alerting/alerts/AlertIndices.kt Outdated Show resolved Hide resolved

PR comments

cdb897a

Signed-off-by: Ravi Thaluru <thalurur@users.noreply.github.com>

lezzago approved these changes Apr 20, 2022

View reviewed changes

qreshi approved these changes Apr 20, 2022

View reviewed changes

thalurur merged commit f0e8e20 into opensearch-project:main Apr 20, 2022

AWSHurneyt pushed a commit to AWSHurneyt/OpenSearch-Alerting that referenced this pull request Apr 21, 2022

Skipping destination migration if alerting index is not initialized (o…

83fbb75

…pensearch-project#417) Signed-off-by: Ravi Thaluru <6005951+thalurur@users.noreply.github.com>

AWSHurneyt pushed a commit to AWSHurneyt/OpenSearch-Alerting that referenced this pull request Apr 21, 2022

Skipping destination migration if alerting index is not initialized (o…

f263120

…pensearch-project#417) Signed-off-by: Ravi 6005951+thalurur@users.noreply.github.com

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Skipping destination migration if alerting index is not initialized #417

Skipping destination migration if alerting index is not initialized #417

thalurur commented Apr 20, 2022 •

edited

Loading

codecov-commenter commented Apr 20, 2022 •

edited

Loading

Skipping destination migration if alerting index is not initialized #417

Skipping destination migration if alerting index is not initialized #417

Conversation

thalurur commented Apr 20, 2022 • edited Loading

codecov-commenter commented Apr 20, 2022 • edited Loading

Codecov Report

thalurur commented Apr 20, 2022 •

edited

Loading

codecov-commenter commented Apr 20, 2022 •

edited

Loading