[libvirt] [PATCH v2] docs, rng: Adjust storage pool name grammar checks

[Peter Krempa]

On Wed, Oct 04, 2017 at 10:55:58 -0400, John Ferlan wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=1475250
> 
> It's possible to define and start a pool with a '.' in the
> name; however, when trying to add a volume to a domain using
> the storage pool source with a '.' in the storage pool name,
> the domain RNG validation fails because RNG uses 'genericName'
> which does not allow a '.' in the name.
> 
> Domain XML def parsing has a virXMLValidateAgainstSchema which
> generates the error. The Storage Pool XML def parsing has no
> call to virXMLValidateAgainstSchema. The only Storage Pool name
> validation occurs in virStoragePoolDefParseXML to ensure the
> name doesn't have a '/' in it and in storagePoolDefineXML to
> call virXMLCheckIllegalChars using the same parameter "\n" as
> qemuDomainDefineXMLFlags would check after the RNG check
> could be succesful.
> 
> In order to resolve this, create a poolName definition in
> storagecommon.rng that will mimic the domain name regex that
> disallows a newline character, but add the "/" in the exclude
> list. Then modify the pool and volume source name definitions
> to key off that poolName.
> 
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
> 
>  v1: https://www.redhat.com/archives/libvir-list/2017-September/msg01037.html
> 
>  Change only to add '/' before the newline in poolName
> 
>  docs/schemas/domaincommon.rng  | 2 +-
>  docs/schemas/storagecommon.rng | 8 ++++++++
>  docs/schemas/storagepool.rng   | 4 ++--
>  3 files changed, 11 insertions(+), 3 deletions(-)

ACK
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20171005/656a5af4/attachment-0001.sig>