Change in vdsm[master]: virt: periodic: add utilities

fromani at redhat.com fromani at redhat.com
Fri Feb 20 15:47:07 UTC 2015


Francesco Romani has posted comments on this change.

Change subject: virt: periodic: add utilities
......................................................................


Patch Set 5:

(2 comments)

notes to self

http://gerrit.ovirt.org/#/c/37885/5//COMMIT_MSG
Commit Message:

Line 3: AuthorDate: 2015-02-17 16:39:47 +0100
Line 4: Commit:     Francesco Romani <fromani at redhat.com>
Line 5: CommitDate: 2015-02-20 10:38:52 +0100
Line 6: 
Line 7: virt: periodic: add utilities
too generic. Mention at least VmDispatcher.
Line 8: 
Line 9: Add utilities to be used by future patches
Line 10: moving operations from VmStatsThread.
Line 11: 


http://gerrit.ovirt.org/#/c/37885/5/vdsm/virt/periodic.py
File vdsm/virt/periodic.py:

Line 179: # When do we dispatch the next event?
Line 180: # Is it possible that the next event will be dispatched before
Line 181: # the first event was finished?
Line 182: # If a vm is stuck, this may cause libvirt queues to fill up
Line 183: # with more jobs for this vm.
will be addressed by the next upload
Line 184: class VmDispatcher(object):
Line 185:     """
Line 186:     Adapter class. Dispatch an Operation to all VMs, to improve
Line 187:     isolation among them.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7d0422bfce99d1e479cb5af49c7b37204ca7e295
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list