Skip to content

[TEST] Opensearch client v3 with v2 image #2763

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Arsnael
Copy link
Contributor

@Arsnael Arsnael commented Jul 3, 2025

Let the CI confirm wether or not the image upgrade is a breaking change or not with current client: #2762 (comment)

@chibenwa
Copy link
Contributor

chibenwa commented Jul 8, 2025

org.awaitility.core.ConditionTimeoutException: Condition with Lambda expression in org.apache.james.backends.opensearch.DockerOpenSearchExtension$DeleteAllIndexDocumentsCleanupStrategy was not fulfilled within 30 seconds.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:26)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1160)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1129)
	at org.apache.james.backends.opensearch.DockerOpenSearchExtension$DeleteAllIndexDocumentsCleanupStrategy.clean(DockerOpenSearchExtension.java:64)
	at org.apache.james.backends.opensearch.DockerOpenSearchExtension.afterEach(DockerOpenSearchExtension.java:107)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)

Can we rely on a harsher isolation strategy in order to move forward?

The fact that the test suite did pass mailbox/opensearch likely means it is safe enough.

Thoughts?

@Arsnael
Copy link
Contributor Author

Arsnael commented Jul 9, 2025

Likely believe so. Run tests locally by class and it's green. So I'm thinking of an isolation issue as well. Weird though that with opensearch 3.1 image i don't seem to have much this issue though

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants