Integ test failure fix #1437

riysaxen-amzn · 2024-02-20T02:30:10Z

*Added sleep of some 30s to fix integ Test failure -> https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/7884/pipeline/

java.lang.AssertionError: Unexpected status expected:<FORBIDDEN> but was:<NOT_FOUND>

        at __randomizedtesting.SeedInfo.seed([B335CFE7D740BC9:7370413AD3F31CC0]:0)

        at org.junit.Assert.fail(Assert.java:89)

        at org.junit.Assert.failNotEquals(Assert.java:835)

        at org.junit.Assert.assertEquals(Assert.java:120)

        at org.opensearch.alerting.resthandler.SecureMonitorRestApiIT.test create monitor with an user without index read role(SecureMonitorRestApiIT.kt:256)

CheckList:

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.

* Added 2.1 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Added 2.1 release notes. Signed-off-by: AWSHurneyt <[email protected]>

As per gradle [docs] add support to remove timestamps and package with same order which is required from [reproducible] builds [docs]: https://docs.gradle.org/current/userguide/working_with_files.html#sec:archives [reproducible]: https://reproducible-builds.org/ Signed-off-by: Leonidas Spyropoulos <[email protected]>

* Version increment automation Signed-off-by: pgodithi <[email protected]> * Version increment automation: task rename updateVersion Signed-off-by: pgodithi <[email protected]>

….1.0.0 zip following deprecation of ODFE. (opensearch-project#510) Signed-off-by: AWSHurneyt <[email protected]>

* Bumping 2.x branch version from 2.1.0 to 2.2.0. Bumped mockito-core version from 4.3.1 to 4.6.1. (opensearch-project#506) Signed-off-by: AWSHurneyt <[email protected]> * Added 2.2 release notes. (opensearch-project#514) * Added 2.2 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Added 2.2 release notes. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]>

* “fix#921-README-forum-link” Signed-off-by: cwillum <[email protected]> * “fix#921-README-forum-link” Signed-off-by: cwillum <[email protected]> Signed-off-by: cwillum <[email protected]>

Signed-off-by: Subhobrata Dey <[email protected]>

…indices (opensearch-project#526) * Adds support in monitor configs for custom alerts, findings, queries indices Signed-off-by: Surya Sashank Nistala <[email protected]>

…s transport layer (opensearch-project#561) * adds support for fetching alerts from custom alert index in Get Alerts transport layer

* added monitorId and findingsIndexName optinal params to GetFindingsAction Signed-off-by: Petar Dzepina <[email protected]>

…roject#570) Signed-off-by: Peter Zhu <[email protected]> Signed-off-by: Peter Zhu <[email protected]>

Signed-off-by: Surya Sashank Nistala <[email protected]>

…nsearch-project#584) Signed-off-by: Surya Sashank Nistala <[email protected]>

Signed-off-by: Surya Sashank Nistala <[email protected]>

…rch-project#592) * Added 2.1 release notes. (opensearch-project#485) (opensearch-project#487) * Added 2.1 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Added 2.1 release notes. Signed-off-by: AWSHurneyt <[email protected]> (cherry picked from commit 9f3c393) Co-authored-by: AWSHurneyt <[email protected]> * Staging for version increment automation (opensearch-project#489) (opensearch-project#503) * Version increment automation Signed-off-by: pgodithi <[email protected]> * Version increment automation: task rename updateVersion Signed-off-by: pgodithi <[email protected]> (cherry picked from commit 3b8bfe7) Signed-off-by: prudhvigodithi <[email protected]> Co-authored-by: Prudhvi Godithi <[email protected]> * Add support for reproducible builds (opensearch-project#472) (opensearch-project#499) As per gradle [docs] add support to remove timestamps and package with same order which is required from [reproducible] builds [docs]: https://docs.gradle.org/current/userguide/working_with_files.html#sec:archives [reproducible]: https://reproducible-builds.org/ Signed-off-by: Leonidas Spyropoulos <[email protected]> (cherry picked from commit a27d5a2) Signed-off-by: prudhvigodithi <[email protected]> Co-authored-by: Leonidas Spyropoulos <[email protected]> * Refactored backwards compatibility tests to point to the OpenSearch 1.1.0.0 zip following deprecation of ODFE. (opensearch-project#510) (opensearch-project#512) Signed-off-by: AWSHurneyt <[email protected]> (cherry picked from commit fea6b4a) Co-authored-by: AWSHurneyt <[email protected]> * Bumping 2.x branch version from 2.1.0 to 2.2.0. Bumped mockito-core version from 4.3.1 to 4.6.1. (opensearch-project#506) Signed-off-by: AWSHurneyt <[email protected]> * Added 2.2 release notes. (opensearch-project#514) * Added 2.2 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Added 2.2 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Bumped 2.x branch from version 2.2.0 to 2.3.0. (opensearch-project#547) * Bumped 2.x branch from version 2.2.0 to 2.3.0. Signed-off-by: AWSHurneyt <[email protected]> * Bumped mockito core version. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Deprecate the Master nomenclature (opensearch-project#548) (opensearch-project#549) Signed-off-by: Subhobrata Dey <[email protected]> * Added 2.3 release notes. (opensearch-project#551) * Added 2.3 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Added 2.3 release notes. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * moving over data models from alerting to common-utils (opensearch-project#556) Signed-off-by: Subhobrata Dey <[email protected]> * backport monitor datasources to 2.x (opensearch-project#558) Signed-off-by: Surya Sashank Nistala <[email protected]> * expose delete monitor api from alerting (opensearch-project#568) Signed-off-by: Subhobrata Dey <[email protected]> * Use findings and alerts models, dtos from common utils (opensearch-project#569) Signed-off-by: Surya Sashank Nistala <[email protected]> * Disable ktlint for alerting as it has errors on Windows (opensearch-project#570) (opensearch-project#571) Signed-off-by: Peter Zhu <[email protected]> Signed-off-by: Peter Zhu <[email protected]> (cherry picked from commit 3a844f8) Co-authored-by: Peter Zhu <[email protected]> * Recreate request object from writeable for Get alerts and get findings (opensearch-project#577) Signed-off-by: Surya Sashank Nistala <[email protected]> * Use alerting models and dtos from common-utils dependencies Signed-off-by: Surya Sashank Nistala <[email protected]> * port over datasources alert index usage correctly Signed-off-by: Surya Sashank Nistala <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: Subhobrata Dey <[email protected]> Signed-off-by: Surya Sashank Nistala <[email protected]> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: AWSHurneyt <[email protected]> Co-authored-by: Prudhvi Godithi <[email protected]> Co-authored-by: Leonidas Spyropoulos <[email protected]> Co-authored-by: Subhobrata Dey <[email protected]> Co-authored-by: Peter Zhu <[email protected]>

…t#598) Signed-off-by: Subhobrata Dey <[email protected]>

Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: prudhvigodithi <[email protected]>

…nsearch-project#590) Signed-off-by: Surya Sashank Nistala <[email protected]>

…endencies (opensearch-project#606) Signed-off-by: Surya Sashank Nistala <[email protected]>

…ensearch-project#608) * suport fetching alerts by alert ids in Get Alerts Action * acknowledge alerts from custom alerts index Signed-off-by: Surya Sashank Nistala <[email protected]>

…ate as AckAlertRequest (opensearch-project#618) Signed-off-by: Surya Sashank Nistala <[email protected]> Signed-off-by: Surya Sashank Nistala <[email protected]>

Signed-off-by: Petar Dzepina <[email protected]>

…#622) Signed-off-by: Surya Sashank Nistala <[email protected]> Signed-off-by: Surya Sashank Nistala <[email protected]>

Signed-off-by: Subhobrata Dey <[email protected]>

…rces (opensearch-project#626) * in case of custom indices, allow moving alerts to history index Signed-off-by: Petar Dzepina <[email protected]> * empty commit Signed-off-by: Petar Dzepina <[email protected]> * added IT for custom datasources alert ack Signed-off-by: Petar Dzepina <[email protected]> Signed-off-by: Petar Dzepina <[email protected]>

…t#628) * refactored DeleteMonitor Action to be synchronious Signed-off-by: Petar Dzepina <[email protected]>

… action on the same node (opensearch-project#631) Signed-off-by: Stevan Buzejic <[email protected]> Signed-off-by: Stevan Buzejic <[email protected]> Co-authored-by: Stevan Buzejic <[email protected]>

Signed-off-by: Surya Sashank Nistala <[email protected]> Signed-off-by: Surya Sashank Nistala <[email protected]>

* Drafted 2.11 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Drafted 2.11 release notes. Signed-off-by: AWSHurneyt <[email protected]> --------- Signed-off-by: AWSHurneyt <[email protected]>

…h-project#1255) Signed-off-by: Subhobrata Dey <[email protected]>

…istribution Build (opensearch-project#1264) * Update get image workflow to use the yml from opensearch-build repo Signed-off-by: Peter Zhu <[email protected]> * Update more Signed-off-by: Peter Zhu <[email protected]> * Remove usages Signed-off-by: Peter Zhu <[email protected]> * Add more Signed-off-by: Peter Zhu <[email protected]> * Restore security test to original as it is not good to run dind for docker Signed-off-by: Peter Zhu <[email protected]> --------- Signed-off-by: Peter Zhu <[email protected]>

@ignore

* Changed @ignore to @AwaitsFix Signed-off-by: Jinlin Zhang <[email protected]> * added bugUrl Signed-off-by: Jinlin Zhang <[email protected]> --------- Signed-off-by: Jinlin Zhang <[email protected]>

…h-project#1271) Signed-off-by: Peter Zhu <[email protected]>

…pensearch-project#1276) Signed-off-by: Surya Sashank Nistala <[email protected]>

…h-project#1242) Signed-off-by: Jacob Choi <[email protected]>

* Added 2.11.1 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Added 2.11.1 release notes. Signed-off-by: AWSHurneyt <[email protected]> --------- Signed-off-by: AWSHurneyt <[email protected]>

Signed-off-by: Subhobrata Dey <[email protected]>

… from common-utils (opensearch-project#1305) * Use get monitor action / req / resp from common-utils Signed-off-by: Tyler Ohlsen <[email protected]> * Clean up remaining references Signed-off-by: Tyler Ohlsen <[email protected]> * Refactor search monitor action/request also Signed-off-by: Tyler Ohlsen <[email protected]> * Rewrite get monitor and search monitor transport action requests Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Tyler Ohlsen <[email protected]>

…arch-project#1302) * optimize doc-level monitor execution for datastreams Signed-off-by: Subhobrata Dey <[email protected]> * add more tests to address comments Signed-off-by: Subhobrata Dey <[email protected]> * add integTest for multiple datastreams inside a single index pattern * add integTest for multiple datastreams inside a single index pattern Signed-off-by: Subhobrata Dey <[email protected]> --------- Signed-off-by: Subhobrata Dey <[email protected]>

Signed-off-by: Chase Engelbrecht <[email protected]>

…alidation (opensearch-project#1330) Signed-off-by: Chase Engelbrecht <[email protected]>

…ct#1346) Signed-off-by: Chase Engelbrecht <[email protected]>

…#1356) Signed-off-by: Riya Saxena <[email protected]>

Signed-off-by: Andriy Redko <[email protected]>

…opensearch-project#1382) Signed-off-by: Tyler Ohlsen <[email protected]>

Co-authored-by: Dennis Toepker <[email protected]>

Signed-off-by: Riya Saxena <[email protected]>

…arch-project#1355) * Bulk index findings and sequentially invoke auto-correlations Signed-off-by: Megha Goyal <[email protected]> * Bulk index findings in batches of 10000 and make it configurable Signed-off-by: Megha Goyal <[email protected]> * Addressing review comments Signed-off-by: Megha Goyal <[email protected]> * Add integ tests to test bulk index findings Signed-off-by: Megha Goyal <[email protected]> * Fix ktlint formatting Signed-off-by: Megha Goyal <[email protected]> --------- Signed-off-by: Megha Goyal <[email protected]>

* Updated alert mappings to accommodate cross-cluster cluster metrics monitors. Signed-off-by: AWSHurneyt <[email protected]> * Implemented support for cross-cluster cluster metrics monitors. Implemented GetRemoteIndexes API to populate the frontend UI with details regarding the remote clusters, and indexes. Signed-off-by: AWSHurneyt <[email protected]> * Fixed a writeable test after changing QueryLevelTriggerRunResult from a data class to an open class for inheritability. Signed-off-by: AWSHurneyt <[email protected]> * Fixed ktlint errors. Signed-off-by: AWSHurneyt <[email protected]> * Removed changes to IndexUtils as they're only needed by doc monitors. Signed-off-by: AWSHurneyt <[email protected]> --------- Signed-off-by: AWSHurneyt <[email protected]>

* Add 2.12 release notes Signed-off-by: Chase Engelbrecht <[email protected]> * Fix release notes PR Signed-off-by: Chase Engelbrecht <[email protected]> * Add 2 more PRs Signed-off-by: Chase Engelbrecht <[email protected]> --------- Signed-off-by: Chase Engelbrecht <[email protected]>

Signed-off-by: AWSHurneyt <[email protected]>

…the object mapping [metadata.source_to_query_index_mapping] (opensearch-project#1432)

Signed-off-by: Riya Saxena <[email protected]>

AWSHurneyt · 2024-02-20T03:14:25Z

Closing in favor of #1438

AWSHurneyt and others added 30 commits July 6, 2022 14:22

Added 2.1 release notes. (opensearch-project#485)

9f3c393

* Added 2.1 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Added 2.1 release notes. Signed-off-by: AWSHurneyt <[email protected]>

Staging for version increment automation (opensearch-project#489)

3b8bfe7

* Version increment automation Signed-off-by: pgodithi <[email protected]> * Version increment automation: task rename updateVersion Signed-off-by: pgodithi <[email protected]>

Refactored backwards compatibility tests to point to the OpenSearch 1…

fea6b4a

….1.0.0 zip following deprecation of ODFE. (opensearch-project#510) Signed-off-by: AWSHurneyt <[email protected]>

Replace Forum link in Alerting plugin README.md (opensearch-project#545)

9218363

* “fix#921-README-forum-link” Signed-off-by: cwillum <[email protected]> * “fix#921-README-forum-link” Signed-off-by: cwillum <[email protected]> Signed-off-by: cwillum <[email protected]>

Deprecate the Master nomenclature (opensearch-project#548)

ac868dd

Signed-off-by: Subhobrata Dey <[email protected]>

Adds support in monitor configs for custom alerts, findings, queries …

5074f7b

…indices (opensearch-project#526) * Adds support in monitor configs for custom alerts, findings, queries indices Signed-off-by: Surya Sashank Nistala <[email protected]>

Adds support for fetching alerts from custom alert index in Get Alert…

415dc82

…s transport layer (opensearch-project#561) * adds support for fetching alerts from custom alert index in Get Alerts transport layer

GetFindingsAction new optional parameters (opensearch-project#563)

4ca0fb0

* added monitorId and findingsIndexName optinal params to GetFindingsAction Signed-off-by: Petar Dzepina <[email protected]>

Disable ktlint for alerting as it has errors on Windows (opensearch-p…

3a844f8

…roject#570) Signed-off-by: Peter Zhu <[email protected]> Signed-off-by: Peter Zhu <[email protected]>

Adds owner field in monitor model (opensearch-project#579)

cda8301

Signed-off-by: Surya Sashank Nistala <[email protected]>

support query deletion from custom query index on delete monitor (ope…

c532061

…nsearch-project#584) Signed-off-by: Surya Sashank Nistala <[email protected]>

Use custom query index in update monitor flow (opensearch-project#591)

2c7c652

Signed-off-by: Surya Sashank Nistala <[email protected]>

Bumps alerting to 2.4 (opensearch-project#596)

19dec99

Signed-off-by: Surya Sashank Nistala <[email protected]>

add tags to trigger condition of doc-level monitor (opensearch-projec…

4165eac

…t#598) Signed-off-by: Subhobrata Dey <[email protected]>

Remove plugin to OS min race condition (opensearch-project#601)

01a28e9

Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: prudhvigodithi <[email protected]>

Enhance Get Alerts and Get Findings for list of monitors in bulk (ope…

f4a3509

…nsearch-project#590) Signed-off-by: Surya Sashank Nistala <[email protected]>

use acknowledge alert request,response, actions from common-utils dep…

e94ea32

…endencies (opensearch-project#606) Signed-off-by: Surya Sashank Nistala <[email protected]>

Support fetching alerts by list of alert ids in Get Alerts Action (op…

3c74744

…ensearch-project#608) * suport fetching alerts by alert ids in Get Alerts Action * acknowledge alerts from custom alerts index Signed-off-by: Surya Sashank Nistala <[email protected]>

searchAlert fix (opensearch-project#613)

996190e

change transport ack alerts action to accept Action Request and recre…

2005185

…ate as AckAlertRequest (opensearch-project#618) Signed-off-by: Surya Sashank Nistala <[email protected]> Signed-off-by: Surya Sashank Nistala <[email protected]>

Custom history indicies (opensearch-project#616)

7546b00

Signed-off-by: Petar Dzepina <[email protected]>

fix alias exists check in findings index creation (opensearch-project…

ce79ffd

…#622) Signed-off-by: Surya Sashank Nistala <[email protected]> Signed-off-by: Surya Sashank Nistala <[email protected]>

fix for windows ktlint issue (opensearch-project#585)

49fbb4d

Signed-off-by: Subhobrata Dey <[email protected]>

refactored DeleteMonitor Action to be synchronious (opensearch-projec…

c6c12db

…t#628) * refactored DeleteMonitor Action to be synchronious Signed-off-by: Petar Dzepina <[email protected]>

Enabled parsing of bucket level monitors when executing the transport…

3491fb2

… action on the same node (opensearch-project#631) Signed-off-by: Stevan Buzejic <[email protected]> Signed-off-by: Stevan Buzejic <[email protected]> Co-authored-by: Stevan Buzejic <[email protected]>

add tests to verify owner field (opensearch-project#637)

80504a9

Signed-off-by: Surya Sashank Nistala <[email protected]> Signed-off-by: Surya Sashank Nistala <[email protected]>

AWSHurneyt and others added 26 commits October 11, 2023 16:21

Drafted 2.11 release notes. (opensearch-project#1251)

3529e03

* Drafted 2.11 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Drafted 2.11 release notes. Signed-off-by: AWSHurneyt <[email protected]> --------- Signed-off-by: AWSHurneyt <[email protected]>

fix for concurrentmodificationexception with linkedhashmap (opensearc…

36d81e4

…h-project#1255) Signed-off-by: Subhobrata Dey <[email protected]>

Changed @ignore to @AwaitsFix (opensearch-project#1245)

c2f45de

* Changed @ignore to @AwaitsFix Signed-off-by: Jinlin Zhang <[email protected]> * added bugUrl Signed-off-by: Jinlin Zhang <[email protected]> --------- Signed-off-by: Jinlin Zhang <[email protected]>

Avoid hardcoding username for uid 1000 on docker container (opensearc…

ca9b75c

…h-project#1271) Signed-off-by: Peter Zhu <[email protected]>

add nested fields param mapping findings index for doc level queries (o…

2146199

…pensearch-project#1276) Signed-off-by: Surya Sashank Nistala <[email protected]>

change Thread.sleep to waitUntil function under test files (opensearc…

a26f4c0

…h-project#1242) Signed-off-by: Jacob Choi <[email protected]>

Added 2.11.1 release notes. (opensearch-project#1306)

06c1b8a

* Added 2.11.1 release notes. Signed-off-by: AWSHurneyt <[email protected]> * Added 2.11.1 release notes. Signed-off-by: AWSHurneyt <[email protected]> --------- Signed-off-by: AWSHurneyt <[email protected]>

fix workflow security tests in alerting (opensearch-project#1310)

32718e8

Signed-off-by: Subhobrata Dey <[email protected]>

Set docData to empty string if actual is null (opensearch-project#1325)

008e076

Signed-off-by: Chase Engelbrecht <[email protected]>

Add token to chime URL used in tests to adhere to new Notifications v…

526433a

…alidation (opensearch-project#1330) Signed-off-by: Chase Engelbrecht <[email protected]>

Don't attempt to parse workflow if it doesn't exist (opensearch-proje…

733fd4e

…ct#1346) Signed-off-by: Chase Engelbrecht <[email protected]>

Removed log entry regarding destination migration (opensearch-project…

48e56bf

…#1356) Signed-off-by: Riya Saxena <[email protected]>

Update to Gradle 8.5 (opensearch-project#1369)

bea2e68

Signed-off-by: Andriy Redko <[email protected]>

Inject namedWriteableRegistry during ser/deser of SearchMonitorAction (…

ffc183f

…opensearch-project#1382) Signed-off-by: Tyler Ohlsen <[email protected]>

removed default admin credentials for alerting (opensearch-project#1399)

3c50f7d

Co-authored-by: Dennis Toepker <[email protected]>

ipaddress lib upgrade as part of cve fix (opensearch-project#1397)

8d59060

Signed-off-by: Riya Saxena <[email protected]>

Add publishToMavenLocal in build.sh (opensearch-project#1418)

4cdc1d1

Added chase as a maintainer. (opensearch-project#1416)

719db46

Signed-off-by: AWSHurneyt <[email protected]>

fix for MapperException[the [enabled] parameter can't be updated for …

5b3e1a2

…the object mapping [metadata.source_to_query_index_mapping] (opensearch-project#1432)

added sleep for flaky tests

57e1820

Signed-off-by: Riya Saxena <[email protected]>

riysaxen-amzn marked this pull request as ready for review February 20, 2024 02:30

AWSHurneyt closed this Feb 20, 2024

riysaxen-amzn deleted the integTestFailureFix branch April 17, 2024 17:05

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Integ test failure fix #1437

Integ test failure fix #1437

riysaxen-amzn commented Feb 20, 2024 •

edited

Loading

AWSHurneyt commented Feb 20, 2024

Integ test failure fix #1437

Integ test failure fix #1437

Conversation

riysaxen-amzn commented Feb 20, 2024 • edited Loading

AWSHurneyt commented Feb 20, 2024

riysaxen-amzn commented Feb 20, 2024 •

edited

Loading