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

danken at redhat.com danken at redhat.com
Thu Mar 10 11:51:02 UTC 2016


Dan Kenigsberg has submitted this change and it was merged.

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


migration: added support for convergence schedule

Part of the migration enhancements effort.
Adds the logic which parses and evaluates the convergence schedule.
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).

The convergence schedule replaces the DowntimeThread (e.g. if the convergence
schedule is set, the DowntimeThread is not started).
If the convergence schedule is not set, the code falls back to use the
DowntimeThread.

Change-Id: I989cff12d08ef1cab36bd10df7daaa999a8dac14
Feature-Page: http://www.ovirt.org/Features/Migration_Enhancements
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/46940
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <fromani at redhat.com>
---
M client/vdsClient.py
M lib/api/vdsmapi-schema.json
M vdsm/API.py
M vdsm/virt/migration.py
4 files changed, 148 insertions(+), 29 deletions(-)

Approvals:
  Tomas Jelinek: Verified
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I989cff12d08ef1cab36bd10df7daaa999a8dac14
Gerrit-PatchSet: 22
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken 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>


More information about the vdsm-patches mailing list