Bug 1339721

Summary: CGroup: /system.slice/mnt.mount not accepting mount options given /etc/fstab for glusterfs.fuse
Product: Red Hat Enterprise Linux 7 Reporter: Riyas Abdulrasak <rnalakka>
Component: systemdAssignee: Jan Synacek <jsynacek>
Status: CLOSED ERRATA QA Contact: Frantisek Sumsal <fsumsal>
Severity: medium Docs Contact:
Priority: medium    
Version: 7.2CC: bblaskov, fsumsal, jsynacek, moagrawa, olim, ovasik
Target Milestone: rcKeywords: EasyFix, Patch
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: systemd-219-22.el7 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-11-04 00:54:17 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:

Description Riyas Abdulrasak 2016-05-25 17:09:35 UTC
Description of problem:

Systemctl mount not setting the mount options specified in /etc/fstab for a gluster.fuse mount.

~~~~~~~~~~~~~~~
systemctl status /mnt
● mnt.mount - /mnt
   Loaded: loaded (/etc/fstab)
   Active: active (mounted) (Result: exit-code) since Wed 2016-05-25 21:20:59 IST; 1min 3s ago
    Where: /mnt
     What: dhcp3-109:/testvol1
     Docs: man:fstab(5)
           man:systemd-fstab-generator(8)
  Process: 3190 ExecRemount=/bin/mount dhcp3-109:/testvol1 /mnt -o remount,defaults,_netdev,acl,ro -n -t glusterfs (code=exited, status=32)
  Process: 3138 ExecMount=/bin/mount dhcp3-109:/testvol1 /mnt -n -t glusterfs -o defaults,_netdev,acl,ro (code=exited, status=0/SUCCESS)
   CGroup: /system.slice/mnt.mount
           └─3171 /usr/sbin/glusterfs --volfile-server=dhcp3-109 --volfile-id=/testvol1 /mnt                  <<<<<<< The mount options some how getting omitted here.

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

Red Hat Enterprise Linux 7.2 (3.10.0-327.el7.x86_64)
systemd-219-19.el7_2.9.x86_64

How reproducible:

Always

Steps to Reproduce:

1. Install RHEL 7.2 

2. Mention a glusterfs share in the /etc/fstab with ro or acl option like below.

dhcp3-109:/testvol1		/mnt			glusterfs	defaults,_netdev,acl,ro	0 0

3. Try to mount the share using 

#systemctl start <mountpoint>

Actual results:

The mount options specified in /etc/fstab  are ignored by "CGroup: /system.slice/mnt.mount"

Expected results:

The mount options should be set as mentioned in fstab

Additional info:

* The mount options were working properly in RHEL 7.1 client , with "systemd-208-20.el7_1.5.x86_64"

* Mount options are properly set when using mount.glusterfs , instead of systemctl.

Comment 2 Mohit Agrawal 2016-05-27 05:14:03 UTC
Hi,

  In RHEL-7.2 mount.glusterfs is not taking argument(passed with -o) because systemd is passing -n(before -o) argument in mount option and script(mount.glusterfs) ignoring the -n option.

The option is obsolete in upstream from the patch(https://github.com/systemd/systemd/pull/755/commits/6f20f850f79df365c2533195214127142013d317) , I think same needs to be merge in downstream also.

Regards
Mohit Agrawal

Comment 3 Oonkwee Lim 2016-06-03 18:10:15 UTC
Question from case:

(In reply to Mohit Agrawal (Bug 1339721))
> Hi,
> 
>   In RHEL-7.2 mount.glusterfs is not taking argument(passed with -o) because systemd is passing -n(before -o) argument in mount option and script(mount.glusterfs) ignoring the -n option.
> 
> The option is obsolete in upstream from the patch(https://github.com/systemd/systemd/pull/755/commits/6f20f850f79df365c2533195214127142013d317) , I think same needs to be merge in downstream also.
> 
> Regards
> Mohit Agrawal

Hi Mohit,

what is the next step for this customer ? do we provide an hotfix or a process to have the argument being passed in the correct order ?

Patrick Philippe
SA EMEA

Comment 4 Mohit Agrawal 2016-06-04 03:46:35 UTC
Hi,

   I think this is not critical for customer aspect.We can share workaround to configure options with glusterfs by write some unit file to start mount as a separate process.
   For specific to hotfix Riyas can confirm more and for about the backport of patch systemd engineering can confirm.

Regards
Mohit Agrawal

Comment 6 Branislav Blaškovič 2016-06-06 11:21:11 UTC
QA acking.

Comment 7 Jan Synacek 2016-06-06 11:44:04 UTC
https://github.com/lnykryn/systemd-rhel/pull/25

Comment 9 Lukáš Nykrýn 2016-06-06 13:43:59 UTC
pushed to staging -> https://github.com/lnykryn/systemd-rhel/commit/9592604df60795ad8b58aa11311a26f267385bae -> post

Comment 11 Frantisek Sumsal 2016-09-21 14:49:40 UTC
Verified with systemd-219-30.el7.

Old package:
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
:: [   LOG    ] :: systemd mounts use deprecated -n option
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::

:: [   INFO   ] :: using '/var/tmp/beakerlib-29664001/backup' as backup destination
:: [   PASS   ] :: Command 'dd if=/dev/zero of=/disk076.img bs=1M count=32' (Expected 0, got 0)
:: [   PASS   ] :: Command 'mkfs -t xfs /disk076.img' (Expected 0, got 0)
:: [   PASS   ] :: Command 'echo '/disk076.img /mnt/systemd-testjAz xfs defaults 0 0' >> /etc/fstab' (Expected 0, got 0)
:: [   PASS   ] :: Command 'systemctl daemon-reload' (Expected 0, got 0)
:: [   PASS   ] :: Command 'systemctl start /mnt/systemd-testjAz' (Expected 0, got 0)
:: [   PASS   ] :: Command 'systemctl status /mnt/systemd-testjAz' (Expected 0, got 0)
:: [   FAIL   ] :: Command 'systemctl status /mnt/systemd-testjAz | grep Process | grep ' \-n '' (Expected 1, got 0)
:: [   PASS   ] :: Command 'systemctl stop /mnt/systemd-testjAz' (Expected 0, got 0)
:: [   PASS   ] :: Command 'rm -vfr /disk076.img /mnt/systemd-testjAz' (Expected 0, got 0)
:: [   LOG    ] :: Duration: 1s
:: [   LOG    ] :: Assertions: 8 good, 1 bad
:: [   FAIL   ] :: RESULT: systemd mounts use deprecated -n option

New package:
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
:: [   LOG    ] :: systemd mounts use deprecated -n option
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::

:: [   INFO   ] :: using '/var/tmp/beakerlib-27007002/backup' as backup destination
:: [   PASS   ] :: Command 'dd if=/dev/zero of=/diskS0H.img bs=1M count=32' (Expected 0, got 0)
:: [   PASS   ] :: Command 'mkfs -t xfs /diskS0H.img' (Expected 0, got 0)
:: [   PASS   ] :: Command 'echo '/diskS0H.img /mnt/systemd-testthM xfs defaults 0 0' >> /etc/fstab' (Expected 0, got 0)
:: [   PASS   ] :: Command 'systemctl daemon-reload' (Expected 0, got 0)
:: [   PASS   ] :: Command 'systemctl start /mnt/systemd-testthM' (Expected 0, got 0)
:: [   PASS   ] :: Command 'systemctl status /mnt/systemd-testthM' (Expected 0, got 0)
:: [   PASS   ] :: Command 'systemctl status /mnt/systemd-testthM | grep Process | grep ' \-n '' (Expected 1, got 1)
:: [   PASS   ] :: Command 'systemctl stop /mnt/systemd-testthM' (Expected 0, got 0)
:: [   PASS   ] :: Command 'rm -vfr /diskS0H.img /mnt/systemd-testthM' (Expected 0, got 0)
:: [   LOG    ] :: Duration: 1s
:: [   LOG    ] :: Assertions: 9 good, 0 bad
:: [   PASS   ] :: RESULT: systemd mounts use deprecated -n option

Comment 13 errata-xmlrpc 2016-11-04 00:54:17 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.

https://rhn.redhat.com/errata/RHBA-2016-2216.html