Dan Kenigsberg has posted comments on this change. Change subject: vdscli: no need to generate using autoconf ......................................................................
Patch Set 2: (2 comments) http://gerrit.ovirt.org/#/c/28676/2/lib/vdsm/vdscli.py File lib/vdsm/vdscli.py: Line 74: pass Line 75: except: Line 76: pass Line 77: Line 78: > Is it possible to drop __guessDefaults() altogether? I'll try to think of a means to rewrite this in a less convoluted way. Line 79: __guessDefaults() Line 80: Line 81: Line 82: def cannonizeHostPort(hostPort=None, port=d_port): Line 117: else: Line 118: transport = wrap_transport(SingleRequestTransport()) Line 119: server = xmlrpclib.Server('http://%s' % hostPort, transport) Line 120: return server Line 121: > Maybe we should load the defaults here: connect() with no args is used extensively (also outside the project) to mean "do whatever needed to connect to the local host". I do not think that we can easily change this behavior (and personally, I find it quite useful). Line 122: if __name__ == '__main__': Line 123: print 'connecting to %s:%s ssl %s ts %s' % (d_addr, d_port, Line 124: d_useSSL, d_tsPath) Line 125: server = connect() -- To view, visit http://gerrit.ovirt.org/28676 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I13059d101068cf6fa292baae4fb93ccf0b30db45 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: mooli tayer <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
