Bug 1541568 - [GSS] logic in S30samba-start.sh hook script needs tweaking
Summary: [GSS] logic in S30samba-start.sh hook script needs tweaking
Alias: None
Product: Red Hat Gluster Storage
Classification: Red Hat
Component: samba
Version: rhgs-3.3
Hardware: All
OS: Linux
Target Milestone: ---
: RHGS 3.4.z Batch Update 3
Assignee: Anoop C S
QA Contact: Vivek Das
Depends On: 1575836
TreeView+ depends on / blocked
Reported: 2018-02-02 22:21 UTC by Pan Ousley
Modified: 2019-02-07 04:50 UTC (History)
12 users (show)

Fixed In Version: glusterfs-3.12.2-33
Doc Type: Enhancement
Doc Text:
With this release, the Samba hook script has been improved for adding volume share section in the smb.conf file. To export gluster volume as SMB share via Samba, one of the following volume options, user.cifs or user.smb is required. The existing glusterfs volume share sections in smb.conf will remain unchanged.
Clone Of:
: 1575836 (view as bug list)
Last Closed: 2019-02-04 07:41:25 UTC
Target Upstream Version:

Attachments (Terms of Use)

System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1668278 0 unspecified CLOSED [Samba] Improve logic in Samba hook scripts for adding volume share section in smb.conf 2021-02-22 00:41:40 UTC
Red Hat Product Errata RHBA-2019:0263 0 None None None 2019-02-04 07:41:37 UTC

Description Pan Ousley 2018-02-02 22:21:03 UTC
Description of problem: 

Syntax issues (shellcheck)
- /var/lib/glusterd/hooks/1/start/post/S30samba-start.sh: line 114: [: =: unary operator expected

Hiding unwanted error messages
- /var/lib/glusterd/hooks/1/start/post/S30samba-start.sh: line 71: smbd: command not found
- Samba is not installed + exit code '1' should be enough

Bad script logic
- volume info file needs to be checked for presence of all configuration possibilities 'user.smb|user.cifs' = 'enable|on'

There is an upstream patch created for these issues here: https://review.gluster.org/#/c/19204/

Version-Release number of selected component (if applicable): RHGS 3.3

Additional info: Please let me know if anything else is needed.

Comment 12 Guenther Deschner 2018-11-13 11:19:34 UTC
Sorry, closed the wrong bug.

Comment 14 Anoop C S 2018-12-05 06:02:36 UTC
Following required patches landed in upstream:

* https://review.gluster.org/c/glusterfs/+/19204
* https://review.gluster.org/c/glusterfs/+/21124

Comment 19 Vivek Das 2019-01-04 08:56:12 UTC
Newly created GlusterFS volumes are not exported as SMB share via Samba unless either of 'user.cifs' or 'user.smb' volume set options are enabled on the volume.
The above is validated and is working as expected in glusterfs-3.12.2-34.

Comment 21 errata-xmlrpc 2019-02-04 07:41:25 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.