Skip to content

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

@Arsnael Arsnael closed this Jul 23, 2025
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