[Backport 2.x] Set up Rest Integration Test framework #302
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![opensearch-trigger-bot[bot]](https://avatars.githubusercontent.com/u/80134844?s=60&v=4){kind=small}
* Setting up rest integration tests Signed-off-by: Joshua Palis <jpalis@amazon.com> * removing stray log Signed-off-by: Joshua Palis <jpalis@amazon.com> * Cleaning up integration test example Signed-off-by: Joshua Palis <jpalis@amazon.com> * Fixing provision transport action to respond only after state has been updated to PROVISIONING Signed-off-by: Joshua Palis <jpalis@amazon.com> * Fixing flaky encryption test Signed-off-by: Joshua Palis <jpalis@amazon.com> * cleaning up old logs Signed-off-by: Joshua Palis <jpalis@amazon.com> * Added helper methods to retrieve state and resources created, fixed integration test set up to wait for ml config index to become created, fixed settings update to oly occur once Signed-off-by: Joshua Palis <jpalis@amazon.com> * Adding another test for update API, input validation, local model registration. Persiting cluster settings between test runs to ensure plugin apis are enabled. Cleaning up resources after all test runs complete, rather than between test runs Signed-off-by: Joshua Palis <jpalis@amazon.com> * Adding test for search workflows API, ensures that returned credentials are encrypted Signed-off-by: Joshua Palis <jpalis@amazon.com> * Update integ test TODOs to match current development progress Signed-off-by: Daniel Widdis <widdis@gmail.com> * Model Group step is not yet implemented Signed-off-by: Daniel Widdis <widdis@gmail.com> * Comment out tests for incomplete register local model implementation Signed-off-by: Daniel Widdis <widdis@gmail.com> * Fix unit tests broken with changes to fix integ tests Signed-off-by: Daniel Widdis <widdis@gmail.com> --------- Signed-off-by: Joshua Palis <jpalis@amazon.com> Signed-off-by: Daniel Widdis <widdis@gmail.com> Co-authored-by: Daniel Widdis <widdis@gmail.com> (cherry picked from commit 7986cbf) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot
bot
requested review from
dbwiddis,
owaiskazi19,
joshpalis,
ohltyler,
amitgalitz and
jackiehanyang
as code owners
dbwiddis
approved these changes
Dec 19, 2023
ohltyler
approved these changes
Dec 19, 2023
|
Uh, this had failing tests with some 3.0 constants not in 2.x, might need to revert it. |
|
(I probably shouldn't have approved until I saw the tests complete. Sorry.) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport 7986cbf from #251.