Change in vdsm[master]: gluster: verbs to enable/disable snapshot scheduling.

barumuga at redhat.com barumuga at redhat.com
Thu Jun 11 14:34:15 UTC 2015


Bala.FA has posted comments on this change.

Change subject: gluster: verbs to enable/disable snapshot scheduling.
......................................................................


Patch Set 8: Code-Review-1

(4 comments)

https://gerrit.ovirt.org/#/c/40141/8//COMMIT_MSG
Commit Message:

Line 7: enable
you should update vdsm.spec.in to reflect minimum glusterfs version.


https://gerrit.ovirt.org/#/c/40141/8/vdsm/gluster/api.py
File vdsm/gluster/api.py:

Line 740: overrideSnapshotSchedule
Change name to snapshotScheduleOverride()


Line 746: resetSnapshotSchedule
Change name to snapshotScheduleReset()


https://gerrit.ovirt.org/#/c/40141/8/vdsm/gluster/apiwrapper.py
File vdsm/gluster/apiwrapper.py:

Line 319: resetSnapshotSchedule
change the name same as api.py for above two function.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iff8f21593921bac18950a9ee42bb3972d278f7aa
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list