Change in vdsm[master]: vm: Unify disk replication terms

nsoffer at redhat.com nsoffer at redhat.com
Fri May 8 11:53:50 UTC 2015


Nir Soffer has posted comments on this change.

Change subject: vm: Unify disk replication terms
......................................................................


Patch Set 7:

Federico, this cannot be squashed with 40023 - it contains changes added in 40024.

I can move it down above 40024, so related changes are kept together.

Squashing this to 2 different patches means risking verified patch and wasting significant time - are you sure this is needed?

I can also abandon this change as it is not required for this feature.

This a cleanup that will help us work with this code in the future, there is no urgency to merge it now. This was the reason I posted this change only after the feature was completed (https://gerrit.ovirt.org/40185).

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0fa2f30f39eee73b107f5c055f5be790ed33da6a
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: No


More information about the vdsm-patches mailing list