You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/02/19 15:09:37 UTC

[GitHub] [pulsar] bsideup commented on pull request #9626: [Testing] Add debugging feature which leaves integration test containers running after test completes

bsideup commented on pull request #9626:
URL: https://github.com/apache/pulsar/pull/9626#issuecomment-782133879


   @lhotari TBH I would advice against this approach. There is no guarantee that the JVM hook will be using `stop()` (we thought about terminating multiple containers by the session label filter). Not to mention that Ryuk generally should not be disabled, unless your CI does not support it (the reason why the flag was added on a first place).
   
   If you need to the container to remain running, consider using [the reusable containers mode](https://github.com/testcontainers/testcontainers-java/pull/1781).


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org