> Package: fai-client
> Version: 3.2.16
> Tags: patch
>
> Hi,
>
> Running the setup-storage command on a config file that defines an LVM
> disk_config, where the preserve_{always,reinstall} or resize options are
> in use, gives the following result:
>
> # setup-storage -d -f dom0
> setup-harddisks is running in test-only mode
> Starting setup-storage 1.0.3
> Using config file: dom0
> INTERNAL ERROR in setup-storage:
> VG re-parse failed
> Please report this error to the Debian Bug Tracking System.
>
> The root cause is the regexp used to extract the VG / LV names which
> forces to have at least one space character in the lvm id:
> m{^([^/,\s\-]+)-([^/,\s\-]+)\s+}
> which is not possible regarding the EBNF grammar.
> Oops, that's really quite a stupid bug... Especially the inconsistency between the LVM and other kinds of configs. Sorry! > The RAID and disk variants of those options do not have this problem, > only the LVM options. > > Following is a quick and dirty patch to get rid of those extra '\s+' > (there are 3 of them.) > [...] The patch may be quick, but not dirty :-) @Holger: That one might be yet another patch that should make it into lenny, if possible. Could you take care of that? Thanks, Michael
pgpe6qGl8upFO.pgp
Description: PGP signature
