Change in vdsm[master]: virt: periodic: port updateVolume

danken at redhat.com danken at redhat.com
Tue Mar 31 10:44:03 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: virt: periodic: port updateVolume
......................................................................


virt: periodic: port updateVolume

During the lifetime of a VM, its volumes need
to be updated. This is a task carried by the
'updateVolumes' operation, which was formerly
piggybacked in the sampling threads.

But actually 'updateVolumes' is a perfect candidate
to motivy the need of a 'periodic operations' infrastructure
to run bookkeeping tasks of the VMs.

So, this patch bootstraps the usage of the newly
added 'periodic' infrastructure by moving 'updateVolumes'
away from sampling threads, and making it a periodic operation.

Change-Id: Idf5f4bff03acf4fb43eb6d296e4c02a6b7a52baa
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/37594
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/virt/periodic.py
M vdsm/virt/vm.py
2 files changed, 34 insertions(+), 14 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified



-- 
To view, visit https://gerrit.ovirt.org/37594
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Idf5f4bff03acf4fb43eb6d296e4c02a6b7a52baa
Gerrit-PatchSet: 31
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list