Bug 1084964

Summary:	SMB: CIFS mount fails with the latest glusterfs rpm's
Product:	[Community] GlusterFS	Reporter:	Poornima G <pgurusid>
Component:	gluster-smb	Assignee:	Raghavendra Talur <rtalur>
Status:	CLOSED UPSTREAM	QA Contact:
Severity:	urgent	Docs Contact:
Priority:	unspecified
Version:	pre-release	CC:	ndevos, nlevinki, pgurusid, sbhaloth, sdharane, ssamanta
Target Milestone:	---	Keywords:	Regression, TestBlocker
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1084958
Clones:	1124728 (view as bug list)		Environment:
Last Closed:	2014-04-09 07:11:09 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1084958
Bug Blocks:	1124728

Description Poornima G 2014-04-07 12:16:17 UTC

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

Description of problem:
=========================================
Tried the following workaround to install glusterfs-server:
mkdir -p /var/lib/glusterd/hooks/1/{start,stop,add-brick,set}/{pre,post}
running that before yum install glusterfs-server 
Once the gluster packages are installed, mount of gluster volume via SMB is failing.

 
Version-Release number of selected component (if applicable):
glusterfs-server-3.5qa2-0.304.git0c1d78f.el6rhs.x86_64

How reproducible:
Tried cifs mount several times.

Steps to Reproduce:
1.Install the latest glusterfs rpm's
2.Mount the volume via cifs 


Actual results:
The mount of gluster volume via SMB fails.

Expected results:
The SMB mount should happen successfully.

Additional info:

Comment 1 Anand Avati 2014-04-07 12:17:10 UTC

REVIEW: http://review.gluster.org/7409 (gfapi: In glfs_set_volfile_server() remove the port number check.) posted (#1) for review on master by Poornima G (pgurusid)

Comment 2 Anand Avati 2014-04-08 08:35:39 UTC

COMMIT: http://review.gluster.org/7409 committed in master by Vijay Bellur (vbellur) 
------
commit 6567d141c1b0112acb35e711371ae5a879fc645a
Author: Poornima G <pgurusid>
Date:   Mon Apr 7 17:38:32 2014 +0530

    gfapi: In glfs_set_volfile_server() remove the port number check.
    
    The documentation for glfs_set_volfile_server() api says that the
    port zero is a valid value. Specifying 0 uses the default port
    number GF_DEFAULT_BASE_PORT. Hence removing the check for port being
    zero.
    
    Change-Id: I942a7fe54a418231a438ab67756537df8f28d2de
    BUG: 1084964
    Signed-off-by: Poornima G <pgurusid>
    Reviewed-on: http://review.gluster.org/7409
    Reviewed-by: Raghavendra Talur <rtalur>
    Reviewed-by: Krishnan Parthasarathi <kparthas>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>

Comment 3 Anand Avati 2014-07-25 07:50:53 UTC

REVIEW: http://review.gluster.org/8374 (gfapi: In glfs_set_volfile_server() remove the port number check.) posted (#1) for review on release-3.5 by Poornima G (pgurusid)

Comment 4 Niels de Vos 2014-07-30 07:58:31 UTC

Bug 1124728 will be used for the glusterfs-3.5.x version. This bug was used for the master branch. The release-3.6 branch has been created after the change was merged, so this should have been fixed in any 3.6.x version.

Please do not use closed bugs, or one bug for patches to different glusterfs versions.