Bug 148764

Created attachment 111084 [details]
netfs.patch

Patch from Ian Amess <ian.amess> with comments about each change

*** This bug has been marked as a duplicate of 147610 ***

The patch attached to this bug (bug
148764) contains more fixes than just the one that it was duped
against. In particular, the activating/deactivating LVM volume groups
on _netdev devices, and the fact that fsck tries to run on _netdev
devices.

The current code causes problems on startup for the initialisation of
iSCSI initiators connected to a NetApp, with LVM volume groups on there.

netdev filesystems shouldn't be marked for check in fstab; that would
be a local config error.

LVM and fsck should probably be tracked when the block devices are
created (in either the iscsi or nbd init scripts.)

Created attachment 111379 [details]
Updated patch to handle lvm/fsck for _netdev devices

This patch adds lvm activation and fsck support for _netdev devices.  It is
basically a reworking of the patch from comment #1, and obsoletes it.  The
reason I am proposing to do this in the netfs script instead of the individual
network block device initscripts is that this keeps it all in one place, and
anyone adding a new nbd doesn't need to reimplement all that checking.

The other thing I was thinking is we could add a netdev initscript that handles
network block device setup, and leave the mounting/unmounting to netfs -
thoughts?

As an aside, shouldn't the fsck line just be -t opts=_netdev?

The reason I don't like it in netfs is because it's a separate
fsck/lvm/vgchange etc invocation on every boot when most users don't
need it; hence, it's better used when those devices are actually in use.

(In reply to comment #9)
>
> The other thing I was thinking is we could add a netdev initscript that handles
> network block device setup, and leave the mounting/unmounting to netfs -
> thoughts?

This seems to be the simplest solution, in my opinion.

Created attachment 115377 [details]
Initial stab at netdev initscript

Here is an initial stab at a netdev initscript.  It simply looks at /etc/fstab
to see if there are any _netdev entries that are to be automounted, and if
there are runs vgscan && vgchange -ay (if lvm is present) and fscks the _netdev
filesystems.  It does nothing on the stop condition currently.

Created attachment 115379 [details]
Initial stab at netdev initscript

Oops - had an echo in the wrong spot.  Updated netdev script with fix.

What would you like to do when fsck fails? :)

Good question.  Since this isn't in the netfs script, we can't really prevent it
from being mounted, which is what I'd prefer.  I could do what rc.sysinit does
and drop the user into a shell, then reboot afterwwards, but I dunno that that's
a good idea at that point in the boot process.  What do you think?

Well...

After more consideration, I suppose netfs is ok for this bit, as long as the
fsck, lvm, etc are only conditionalized on netdev devices actually being in use.

As to what to do when fsck fails:

- Normally, rebooting after manual intervention on a failed fsck is done because
it's some sort of critical system filesystem. Is that normally the case for
filesystems of this sort?

- Probably the biggest reason why you'd want to reboot after failure is because
you may not have fixed in in the shell, and therefore you don't want to try and
mount it. For this, if fsck failed, we'd probably want to change the mounting to
be -O no_netdev. However, in the case of multiple netdev filesystems where only
one failed fsck, this would cause *all* of them to not get mounted.

I suppose shell then reboot is the safest option, but just slapping 'shutdown -r
now' in the middle of the script seems ugly. :/

Merged in 7.31.24.EL-1/7.93.14.EL-1.

I tried the latest patch and it still doesn't work for me.  It looks like it's 
supposed to do a vgscan and then activate them all before trying to mount the 
iscsi LVM luns.  But it didn't work for me.   

When will 7.31.24.EL-1/7.93.14.EL-1 be out?
Where can I find it?

Is this supposed to unmount the VGs, and deactivate the VGs before it stops 
iSCSI during a shutdown?  I don't see the code for that.

Created attachment 118744 [details]
patches initscripts-7.31.22.EL-2 netfs, includes shutdown fixes, too

this one works for me.. the previous one had a bug, or wasn't for my
initscripts version or something.  I also added 3 lines to make it set the
volume groups to inactive, so that iscsi module can stop properly.

An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2005-606.html

An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2005-607.html

Created attachment 133796 [details]
netfs.patch

This patch fixes netfs within initscripts-7.31.31.EL-1 to support iscsi devices
in LVM volume groups.

We don't normally do vgchange -a n on shutdown - why would you need this?

because the iscsi service cannot stop unless the devices are free.  with the 
VG's active, then the underlying iscsi devices are not free.