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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes