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

Fix for snapshot tests race condition #1113

Merged

Conversation

CaptainDredge
Copy link
Contributor

Description

This PR adds fix for AssociatedComponentsModal snapshot test failure

Issues Resolved

NA

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: Prabhat Sharma <ptsharma@amazon.com>
@SuZhou-Joe SuZhou-Joe merged commit 883da5a into opensearch-project:main Aug 16, 2024
9 of 11 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 16, 2024
Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>
(cherry picked from commit 883da5a)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
astute-decipher pushed a commit to astute-decipher/index-management-dashboards-plugin that referenced this pull request Aug 16, 2024
Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>
SuZhou-Joe pushed a commit that referenced this pull request Aug 17, 2024
* Initial commit for header update for snapshot management policy page

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* some correction

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* final commit

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* Update pages snapshots (#1111)

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>

* Fix for snapshot test race condition (#1113)

Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>

* Few refactoring

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* resolving UT build failures

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

---------

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Shubh Sahu <shubhvs@amazon.com>
Co-authored-by: Sandeep Kumawat <2025sandeepkumawat@gmail.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Prabhat <20185657+CaptainDredge@users.noreply.github.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 17, 2024
* Initial commit for header update for snapshot management policy page

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* some correction

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* final commit

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* Update pages snapshots (#1111)

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>

* Fix for snapshot test race condition (#1113)

Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>

* Few refactoring

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* resolving UT build failures

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

---------

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Shubh Sahu <shubhvs@amazon.com>
Co-authored-by: Sandeep Kumawat <2025sandeepkumawat@gmail.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Prabhat <20185657+CaptainDredge@users.noreply.github.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>
(cherry picked from commit 12f28d1)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
astute-decipher added a commit to astute-decipher/index-management-dashboards-plugin that referenced this pull request Aug 17, 2024
…ect#1108)

* Initial commit for header update for snapshot management policy page

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* some correction

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* final commit

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* Update pages snapshots (opensearch-project#1111)

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>

* Fix for snapshot test race condition (opensearch-project#1113)

Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>

* Few refactoring

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* resolving UT build failures

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

---------

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Shubh Sahu <shubhvs@amazon.com>
Co-authored-by: Sandeep Kumawat <2025sandeepkumawat@gmail.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Prabhat <20185657+CaptainDredge@users.noreply.github.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>
SuZhou-Joe pushed a commit that referenced this pull request Aug 19, 2024
(cherry picked from commit 883da5a)

Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>
SuZhou-Joe pushed a commit that referenced this pull request Aug 19, 2024
* Initial commit for header update for snapshot management policy page

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* some correction

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* final commit

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* Update pages snapshots (#1111)

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>

* Fix for snapshot test race condition (#1113)

Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>

* Few refactoring

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

* resolving UT build failures

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>

---------

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Shubh Sahu <shubhvs@amazon.com>
Co-authored-by: Sandeep Kumawat <2025sandeepkumawat@gmail.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Prabhat <20185657+CaptainDredge@users.noreply.github.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>
(cherry picked from commit 12f28d1)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
SuZhou-Joe pushed a commit that referenced this pull request Aug 19, 2024
* Initial commit for header update for snapshot management policy page



* some correction



* final commit



* Update pages snapshots (#1111)




* Fix for snapshot test race condition (#1113)




* Few refactoring



* resolving UT build failures



---------









(cherry picked from commit 12f28d1)

Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Shubh Sahu <shubhvs@amazon.com>
Co-authored-by: Sandeep Kumawat <2025sandeepkumawat@gmail.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Prabhat <20185657+CaptainDredge@users.noreply.github.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants