[Bugs] [Bug 1169999] New: Debian 7 wheezy, /sbin/mount.glusterfs fails to automount and print help

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1169999

            Bug ID: 1169999
           Summary: Debian 7 wheezy, /sbin/mount.glusterfs fails to
                    automount and print help
           Product: GlusterFS
           Version: 3.6.1
         Component: scripts
          Assignee: bugs at gluster.org
          Reporter: thomas.downes at ligo.org
                CC: bugs at gluster.org, gluster-bugs at redhat.com



Description of problem: On Debian 7 wheezy, /sbin/mount.glusterfs does not
accept -n option and fails for less than two options (e.g. -h). Makes glusterfs
native client unusable in autofs.

Version-Release number of selected component (if applicable): 3.6.1-1

How reproducible: 100%

Steps to Reproduce:
1. Install 3.6.1-1 on Debian wheezy with autofs and glusterfs client
2. Enable verbose logging (/etc/default/autofs and restart service)
3. Create a very basic autofs entry.

dest -fstype=glusterfs houseofbricks:/gvol

4. 

Actual results:

When cd'ing to automount directory:

-bash: cd: dest: Too many levels of symbolic links

In /var/log/syslog

Dec  2 17:23:59 testautofs automount[9114]: spawn_mount: mtab link detected,
passing -n to mount
Dec  2 17:23:59 testautofs automount[9114]: >> WARNING: getfattr not found,
certain checks will be skipped..
Dec  2 17:23:59 testautofs automount[9114]: >> Illegal option -n
...
Dec  2 17:27:35 testautofs automount[11193]: mounted /spot/dest

(the last line is misleading)

Expected results:
Dec  2 17:54:51 testautofs automount[12189]: attempting to mount entry
/spot/dest
Dec  2 17:54:51 testautofs automount[12189]: >> WARNING: getfattr not found,
certain checks will be skipped..
Dec  2 17:54:51 testautofs automount[12189]: >> rw
Dec  2 17:54:52 testautofs automount[12189]: mounted /spot/dest

Additional info:

Automounting works after the hackiest of hacks. mount.glusterfs still fails on
basic arguments like -h, however, due to lack of error checking on argument
list.

*** mount.glusterfs    Wed Nov 12 13:02:44 2014
--- mount.glusterfs.hack    Tue Dec  2 18:04:21 2014
*************** main ()
*** 555,563 ****
          ## `mount` specifies options as a last argument
          shift 2;
      fi
!     while getopts "Vo:h" opt; do
          case "${opt}" in
              o)
                  parse_options ${OPTARG};
          shift 2;
                  ;;
--- 555,566 ----
          ## `mount` specifies options as a last argument
          shift 2;
      fi
!     while getopts "Vo:hn" opt; do
          case "${opt}" in
+             n)
+                 ;; 
              o)
+             echo "${OPTARG}" 2>/dev/stderr
                  parse_options ${OPTARG};
          shift 2;
                  ;;

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.