ganeshmurthy commented on a change in pull request #1147:
URL: https://github.com/apache/qpid-dispatch/pull/1147#discussion_r618006603
##########
File path: tests/system_tests_qdstat.py
##########
@@ -565,6 +565,7 @@ def setUpClass(cls):
cls.router_2 = cls.tester.qdrouterd('test_router_2', config_2,
wait=True)
cls.router_1 = cls.tester.qdrouterd('test_router_1', config_1,
wait=True)
cls.router_1.wait_router_connected('R2')
+ cls.router_2.wait_router_connected('R1')
Review comment:
If you look here -
https://github.com/apache/qpid-dispatch/blob/main/tools/qdstat.in#L1052
The has_nodes() function returns False if the number of nodes is less than
2. And the string "UTC" is being displayed if has_nodes() is True, hence this
PR. Also take a look at this output when we print the output of qdstat, one of
the routers is missing. That being said, I have been unable to reproduce this
problem locally and based on the evidence, I am hoping that this PR would fix
the problem.
--
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
