Change in ovirt-hosted-engine-setup[master]: packaging: setup: avoid endless loop when setVmTicket fails

sbonazzo at redhat.com sbonazzo at redhat.com
Wed Aug 7 07:18:40 UTC 2013


Sandro Bonazzola has submitted this change and it was merged.

Change subject: packaging: setup: avoid endless loop when setVmTicket fails
......................................................................


packaging: setup: avoid endless loop when setVmTicket fails

- avoid endless loop when setVmTicket fails
- use private variable for range iterations
  (pylint warned about unused variable i)

Change-Id: I664fb5ce7620bda4f8e8d6d46a0b024aecae149a
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/vm/runvm.py
1 file changed, 25 insertions(+), 10 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/17539
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I664fb5ce7620bda4f8e8d6d46a0b024aecae149a
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde at gmail.com>
Gerrit-Reviewer: Moran Goldboim <mgoldboi at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>



More information about the Engine-commits mailing list