Yaniv Bronhaim has posted comments on this change.
Change subject: lib: vdscli: switch to the new devel_warn helper ......................................................................
Patch Set 2: Code-Review+1
(1 comment)
https://gerrit.ovirt.org/#/c/62211/2/lib/vdsm/vdscli.py File lib/vdsm/vdscli.py:
Line 139: Line 140: Line 141: devel_warn Line 142: "vdscli uses xmlrpc. since ovirt 3.6 xmlrpc is deprecated, please use " Line 143: "vdsm.jsonrpcvdscli")
We don't need this warning, adding a comment that the module is deprecated
Dan requested for that, and I agree that it helps to notice when vdscli is being used. Line 144: Line 145: Line 146: if __name__ == '__main__': Line 147: print('connecting to %s:%s ssl %s ts %s' % (
vdsm-patches@lists.fedorahosted.org