Bug#705052: [initscripts] if-up.d/mountnfs: won't mount if lo is last interface to be configured

2013-04-14 Thread Roger Leigh
tags 705052 + pending
thanks

On Tue, Apr 09, 2013 at 04:35:58PM +0200, Timo Weingärtner wrote:
> When doing ifup -a (as done on boot) it can happen that lo is brought up 
> last. 
> Then the combination of
> # Not for loopback!
> [ "$IFACE" != "lo" ] || exit 0
> and
> # Wait until all auto interfaces are up before attemting to mount
> # network file systems.
> exit_unless_last_interface
> ensures that nfs is not mounted.
> 
> Attached patch ignores lo in exit_unless_last_interface.

Patch applied to git.  Many thanks!


Thanks,
Roger

-- 
  .''`.  Roger Leigh
 : :' :  Debian GNU/Linuxhttp://people.debian.org/~rleigh/
 `. `'   schroot and sbuild  http://alioth.debian.org/projects/buildd-tools
   `-GPG Public Key  F33D 281D 470A B443 6756 147C 07B3 C8BC 4083 E800


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#705052: [initscripts] if-up.d/mountnfs: won't mount if lo is last interface to be configured

2013-04-09 Thread Timo Weingärtner
Package: initscripts
Version: 2.88dsf-41
Severity: important
Tags: patch

When doing ifup -a (as done on boot) it can happen that lo is brought up last. 
Then the combination of
# Not for loopback!
[ "$IFACE" != "lo" ] || exit 0
and
# Wait until all auto interfaces are up before attemting to mount
# network file systems.
exit_unless_last_interface
ensures that nfs is not mounted.

Attached patch ignores lo in exit_unless_last_interface.


Greetings
Timo
--- mountnfs.dpkg-dist	2012-10-15 19:30:41.0 +0200
+++ mountnfs	2013-04-09 15:45:56.375942703 +0200
@@ -121,6 +121,9 @@
 exit_unless_last_interface() {
 ifaces="$(ifquery --list)"
 for i in $ifaces ; do
+	if [ "$i" = "lo" ]; then
+	continue
+	fi
 	if ! grep -q $i /etc/network/run/ifstate ; then
 	msg="if-up.d/mountnfs[$IFACE]: waiting for interface $i before doing NFS mounts"
 	log_warning_msg "$msg"


signature.asc
Description: This is a digitally signed message part.