Douglas Schilling Landgraf has posted comments on this change. Change subject: API: setHaMaintenanceMode command ......................................................................
Patch Set 3: Code-Review+1 (1 comment) http://gerrit.ovirt.org/#/c/23264/3/client/vdsClient.py File client/vdsClient.py: Line 1445: return 0, '' Line 1446: Line 1447: def do_setHaMaintenanceMode(self, args): Line 1448: mode = args[0] Line 1449: enabled = utils.tobool(args[1]) > Thanks for the feedback! It's not as clear/succinct, but args[1] is conver Re-reading the code, you are good. :-) Thanks! Line 1450: stats = self.s.setHaMaintenanceMode(mode, enabled) Line 1451: if stats['status']['code']: Line 1452: return stats['status']['code'], stats['status']['message'] Line 1453: return 0, '' -- To view, visit http://gerrit.ovirt.org/23264 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic08c5edb0e9b8cc11eb70ef6a66301335c42aad3 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Greg Padgett <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Doron Fediuck <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Greg Padgett <[email protected]> Gerrit-Reviewer: Oved Ourfali <[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
