On Fri, Sep 2, 2011 at 2:24 PM, Michael Tautschnig <m...@debian.org> wrote:

> Hi,
>
> [...]
> > This mail is to inform you that a new lens for FAI disk_config files was
> > added to Augeas. It can currently be found in the git repository of the
> > Augeas project [1]. This lens will turn disk_config files into a tree
> that
> > can be parsed with xpath using Augeas.
> >
> [...]
>
> Thanks a lot for this nice work!
>
> [...]
> >
> > Bug reports, suggestions, patches are all welcome.
> >
>
> I'd have two suggestions, yes:
>
> - You should probably use http://wiki.fai-project.org/wiki/Setup-storageas a
>  more permanent URL.
>


This URL is only put in my email as an example of what the lens can do. It
is not actually used anywhere in the code.



> - Yet I'm afraid the information on the wiki is partly outdated. The most
>  up-to-date information is in the man page of setup-storage. If you find
> the
>  time to check consistency of the grammars that would be awesome.
>


I have noticed that the schema on the wiki is outdated, and sometimes not
functional even. With the lens in the current state, I've been able to
successfully parse 477 disk_config files we use in production. I'd be happy
to get more tricky cases to test if you wish to send some to me.



Raphaël

Reply via email to