Nir Soffer has posted comments on this change. Change subject: virt: Make boolean values from boolean migration options ......................................................................
Patch Set 1: -Code-Review (1 comment) https://gerrit.ovirt.org/#/c/65007/1/vdsm/virt/migration.py File vdsm/virt/migration.py: Line 111: kwargs.get('maxBandwidth') or Line 112: config.getint('vars', 'migration_max_bandwidth') Line 113: ) Line 114: self._autoConverge = utils.tobool(autoConverge) Line 115: self._compressed = utils.tobool(compressed) > Nir, you are right; and there has been a long-standing effort to normalize OK, lets have a TODO here for now. Line 116: self._incomingLimit = kwargs.get('incomingLimit') Line 117: self._outgoingLimit = kwargs.get('outgoingLimit') Line 118: self.status = { Line 119: 'status': { -- To view, visit https://gerrit.ovirt.org/65007 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I642eb607785a1b6f877092e187c91b7a065b38e1 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org