Change in ovirt-engine[master]: packaging: changed tasks polling in engine-upgrade

juan.hernandez at redhat.com juan.hernandez at redhat.com
Wed Feb 27 08:42:33 UTC 2013


Juan Hernandez has submitted this change and it was merged.

Change subject: packaging: changed tasks polling in engine-upgrade
......................................................................


packaging: changed tasks polling in engine-upgrade

Modified checkRunningTasks: during engine upgrade, sample the current
situation of asynchronous tasks and compensations and continue the
upgrade, exiting the check loop one second after there are no async
tasks or compensations left.
Currently the stopping of asynchronous tasks and compensations
could take 3 minutes so every ~3 minutes
(MAINTENANCE_TASKS_WAIT_PERIOD) if there are async tasks still around,
allows the user to retry or abort.

Other than that it doesn't change the behavior of the script.

Removed MAINTENANCE_TASKS_CYCLES since it's not used anymore.

Fixed a small typo in MSG_ERROR_FAILED_CONVERT_ENGINE_KEY.

Change-Id: I32eaeb2427fcc9c4566ca12bf547d49a308a8937
Bug-Url: https://bugzilla.redhat.com/877749
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
---
M packaging/fedora/setup/engine-upgrade.py
1 file changed, 23 insertions(+), 12 deletions(-)

Approvals:
  Sandro Bonazzola: Verified
  Juan Hernandez: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I32eaeb2427fcc9c4566ca12bf547d49a308a8937
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgoldboi at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>



More information about the Engine-commits mailing list