Dan Kenigsberg has submitted this change and it was merged.

Change subject: gluster: bug fix for listing and adding hook
......................................................................


gluster: bug fix for listing and adding hook

Earlier, with regard to gluster hooks there would be definitely a pre and post
directory for each and every gluster command. But presently, if either pre
or post script is not present there will be no pre or post directories also.
Therefore, if there is no such directories available the gluster hook list
will simply omit it.
When a gluster hook is added if pre or post directories are absent, they
will be created accordingly, then the hook will be added.

Change-Id: I470470da153edef3fe7eb3a7d41a775bac208449
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=999795
Signed-off-by: Timothy Asir <tjeya...@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/18345
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
---
M vdsm/gluster/hooks.py
1 file changed, 16 insertions(+), 0 deletions(-)

Approvals:
  Timothy Asir: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I470470da153edef3fe7eb3a7d41a775bac208449
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Timothy Asir <tjeya...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Bala.FA <barum...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeya...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to