Dan Kenigsberg has posted comments on this change. Change subject: jsonrpc: renaming for pep8 compliance ......................................................................
Patch Set 1: Code-Review+2 (1 comment) https://gerrit.ovirt.org/#/c/41120/1/debian/vdsm.install File debian/vdsm.install: Line 82: ./usr/share/vdsm/protocoldetector.py Line 83: ./usr/share/vdsm/respawn Line 84: ./usr/share/vdsm/rpc/__init__.py Line 85: ./usr/share/vdsm/rpc/bindingjsonrpc.py Line 86: ./usr/share/vdsm/rpc/BindingXMLRPC.py Keeping BindingXMLRPC is a sore in my eye. But let's keep it to another patch. Line 87: ./usr/share/vdsm/rpc/Bridge.py Line 88: ./usr/share/vdsm/rpc/process-schema.py Line 89: ./usr/share/vdsm/rpc/vdsmapi.py Line 90: ./usr/share/vdsm/set-conf-item -- To view, visit https://gerrit.ovirt.org/41120 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If96815423630cb43b58411e6c3e46afead4dfda2 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
