Bug 763735 (GLUSTER-2003)

Summary:	If you recursively mount Gluster freezes. It should error and fail to mount.
Product:	[Community] GlusterFS	Reporter:	Craig Carl <craig>
Component:	scripts	Assignee:	Rajesh <rajesh>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	medium	Docs Contact:
Priority:	low
Version:	mainline	CC:	aavati, gluster-bugs, jdarcy, rajesh, vagarwal, vijay
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Craig Carl 2010-10-22 11:41:49 UTC

Gluster 3.0.5-1
>
>  i use glusterfs on debian squeeze.
>
>  i have 2 server with the glusterfs server running (replicate), and a
>  mount with glusterfs client.
>
>  i have glusterfs-server that use /srv/glusterfs for export
> directory.
>
>  i made a mistake to mount the client on the same directory in the
>  server of one of the glusterfs-server.
>
>  glusterfs -f /etc/glusters/myfs /srv/glusterfs.
>
>  my glusterfs freeze.
>
>  bst regards.

Comment 1 Junaid 2011-01-24 07:41:38 UTC

Glusterfs program cannot control whether the mount point is valid or not.

Comment 2 Craig Carl 2011-01-24 19:47:44 UTC

True, but Gluster can send an error instead of just crashing. We don't need recursive mounts to work, we just want a meaningful error message and Gluster not to crash.

Comment 3 Anand Avati 2011-01-24 20:15:01 UTC

(In reply to comment #2)
> True, but Gluster can send an error instead of just crashing. We don't need
> recursive mounts to work, we just want a meaningful error message and Gluster
> not to crash.

Gluster is not crashing in this case. It freezes. It is hard to detect, actually almost impossible to detect, if the same directory which is about to be mounted upon, also happens to be the export directory (or its parent directory, or its sub-directory - it would have frozen in those cases as well) of the same volume. Or in a more complicated setup, volume A's mount being volume B's export and volume B's mount being volume A's export (or more such hops).


Avati

Comment 4 Rajesh 2011-08-08 03:58:09 UTC

*** Bug 2495 has been marked as a duplicate of this bug. ***

Comment 5 Amar Tumballi 2011-09-27 05:50:05 UTC

Planing to keep 3.4.x branch as "internal enhancements" release without any features. So moving these bugs to 3.4.0 target milestone.

Comment 6 Anand Avati 2011-11-11 07:01:31 UTC

CHANGE: http://review.gluster.com/314 (Adding check_recursive_mount() in mount.glusterfs.in) merged in master by Anand Avati (avati)