Change in vdsm[master]: gluster: Publish gluster get volume info

nsoffer at redhat.com nsoffer at redhat.com
Wed Aug 19 15:51:29 UTC 2015


Nir Soffer has posted comments on this change.

Change subject: gluster: Publish gluster get volume info
......................................................................


Patch Set 3:

(1 comment)

Looks good but needs better commit message.

https://gerrit.ovirt.org/#/c/45098/3//COMMIT_MSG
Commit Message:

Line 5: CommitDate: 2015-08-19 18:07:32 +0300
Line 6: 
Line 7: gluster: Publish gluster get volume info
Line 8: 
Line 9: Publish glusterGetVolumeInfo when RHEV enabled.
Please explain what was the previous situation, and the new situation from the user point of view.

Also explain why we always need glusterGetVolumeInfo.
Line 10: 
Line 11: Change-Id: Ib9c70e9afa2fd776dfd9c2767c2e83e7f179b5c2
Line 12: Bug-Url: https://bugzilla.redhat.com/1254499


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib9c70e9afa2fd776dfd9c2767c2e83e7f179b5c2
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list