gluster-containers icon indicating copy to clipboard operation
gluster-containers copied to clipboard

Use env. variable GLUSTER_BLOCK_ENABLED to enable/disable gluster-block-service

Open nixpanic opened this issue 6 years ago • 3 comments

Enable gluster-block services if GLUSTER_BLOCK_ENABLED is "1" and disable otherwise.

By default, GLUSTER_BLOCK_ENABLED is set as "1".

This change was recently reverted with commit f5409d94. With a small correction to the change, this should now function as expected. The status-probe.sh script checks the value for GLUSTER_BLOCK_ENABLED and compares it to 1. However the update-params.sh script set the variable by default to TRUE. gluster-kubernetes and openshift-ansible both set the variable to "1" by default, so should update-params.sh.

Updates: #120

nixpanic avatar Jan 15 '19 10:01 nixpanic

@nixpanic But status-probe refers the variable GLUSTER_BLOCKD_STATUS_PROBE_ENABLE( which is for whether to check gluster blockd status or not)
Here, it is GLUSTER_BLOCK_ENABLED which is for whether enable/disable GLUSTER_BLOCK ?

Note: It is good idea to merge them both, but currently they are different variables(with different meaning).

SaravanaStorageNetwork avatar Jan 15 '19 11:01 SaravanaStorageNetwork

On Tue, Jan 15, 2019 at 03:51:06AM -0800, Saravanakumar Arumugam wrote:

@nixpanic But status-probe refers the variable GLUSTER_BLOCKD_STATUS_PROBE_ENABLE( which is for whether to check gluster blockd status or not)
Here, it is GLUSTER_BLOCK_ENABLED which is for whether enable/disable GLUSTER_BLOCK ?

Note: It is good idea to merge them both, but currently they are different variables(with different meaning).

Ah, right! I'll rename them and only keep the one we have in gluster-kubernetes and openshift-ansible.

nixpanic avatar Jan 15 '19 12:01 nixpanic

I have modified the probe-status.sh script to also check the GLUSTER_BLOCK_ENABLED variable now. By default gluster-block is enabled, and so is the probe. GLUSTER_BLOCKD_STATUS_PROBE_ENABLE can still be used to disable the probe, there could be users that have configured it that way.

nixpanic avatar Jan 16 '19 07:01 nixpanic