[Bugs] [Bug 1388348] New: glusterd: Display proper error message and fail the command if S32gluster_enable_shared_storage.sh hook script is not present during gluster volume set all cluster.enable-shared-storage <enable/disable> command

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1388348

            Bug ID: 1388348
           Summary: glusterd: Display proper error message and fail the
                    command if S32gluster_enable_shared_storage.sh hook
                    script is not present during gluster volume set all
                    cluster.enable-shared-storage <enable/disable> command
           Product: GlusterFS
           Version: mainline
         Component: glusterd
          Keywords: Triaged
          Severity: medium
          Assignee: bugs at gluster.org
          Reporter: asengupt at redhat.com
                CC: amukherj at redhat.com, bugs at gluster.org,
                    jflory at datapipe.com
        Depends On: 1377448



+++ This bug was initially created as a clone of Bug #1377448 +++

Description of problem:


Version-Release number of selected component (if applicable): GlusterFS 3.8.4
(latest)

How reproducible: Always


Steps to Reproduce:
1. Run command: "gluster volume set all cluster.enable-shared-storage enable"

Actual results:
1. The running Volume now lists "cluster.enable-shared-storage: enable"
a. The command returned success.
2. However the shared volume was not created.

Expected results:
1. When the command fails, a failure return code is expected.
a. With a relevant description.
2. It is expected that the command will create the shared volume.

Additional info:

OS: Debian Jessie

1. Rebooting did not help.
2. Rebuilding the clustered data volume did not help.
3. Clearing files in /var/run/gluster did not help. 
4. Logs lack error messages except:
   readv on /var/run/gluster/d373e2b2e599fb58e944558312eb1ae5.socket failed
(Invalid argument)

--- Additional comment from Avra Sengupta on 2016-10-21 05:57:32 EDT ---

Was the hook script
/var/lib/glusterd/hooks/1/set/post/S32gluster_enable_shared_storage.sh present
at the time this was seen. Hook scripts get deleted when we clean up
/var/lib/glusterd, and need to be installed again.

--- Additional comment from Jay Flory on 2016-10-23 11:38:06 EDT ---

1. At this point I am not able to tell if it was installed.
a. I have long since moved on and installed by other means.
2. Where is this documented about the scripts?
3. If the scripts are missing but needed, the command should fail and
specifically state that the scripts are missing.
a. The command should not fail but report success.

--- Additional comment from Avra Sengupta on 2016-10-24 04:55:18 EDT ---

I agree, maybe we can edit the summary of this bug to reflect the same, and
make the necessary changes so that the command fails with the appropriate
message.

--- Additional comment from Avra Sengupta on 2016-10-25 02:46:39 EDT ---

Based on comment #3, updating the summary of the bug.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1377448
[Bug 1377448] glusterd: Display proper error message and fail the command
if S32gluster_enable_shared_storage.sh hook script is not present during
gluster volume set all cluster.enable-shared-storage <enable/disable>
command
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.