opensearch-project / cross-cluster-replication

Synchronize your data across multiple clusters for lower latencies and higher availability
https://opensearch.org/docs/latest/replication-plugin/index/
Apache License 2.0
47 stars 57 forks source link

Disabling docrep to remote store migration test for integTestRemote #1379

Closed astute-decipher closed 3 months ago

astute-decipher commented 4 months ago

Description

Disabling docrep to remote store migration test for integTestRemote suite, since it's failing to pass opensearch.experimental.feature.remote_store.migration.enabled feature flag.

Issues Resolved

[List any issues this PR will resolve]

Check List

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.

astute-decipher commented 4 months ago

Change looks okay to me. How did we verify that integTestRemote is excluding this test method?

While running the individual test : ./gradlew ':integTestRemote' --tests "org.opensearch.replication.integ.rest.StartReplicationIT.test operations are fetched from lucene when leader is in mixed mode"

it fails with error :

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':integTestRemote'.
> No tests found for given includes: [**/*IT.class](include rules) [**/*$*.class](exclude rules) [org.opensearch.replication.integ.rest.StartReplicationIT.test operations are fetched from lucene when leader is in mixed mode](--tests filter)

Whereas we don't see this error, before the change. cc : @ankitkala

nisgoel-amazon commented 4 months ago

We can also run the integTestRemote complete suite by creating 2 clusters using docker and by passing the appropriate parameter


./gradlew integTestRemote -Dleader.http_host=172.18.0.2:9200 -Dfollower.http_host=172.18.0.3:9201 -Dfollower.transport_host=172.18.0.3:9301  -Dleader.transport_host=172.18.0.2:9300  -Dsecurity_enabled=false -Duser=admin -Dpassword=<Password of host> -PnumNodes=1