Shubhendu Tripathi has posted comments on this change.

Change subject: gluster: Added VDSM verb to stop gluster related processes
......................................................................


Patch Set 3:

(2 comments)

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

Line 3: AuthorDate: 2015-07-20 10:33:01 +0530
Line 4: Commit:     Ramesh Nachimuthu <rnach...@redhat.com>
Line 5: CommitDate: 2015-10-23 11:50:03 +0530
Line 6: 
Line 7: gluster: Added VDSM verb to stop gluster related processes
> Is there a similar verb to start gluster processes?
I dont think so.
Line 8: 
Line 9: Added a verb which stops the gluster related process like
Line 10: brick processes, gsyncd process.
Line 11: This needs to be done as part of a host moving to maintenance


https://gerrit.ovirt.org/#/c/43821/3/vdsm/gluster/api.py
File vdsm/gluster/api.py:

Line 53: )
Line 54: 
Line 55: _stopAllProcessesPath = utils.CommandPath(
Line 56:     "stop-all-gluster-processes.sh",
Line 57:     "/usr/share/glusterfs/scripts/stop-all-gluster-processes.sh",
> Will this be the standard location in all deployments?
I think so. But may be we should verify with gluster team as well.
Line 58: )
Line 59: 
Line 60: 
Line 61: GLUSTER_RPM_PACKAGES = (


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id686e098b323eededcf1f89de331a1d524274995
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi <shtri...@redhat.com>
Gerrit-Reviewer: Bala.FA <barum...@redhat.com>
Gerrit-Reviewer: Darshan Narayana Murthy <dnara...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ramesh N <rnach...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtri...@redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeya...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to