open-iscsi shutdown failure due to missing dir

Bug #541512 reported by brand1234
128
This bug affects 27 people
Affects Status Importance Assigned to Milestone
mountall (Ubuntu)
Fix Released
High
Colin Watson
Nominated for Karmic by Zhang Linshan
Nominated for Lucid by Zhang Linshan
sysvinit (Ubuntu)
Fix Released
High
Colin Watson
Nominated for Karmic by Zhang Linshan
Nominated for Lucid by Zhang Linshan

Bug Description

Binary package hint: open-iscsi

Ubuntu 9.10
open-iscsi 2.0.870.1-0ubuntu12

/etc/init.d/open-iscsi restart

 * Disconnecting iSCSI targets [ OK ]
 * Stopping iSCSI initiator service [ OK ]
 * Starting iSCSI initiator service iscsid [ OK ]
ln: target `/lib/init/rw/sendsigs.omit.d/' is not a directory: No such file or directory
 * Setting up iSCSI targets

This causes iscsid to be killed prematurely during shutdown and eventually causes the shutdown to hang.

Revision history for this message
dhchen (dhchen-tw) wrote :

I have the same issue. It seems that the removal of /etc/init.d/mountkernfs.sh introduced this problem.

Revision history for this message
Andreas Florath (florath) wrote :

Same problem with
Ubuntu 10.04
open-iscsi 2.0.871-0ubuntu4

Revision history for this message
Patrick J. LoPresti (lopresti) wrote :

I can confirm that the patch in Bug 567143 (mkdir -p /lib/init/rw/sendsigs.omit.d in /etc/init.d/open-iscsi) works around this issue.

But open-iscsi is not the only init script that relies on /lib/init/rw/sendsigs.omit.d, so it is not open-iscsi's job to create it. This is clearly a bug in the initscripts package (source package sysvinit), and I have opened a bug against that package:

https://bugs.launchpad.net/ubuntu/+source/sysvinit/+bug/603760

Colin Watson (cjwatson)
affects: open-iscsi (Ubuntu) → sysvinit (Ubuntu)
Revision history for this message
Colin Watson (cjwatson) wrote :

I certainly don't think we should be fixing this in every package that uses sendsigs.omit.d.

The convention in Ubuntu has been to use /var/run rather than /lib/init/rw for these things.
I can easily add /var/run/sendsigs.omit.d/ support to sendsigs, and I've done so in my local tree, but I don't really feel like playing whack-a-mole to move all the users over to that directory, and having /var/run/sendsigs.omit on the one hand and /lib/init/rw/sendsigs.omit.d/ on the other seems wilfully confusing.

I've asked Scott if he'd object to a new mounted-libinitrw.conf job in mountall that just does 'exec mkdir -p /lib/init/rw/sendsigs.omit.d'. I think this would be the easiest fix. (Note that, since /lib/init/rw is a mountpoint, we can't just ship the directory in the initscripts package.)

Colin Watson (cjwatson)
Changed in mountall (Ubuntu):
assignee: nobody → Colin Watson (cjwatson)
Changed in sysvinit (Ubuntu):
assignee: nobody → Colin Watson (cjwatson)
Changed in mountall (Ubuntu):
status: New → Triaged
Changed in sysvinit (Ubuntu):
status: New → Triaged
Changed in mountall (Ubuntu):
importance: Undecided → Medium
Changed in sysvinit (Ubuntu):
importance: Undecided → Medium
Changed in mountall (Ubuntu):
importance: Medium → High
Changed in sysvinit (Ubuntu):
importance: Medium → High
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package sysvinit - 2.87dsf-4ubuntu18

---------------
sysvinit (2.87dsf-4ubuntu18) maverick; urgency=low

  * Allocate pidof/killall5 omitpid buffers dynamically. 16 is too small
    for killall5 now that all Upstart jobs are omitted.
  * Create /lib/init/rw as a symlink to /var/run on new installations, and
    fix it up in /etc/init.d/umountroot on upgrade, as it's difficult to do
    this at any other time; this saves us chasing around all the individual
    packages that use one or the other for sendsigs.omit.d (LP: #541512).
  * Handle /var/run/sendsigs.omit.d explicitly, just in case.
 -- Colin Watson <email address hidden> Fri, 24 Sep 2010 10:48:28 +0100

Changed in sysvinit (Ubuntu):
status: Triaged → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package mountall - 2.18

---------------
mountall (2.18) maverick; urgency=low

  [ Dustin Kirkland ]
  * conf/mounted-varrun.conf: seed /var/run/motd on boot, to ensure that
    initial logins are quick, LP: #587858

  [ Steve Langasek ]
  * Don't check /etc/environment for locale settings; this is an obsolete
    usage we transitioned away from years ago.
  * Add LC_ALL to the list of exported locale variables.

  [ Colin Watson ]
  * Flush standard output and error before daemonising, so that we don't get
    three copies of our early debug output.
  * If a mountpoint is a symlink to another mountpoint, add a dependency on
    the link target rather than trying to mount it again and failing
    (LP: #541512).
 -- Colin Watson <email address hidden> Fri, 24 Sep 2010 10:52:20 +0100

Changed in mountall (Ubuntu):
status: Triaged → Fix Released
Revision history for this message
Alkis Georgopoulos (alkisg) wrote :

Could we please have this in lucid-updates?
It breaks nbd-client too.

Revision history for this message
shigeo (spock-gmx) wrote :

any word on if we can have this in lucid-updates?

Revision history for this message
Maxxer (lorenzo-milesi) wrote :

agree, need this on LTS please!

Revision history for this message
JasonStokes (j32493) wrote :

Agree! we need this on LTS ... please......

Revision history for this message
Andrew Radke (andrew-radke) wrote :

Since this has not been fixed in lucid I personally believe that the status cannot be listed as fix released. maverick is not LTS and is therefore not suitable in many server environments where iSCSI would be used.

Revision history for this message
Andreas Ntaflos (daff) wrote :

This is still a problem on Ubuntu 10.04.3.

Revision history for this message
Paul Belanger (pabelanger) wrote :

Just ran into this issue too, going to try and get this into lucid-updates

Revision history for this message
Adam Barton (mrsnakeoil) wrote :

I can confirm that problem replicates in our own Lucid 10.04.3. I also agree that a bug that impacts an current LTS should be fixed in the updates for the current LTS.

Revision history for this message
Thomas Leavitt (u-tho4as-f) wrote :

FYI: the portmap package in Lucid creates this directory on boot, so that a symlink to the portmap.pid can be placed there. Installing it is a way of ensuring that this directory is created properly without creating a custom hack. I believe that later versions of this included in packages after Lucid may change this behavior.

Revision history for this message
Ryan Tandy (rtandy) wrote :

I discovered this bug on a fresh 10.04.4 Server install today. I'm surprised it hasn't been fixed in Lucid. Ubuntu folks, is it possible that this could still be considered for an SRU? Is there something I could do (e.g. prepare a debdiff) to move the process forward?

Revision history for this message
Maxxer (lorenzo-milesi) wrote :

Freshly installed LTS, apparently portmap creates the needed directory, still the iscsi shutdown hangs...

Revision history for this message
Brian Morton (rokclimb15) wrote :

I can confirm that installing portmap fixes the shutdown issue on 10.04.4. The mkdir workaround in the init script does not work correctly any longer. This should be addressed in SRU as it is a serious loss of stability when working with remote servers.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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