Skip to content

Add Integtest.sh for OpenSearch integtest setups#44

Merged
peterzhuamazon merged 2 commits intomainfrom
opensearch-integtest-demo
Jul 9, 2021
Merged

Add Integtest.sh for OpenSearch integtest setups#44
peterzhuamazon merged 2 commits intomainfrom
opensearch-integtest-demo

Conversation

@peterzhuamazon
Copy link
Copy Markdown
Member

@peterzhuamazon peterzhuamazon commented Jun 15, 2021

Description

Add Integtest.sh for OpenSearch integtest setups
This is a file that can be used as an abstraction for integTests to run.
This is better than what ODFE has in which all plugins runs saved in a single yml.

Issues Resolved

#45

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed as 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.

@peterzhuamazon peterzhuamazon force-pushed the opensearch-integtest-demo branch from adf639b to ad2555f Compare June 15, 2021 21:33
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon force-pushed the opensearch-integtest-demo branch from ad2555f to be2bcc5 Compare June 15, 2021 22:10
@jmazanec15
Copy link
Copy Markdown
Member

Is there a parent issue for this @peterzhuamazon?

@peterzhuamazon
Copy link
Copy Markdown
Member Author

peterzhuamazon commented Jun 16, 2021

Is there a parent issue for this @peterzhuamazon?

Here is the issue I submit: #45

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Copy link
Copy Markdown
Member

@vamshin vamshin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks

@peterzhuamazon
Copy link
Copy Markdown
Member Author

Please do not merge this I will merged once all the PRs are up.

@peterzhuamazon peterzhuamazon added the Infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc. label Jul 1, 2021
@peterzhuamazon peterzhuamazon merged commit 6a10fe1 into main Jul 9, 2021
jmazanec15 pushed a commit to jmazanec15/k-NN-1 that referenced this pull request Aug 31, 2021
* Add integtest automation

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* correct the credential option param here

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@jmazanec15 jmazanec15 mentioned this pull request Aug 31, 2021
jmazanec15 added a commit that referenced this pull request Aug 31, 2021
Signed-off-by: John Mazanec <jmazane@amazon.com>

Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
martin-gaievski pushed a commit to martin-gaievski/k-NN that referenced this pull request Mar 7, 2022
* Add integtest automation

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* correct the credential option param here

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
martin-gaievski pushed a commit to martin-gaievski/k-NN that referenced this pull request Mar 7, 2022
* Add integtest automation

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* correct the credential option param here

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Martin Gaievski <gaievski@amazon.com>
martin-gaievski pushed a commit to martin-gaievski/k-NN that referenced this pull request Mar 30, 2022
* Add integtest automation

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* correct the credential option param here

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
jingqimao77-spec pushed a commit to jingqimao77-spec/k-NN that referenced this pull request Mar 15, 2026
* Add integtest automation

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* correct the credential option param here

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement] Add integtest.sh to unify the integTest for k-NN in OpenSearch

3 participants