Ubuntu
upstart package

14.10: NFS drives in fstab not mounted automatically

Bug #1391296 reported by Tommy Vestermark
52
This bug affects 9 people
Affects Status Importance Assigned to Milestone
nfs-utils (Ubuntu)
Fix Released
High
Marc Deslauriers
Utopic
Fix Released
High
Marc Deslauriers
Vivid
Fix Released
High
Marc Deslauriers
upstart (Ubuntu)
Invalid
High
Unassigned
Utopic
Invalid
High
Unassigned
Vivid
Invalid
High
Unassigned

Bug Description

[SRU Request]

Due to a change in Upstart behaviour, the statd daemon no longer starts automatically at boot, resulting in nfs mounts not being mounted at boot.

This has been corrected by modifying the statd upstart job to wait for the rpcbind job to start, instead of waiting for the compatibility portmap event.

[Test Case]
1- set up an NFS mount in /etc/fstab
2- Reboot, notice the directory didn't get mounted
3- Install update
4- Reboot, notice the directory is mounted

[Regression Potential]
The upstart jobs to get the proper daemons started up at boot have complex relationships, and have suffered from race conditions in the past. Although this change is small, it may slightly change previous behaviour. Of course, not having it work at all is worse than having a possible race condition, so this fix is unlikely to be any worse than the broken behaviour.

Original description:

After upgrading to 14.10 (fresh install) my NFS drive does no longer mounts automatically at boot when the network is up and running. Manually running mount -a mounts the drive as expected and hacking a mount -a command into mountall-net.conf makes my system function normally again. Trying to manually to killall -USR1 mountall does not work.

ProblemType: Bug
DistroRelease: Ubuntu 14.10
Package: mountall 2.54build1
ProcVersionSignature: Ubuntu 3.16.0-24.32-generic 3.16.4
Uname: Linux 3.16.0-24-generic x86_64
ApportVersion: 2.14.7-0ubuntu8
Architecture: amd64
CurrentDesktop: Unity
Date: Mon Nov 10 20:37:39 2014
EcryptfsInUse: Yes
InstallationDate: Installed on 2014-11-09 (1 days ago)
InstallationMedia: Ubuntu 14.10 "Utopic Unicorn" - Release amd64 (20141022.1)
ProcKernelCmdline: BOOT_IMAGE=/boot/vmlinuz-3.16.0-24-generic root=UUID=e1197618-b55d-40d3-9b81-df2dcb847c1f ro quiet splash vt.handoff=7
SourcePackage: mountall
UpgradeStatus: No upgrade log present (probably fresh install)
mtime.conffile..etc.init.mountall.net.conf: 2014-11-10T20:26:00.795161

See original description
Revision history for this message
Tommy Vestermark (tov) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in mountall (Ubuntu):
status: New → Confirmed
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Ok, I've investigated this, and it turns out:

statd-mounting.conf intercepts nfs mounts and waits for the statd daemon to be run
statd.conf starts on (virtual-filesystems and started portmap ON_BOOT=y)
rpcbind.conf is what emits the started portmap event:

# For compatibility with older upstart jobs in Ubuntu
post-start exec initctl emit --no-wait started JOB=portmap ON_BOOT=$ON_BOOT
pre-stop exec initctl emit --no-wait stopping JOB=portmap

But, to actually work with upstart in 14.10, the INSTANCE variable has to be added, like so:
# For compatibility with older upstart jobs in Ubuntu
post-start exec initctl emit --no-wait started JOB=portmap INSTANCE='' ON_BOOT=$ON_BOOT
pre-stop exec initctl emit --no-wait stopping JOB=portmap INSTANCE=''

affects: mountall (Ubuntu) → upstart (Ubuntu)
Changed in nfs-utils (Ubuntu):
status: New → Confirmed
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Upstart shouldn't require an INSTANCE variable for that to work, and nfs-utils shouldn't be waiting on the portmap job, as that is for legacy compatibility only.

Marc Deslauriers (mdeslaur)
Changed in nfs-utils (Ubuntu Utopic):
status: New → Confirmed
Changed in upstart (Ubuntu Utopic):
status: New → Confirmed
Changed in nfs-utils (Ubuntu Utopic):
assignee: nobody → Marc Deslauriers (mdeslaur)
Changed in nfs-utils (Ubuntu Vivid):
assignee: nobody → Marc Deslauriers (mdeslaur)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nfs-utils - 1:1.2.8-9ubuntu2

---------------
nfs-utils (1:1.2.8-9ubuntu2) vivid; urgency=medium

  * debian/nfs-common.statd.upstart: wait for rpcbind job instead of
    relying on portmap compatibility event. (LP: #1391296)
 -- Marc Deslauriers <email address hidden> Tue, 18 Nov 2014 19:07:15 -0500

Changed in nfs-utils (Ubuntu Vivid):
status: Confirmed → Fix Released
Revision history for this message
Cyrus Harmon (ch-launchpad) wrote :

Any chance of seeing a fix for utopic for this?

Marc Deslauriers (mdeslaur)
description: updated
Changed in nfs-utils (Ubuntu Utopic):
status: Confirmed → In Progress
Alberto Salvia Novella (es20490446e)
Changed in nfs-utils (Ubuntu Utopic):
importance: Undecided → High
Changed in nfs-utils (Ubuntu Vivid):
importance: Undecided → High
Changed in upstart (Ubuntu Utopic):
importance: Undecided → High
Changed in upstart (Ubuntu Vivid):
importance: Undecided → High
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Tommy, or anyone else affected,

Accepted nfs-utils into utopic-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/nfs-utils/1:1.2.8-9ubuntu1.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in nfs-utils (Ubuntu Utopic):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

FWIW, the package in utopic-proposed fixes the issue for me.

I'm not marking as verification-done, as I would really like for someone else who is affected by this bug to test it also.

Revision history for this message
Cyrus Harmon (ch-launchpad) wrote :

The fix in proposed fixed this problem on two separate boxes for me. Would be great to see this in the main repository.

thanks!

Scott Kitterman (kitterman)
tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nfs-utils - 1:1.2.8-9ubuntu1.1

---------------
nfs-utils (1:1.2.8-9ubuntu1.1) utopic; urgency=medium

  * debian/nfs-common.statd.upstart: wait for rpcbind job instead of
    relying on portmap compatibility event. (LP: #1391296)
 -- Marc Deslauriers <email address hidden> Tue, 18 Nov 2014 19:07:15 -0500

Changed in nfs-utils (Ubuntu Utopic):
status: Fix Committed → Fix Released
Revision history for this message
Scott Kitterman (kitterman) wrote : Update Released

The verification of the Stable Release Update for nfs-utils has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

This was fixed in nfs-utils. Closing upstart tasks as they are no longer necessary.

Changed in upstart (Ubuntu):
status: Confirmed → Won't Fix
status: Won't Fix → Invalid
Changed in upstart (Ubuntu Utopic):
status: Confirmed → Invalid
Changed in upstart (Ubuntu Vivid):
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.