Change in vdsm[master]: migration: added support for convergence schedule

fromani at redhat.com fromani at redhat.com
Mon Feb 15 16:57:01 UTC 2016


Francesco Romani has posted comments on this change.

Change subject: migration: added support for convergence schedule
......................................................................


Patch Set 14: Code-Review-1

(1 comment)

-1 for visibility. We need better schema/documentation, codewise still OK.

https://gerrit.ovirt.org/#/c/46940/14//COMMIT_MSG
Commit Message:

Line 11: The convergence schedule contains of 2 parts:
       : - initialization: will be evaluated before the migration starts. It is a list
       :   of tuples:
       :   - action: one of
       :     - setDowntime: sets the 'migrateSetMaxDowntime' of the migrating VM
       :     - abort: aborts the migration
       :   - params: parameters of the action (e.g. the setDowntime takes one parameter,
       :     the actual downtime).
       : - convergence: will be evaluated during migration as a part of the monitoring
       :   thread on each monitoring cycle. It is a list of tuples:
       :   - stalling limit: if the VM is stalling longer than this limit, execute the
       :     action with its params.
       :   - action with params: the actual action with params which has to be executed
       :     if the VM is stalling longer than the stalling limit. The value is the same
       :     as for the initialization, namely:
       :     - action: one of
       :       - setDowntime: sets the 'migrateSetMaxDowntime' of the migrating VM
       :       - abort: aborts the migration
       :     - params: parameters of the action (e.g. the setDowntime takes one parameter,
       :     the actual downtime).
after another mtg, turns out we should add better documentation and use more schema types whatever possible, to make sure the convergenceSchedule we pass is less opaque and verifiable once we have Piotr's schema converter. Otherwise fine.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I989cff12d08ef1cab36bd10df7daaa999a8dac14
Gerrit-PatchSet: 14
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list