> On Oct. 22, 2014, 2:59 p.m., Kenneth Giusti wrote: > > trunk/qpid/extras/qmf/src/py/qmf/console.py, line 57 > > <https://reviews.apache.org/r/27032/diff/1/?file=728747#file728747line57> > > > > I think this change breaks the API. The Console object is a public > > interface and it is expected to be inherited by the user's application. > > Currently, any application that uses the brokerConnectionFailed will define > > it with only one argument. If console.py invokes it with two (adding the > > new 'e' parameter), won't that cause the application to fail?
Just a quick follow up - the error doesn't have to be passed explicitly - there is a .error on the passed broker itself that contains the error string, and .conn_exc contains the exception itself. - Kenneth ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27032/#review57795 ----------------------------------------------------------- On Oct. 22, 2014, 1:38 p.m., Pavel Moravec wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27032/ > ----------------------------------------------------------- > > (Updated Oct. 22, 2014, 1:38 p.m.) > > > Review request for qpid, Ernie Allen, Kenneth Giusti, and Ted Ross. > > > Bugs: QPID-6177 > https://issues.apache.org/jira/browse/QPID-6177 > > > Repository: qpid > > > Description > ------- > > Simple patch that overwrites default empty brokerConnectionFailed method > raised when connection fails. It makes sense to react only when connecting > for the first time. > > Changes in package imports are irrelevant to this JIRA - I had to do the > change to let qpid-tool to start working on trunk (?). > > As the logic of QMF console is to retry failed connections, I suggest just > raising some warning than directly stopping qpid-tool. > > > Diffs > ----- > > trunk/qpid/extras/qmf/src/py/qmf/console.py 1633165 > trunk/qpid/tools/src/py/qpid-tool 1633165 > > Diff: https://reviews.apache.org/r/27032/diff/ > > > Testing > ------- > > Testing with invalid hostname: > > $ ./tools/src/py/qpid-tool invalidHost > Management Tool for QPID > qpid: Failed to connect: [Errno -2] Name or service not known > > > Testing with no SASL mech in common: > > $ ./tools/src/py/qpid-tool > Management Tool for QPID > qpid: Failed to connect: (None, 'sasl negotiation failed: no mechanism > agreed') > > > Thanks, > > Pavel Moravec > >
