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

Posted to notifications@geode.apache.org by "upthewaterspout (GitHub)" <gi...@apache.org> on 2018/10/22 22:24:22 UTC

[GitHub] [geode] upthewaterspout commented on pull request #2679: GEODE-5910: Disabling acceptance tests in StressNewTest

You're right, some acceptance tests can't run in docker. In concourse, AcceptanceTest doesn't pass flags to enable docker, so it did pass for this PR!

I think it's better behavior for someone running these targets to get docker if they ask for docker, rather than having the docker flags (but not the parallel forks!!!) silently ignored. Most acceptance tests can actually run in docker, just a few fail. In fact, they *should* be run in docker because they use default ports, etc.

[ Full content available at: https://github.com/apache/geode/pull/2679 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org