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

[2.10] Latest tests security analytics #826

Conversation

amsiglan
Copy link
Contributor

@amsiglan amsiglan commented Sep 6, 2023

Description

Latest tests in security analytics plugin

Check List

  • 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.

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
@amsiglan
Copy link
Contributor Author

amsiglan commented Sep 7, 2023

@ruanyl Any idea why the tests are taking so long to run?
Also, can you please review the changes, need them for the 2.10 release

@ruanyl
Copy link
Member

ruanyl commented Sep 7, 2023

@amsiglan should this backport to main?

@amsiglan amsiglan force-pushed the latest-tests-security-analytics branch from 76d7b85 to 45104b8 Compare September 7, 2023 17:34
@peterzhuamazon peterzhuamazon merged commit 327dd23 into opensearch-project:2.x Sep 7, 2023
65 of 67 checks passed
AWSHurneyt pushed a commit to AWSHurneyt/opensearch-dashboards-functional-test that referenced this pull request Oct 15, 2023
* updated tests

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* excluded a couple tests; fixed alerts tests

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

---------

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
ruanyl pushed a commit that referenced this pull request Oct 16, 2023
…930)

* [2.10] Latest tests security analytics (#826)

* updated tests

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* excluded a couple tests; fixed alerts tests

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

---------

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* updated mappings for test index (#841)

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* avoid clicking filter menu btn twice (#849)

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* removed check for url since it differs with and without security (#863)

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* excluded flaky test (#891)

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* Updated security analytics cypress tests for 2.9 patch.

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

* Skipping flaky test.

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

* Updated alerting cypress tests.

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

---------

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
Co-authored-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
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