[GitHub] [geode] demery-pivotal opened a new pull request #6583: GEODE-8772: Fix ClusterComms test port conflicts

[GitBox <gi...@apache.org>]

demery-pivotal opened a new pull request #6583:
URL: https://github.com/apache/geode/pull/6583


   Change `ClusterCommunicationsDUnitTest.performARollingUpgrade()` to
   disable the HTTP service.
   
   BACKGROUND
   
   I am working on a project to allow Geode tests to run in parallel
   outside of Docker. Running in parallel outside of Docker requires that
   tests not start services using default ports.
   
   This commit prepares for those changes.
   
   PROBLEMS
   
   When `ClusterCommunicationsDUnitTest.performARollingUpgrade()` upgrades
   the locator, it starts the new locator with HTTP service enabled on the
   default port (7070). If another test running in parallel also uses this
   default port, at least one will throw an exception when it attempts to
   bind.
   
   THIS COMMIT
   
   Change `ClusterCommunicationsDUnitTest` to start the upgraded locator
   with HTTP service disabled. The test does not use the HTTP service.
   


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

[GitHub] [geode] demery-pivotal merged pull request #6583: GEODE-8772: Fix ClusterComms test port conflicts

[GitBox <gi...@apache.org>]

demery-pivotal merged pull request #6583:
URL: https://github.com/apache/geode/pull/6583


   


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