Change in vdsm[master]: hsm: remove master info from getStorageDomainInfo

danken at redhat.com danken at redhat.com
Thu Jan 2 12:20:55 UTC 2014


Dan Kenigsberg has posted comments on this change.

Change subject: hsm: remove master info from getStorageDomainInfo
......................................................................


Patch Set 2:

(1 comment)

....................................................
File vdsm/storage/hsm.py
Line 2722
Line 2723
Line 2724
Line 2725
Line 2726
I've read the terse commit message. But given this long and tedious comment, I'd live to make sure what is the exact situation. Would you assert that this comment (and code) was never needed? Can we get Saggi's take on that?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8c046db3884b56b57cc1bf040f3e9cc136c6d877
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list