Bug 762088 (GLUSTER-356)

Summary:	ls on samba mount gives wrong output.
Product:	[Community] GlusterFS	Reporter:	Raghavendra G <raghavendra>
Component:	booster	Assignee:	Shehjar Tikoo <shehjart>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	medium	Docs Contact:
Priority:	low
Version:	2.0.7	CC:	gluster-bugs
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:	RTP	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Raghavendra G 2009-11-03 19:03:29 UTC

raghu@booradley:/mnt/raghu$ ls linux-2.6.29.1
X11R6.bak/      directory/  io-zonefile/       iozone-newfile-1-2/      man/
apache_pb.gif/  file/       iozone-file/       libexec/                 manual/
clients/        file-new/   iozone-newfile-1/  linux-2.6.29.1.tar.bz2/  memleak-2.vol/

Note that linux-2.6.29.1.tar.bz2/ being displayed as a directory.

raghu@booradley:/mnt/raghu$ mount
/dev/sda4 on / type ext3 (rw)
proc on /proc type proc (rw)
sysfs on /sys type sysfs (rw)
usbfs on /proc/bus/usb type usbfs (rw)
tmpfs on /dev/shm type tmpfs (rw)
nfsd on /proc/fs/nfs type nfsd (rw)
//localhost/gluster/ on /mnt/raghu type cifs (rw,mand)

raghu@booradley:/mnt/raghu$ ls
X11R6.bak/     directory/  index.html   iozone-newfile-1    linux-2.6.29.1/         manual/
apache_pb.gif  file        io-zonefile  iozone-newfile-1-2  linux-2.6.29.1.tar.bz2  memleak-2.vol
clients/       file-new    iozone-file  libexec/            man/

I was running these tests on top of shehjar's c618b7bc59ad46e8b9 in release-2.0 branch. Also relative path support is not working If I happen to give a relative path to samba.conf.

Comment 1 Shehjar Tikoo 2009-11-04 02:51:19 UTC

(In reply to comment #0)
> raghu@booradley:/mnt/raghu$ ls linux-2.6.29.1
> X11R6.bak/      directory/  io-zonefile/       iozone-newfile-1-2/      man/
> apache_pb.gif/  file/       iozone-file/       libexec/                 manual/
> clients/        file-new/   iozone-newfile-1/  linux-2.6.29.1.tar.bz2/ 
> memleak-2.vol/
> 
> Note that linux-2.6.29.1.tar.bz2/ being displayed as a directory.
> 


I fixed a bug yesterday which could've resulted in corrupt attrs to be returned in the stat structure. Can you try again once the patches for bug 762086 are committed. That bug was in both mainline and release2.

OR

Apply my two latest patches for bug 762086 from patchwork.

<snipped>
> branch. Also relative path support is not working If I happen to give a
> relative path to samba.conf.

You cant give relative paths in samba.conf. The current code assumes that at least
the VMP in booster fstab file and the smb.conf are absolute paths.

Besides, samba is a daemon service, you should never specify a relative path to
such a program.

Comment 2 Raghavendra G 2009-11-04 04:45:14 UTC

even after applying the patches submitted for bug 762086, ls shows wrong directory listing.

Comment 3 Shehjar Tikoo 2009-11-12 03:24:15 UTC

I have

Comment 4 Shehjar Tikoo 2009-11-12 03:25:50 UTC

(In reply to comment #3)
> I have
a feeling that some of the bugs fixed by Du recently were all related to this bug also. In any case, his tests on samba-booster have been working fine so I am closing this particular bug.