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

Update integTest gradle scripts to run via remote cluster independently #1499

Closed
Tracked by #103
saratvemulapalli opened this issue Nov 11, 2021 · 3 comments
Closed
Tracked by #103
Labels
enhancement New feature or request v2.0.0

Comments

@saratvemulapalli
Copy link
Member

Is your feature request related to a problem?
With the existing implementation of integration tests, opensearch plugins use the same test framework as local integration tests which spin up test clusters by itself. This is a hacky way to override the endpoint from local spun up test clusters (via opensearch.testclusters gradle plugin) with the remote cluster endpoint.

The solution doesn't work when we would like to test the opensearch bundle as part of the release. The test frameworks expect the artifact to be available before we could test them. To clean this up, plugins have to create gradle scripts for remote endpoints to run https://github.com/opensearch-project/opensearch-build#integration-tests for the release cycle.

What solution would you like?
Refer to: opensearch-project/anomaly-detection#298 and make similar changes to the plugin.
Also if you have a custom name to invoke these tests, update the integtest.sh script in https://github.com/opensearch-project/opensearch-build/blob/main/scripts/default/integtest.sh

@dblock
Copy link
Member

dblock commented Apr 18, 2022

Was this done?

@dblock dblock added the v2.0.0 label Apr 18, 2022
@davidlago
Copy link

No. We had a discussion around it, as a result amending the recommendations on the TESTING document at the project level here. TL;DR in some cases it is not practical to run tests pointing to an external cluster if the cluster settings themselves are under test and we need to recreate a variety of scenarios.

@davidlago
Copy link

Closing as the security plugin will not be moving to the infra cluster for the reasons in the discussion linked above. If there are any concerns, let's resuscitate that discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v2.0.0
Projects
None yet
Development

No branches or pull requests

4 participants