Change in vdsm[master]: RFE: Report downtime for each live migration

fromani at redhat.com fromani at redhat.com
Thu Mar 19 08:11:03 UTC 2015


Francesco Romani has posted comments on this change.

Change subject: RFE: Report downtime for each live migration
......................................................................


Patch Set 8:

Yes, sorry about my slowness with migrateStatus (still low on caffeine).
It could indeed lead to nicer code, but needs quite some work. For example I find _really_ ugly this part

#
# Notes: Migration status is returned as the command status ('code' and
#        'message')

(from schema) and for the usual backward compatibility reasons this has really to be preserved (but also wiped away the moment we start work on 4.0).

But yes, could definitely make sense.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I34260160404a2b3cc7d26f76a16cfa2385fa9017
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list