nicoloboschi opened a new pull request #12694:
URL: https://github.com/apache/pulsar/pull/12694
### Motivation
I often see `ElasticSearchClientTests` tests failing in this way
`org.testng.internal.thread.ThreadTimeoutException: Method
org.apache.pulsar.io.elasticsearch.ElasticSearchClientTests.testBulkRetry()
didn't finish within the time-out 300000`
The problem is when you stop ChaosContainer, you have to wait the docker
container to stop the current chaos command (`pause` in this case). Currenty,
if the `stop()` method is called before the actual `stop pause`, the
ElasticSearch container is stopped forever and the subsequent tests will fail.
### Modifications
Before stopping `ChaosContainer`, we wait for the `stop pause` command
checking the container logs.
### Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
### Documentation
Need to update docs?
- [x] `no-need-doc`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
