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

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

astitcher commented on a change in pull request #989:
URL: https://github.com/apache/qpid-dispatch/pull/989#discussion_r560401706



##########
File path: tools/qdstat.in
##########
@@ -44,18 +44,17 @@ from qpid_dispatch_internal.compat import UNICODE
 def get(obj, attr):
     return getattr(obj, attr, None)
 
-class BusManager(Node):
+class BusManager():

Review comment:
       @jiridanek Thanks for the pointers! I was wrong about new style classes. 
I did a quick grep and there are a bunch of old style classes in this code 
base. For leaf classes (actually such as this one) I'm not sure it's that 
important as the differences seem to mostly be about metaclass and inheritance 
features.
   I'll change this PR before I merge it though.




----------------------------------------------------------------
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:
us...@infra.apache.org


> All system tests that run qdstat/qdmanage fail on Ubuntu
> --------------------------------------------------------
>
>                 Key: DISPATCH-1913
>                 URL: https://issues.apache.org/jira/browse/DISPATCH-1913
>             Project: Qpid Dispatch
>          Issue Type: Test
>          Components: Tests
>            Reporter: Ganesh Murthy
>            Assignee: Andrew Stitcher
>            Priority: Major
>
> Lots of tests fail in travis with the following error -
> {noformat}
> AttributeError: 'str' object has no attribute 'path' {noformat}
>  
> For example,
> {noformat}
>  34: 
> ======================================================================34: 
> ERROR: test_01_link_route 
> (system_tests_two_routers.TwoRouterExtensionStateTest)
> 34: Verify non-terminal state and data propagates over a link route
> 34: ----------------------------------------------------------------------
> 34: Traceback (most recent call last):
> 34:   File 
> "/home/travis/build/apache/qpid-dispatch/tests/system_tests_two_routers.py", 
> line 2034, in test_01_link_route
> 34:     self.RouterA.wait_address("RoutieMcRouteFace", remotes=1, count=2)
> 34:   File "/home/travis/build/apache/qpid-dispatch/tests/system_test.py", 
> line 672, in wait_address
> 34:     assert retry(check, **retry_kwargs)
> 34:   File "/home/travis/build/apache/qpid-dispatch/tests/system_test.py", 
> line 141, in retry
> 34:     ret = function()
> 34:   File "/home/travis/build/apache/qpid-dispatch/tests/system_test.py", 
> line 662, in check
> 34:     addrs = self.management.query(
> 34:   File "/home/travis/build/apache/qpid-dispatch/tests/system_test.py", 
> line 504, in management
> 34:     self._management = Node.connect(self.addresses[0], timeout=TIMEOUT)
> 34:   File 
> "/home/travis/build/apache/qpid-dispatch/python/qpid_dispatch/management/client.py",
>  line 128, in connect
> 34:     return Node(Node.connection(url, router, timeout, ssl_domain, sasl,
> 34:   File 
> "/home/travis/build/apache/qpid-dispatch/python/qpid_dispatch/management/client.py",
>  line 143, in __init__
> 34:     self.client = SyncRequestResponse(connection, self.url.path)
> 34: AttributeError: 'str' object has no attribute 'path'
> 34: 
> 34: ======================================================================
> 34: ERROR: test_03_multicast 
> (system_tests_two_routers.TwoRouterExtensionStateTest)
> 34: Verify that disposition state set by the publisher is available to all
> 34: ----------------------------------------------------------------------
> 34: Traceback (most recent call last):
> 34:   File 
> "/home/travis/build/apache/qpid-dispatch/tests/system_tests_two_routers.py", 
> line 2061, in test_03_multicast
> 34:     self.RouterA.wait_address("multicast/thingy", subscribers=3)
> 34:   File "/home/travis/build/apache/qpid-dispatch/tests/system_test.py", 
> line 672, in wait_address
> 34:     assert retry(check, **retry_kwargs)
> 34:   File "/home/travis/build/apache/qpid-dispatch/tests/system_test.py", 
> line 141, in retry
> 34:     ret = function()
> 34:   File "/home/travis/build/apache/qpid-dispatch/tests/system_test.py", 
> line 662, in check
> 34:     addrs = self.management.query(
> 34:   File "/home/travis/build/apache/qpid-dispatch/tests/system_test.py", 
> line 504, in management
> 34:     self._management = Node.connect(self.addresses[0], timeout=TIMEOUT)
> 34:   File 
> "/home/travis/build/apache/qpid-dispatch/python/qpid_dispatch/management/client.py",
>  line 128, in connect
> 34:     return Node(Node.connection(url, router, timeout, ssl_domain, sasl,
> 34:   File 
> "/home/travis/build/apache/qpid-dispatch/python/qpid_dispatch/management/client.py",
>  line 143, in __init__
> 34:     self.client = SyncRequestResponse(connection, self.url.path)
> 34: AttributeError: 'str' object has no attribute 'path'{noformat}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to