[ 
https://issues.apache.org/jira/browse/DISPATCH-2266?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17444774#comment-17444774
 ] 

ASF GitHub Bot commented on DISPATCH-2266:
------------------------------------------

kgiusti commented on a change in pull request #1437:
URL: https://github.com/apache/qpid-dispatch/pull/1437#discussion_r750616680



##########
File path: tests/system_tests_edge_router.py
##########
@@ -1895,6 +1895,10 @@ def test_51_link_route_proxy_configured(self):
         if self.skip['test_51'] :
             self.skipTest("Test skipped during development.")
 
+        # The previous test waited for address CfgLinkRoute1 on router INT_B
+        # We will wait for address CfgLinkRoute1 to be unsubscribed
+        self.INT_B.wait_address_unsubscribed("CfgLinkRoute1")

Review comment:
       I agree with @jiridanek that the proper fix would to require each test 
to wait until all addresses introduced by the test have been purged from all 
the routers.  But only having one test do this which happens to be ordered 
before the failing test is brittle in that there's nothing stopping some poor 
unfortunate dev from re-naming tests and inadvertently re-breaking this 
particular test. 




-- 
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: dev-unsubscr...@qpid.apache.org

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


> system_tests_edge_router stalls on Travis CI
> --------------------------------------------
>
>                 Key: DISPATCH-2266
>                 URL: https://issues.apache.org/jira/browse/DISPATCH-2266
>             Project: Qpid Dispatch
>          Issue Type: Test
>          Components: Tests
>    Affects Versions: 1.17.0
>            Reporter: Ganesh Murthy
>            Assignee: Ganesh Murthy
>            Priority: Major
>
> The system_tests_edge_router stalls on Travis CI intermittently
> {noformat}
> 55: test_50_link_topology (system_tests_edge_router.LinkRouteProxyTest) ... ok
> No output has been received in the last 10m0s, this potentially indicates a 
> stalled build or something wrong with the build itself.
> Check the details on how to adjust your build configuration on: 
> https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
> The build has been terminated
>  {noformat}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org

Reply via email to