From: Osier Yang <jyang@xxxxxxxxxx>
The attributes/elements for auth type "chap" and "ceph" are completely
different, but the schema didn't reflect that. This patch separates each
of the definitions into groups.
Changed "chap" type "login" and "passwd" attributes to be be interleaved.
Also adjusted the documentation of types to match code. The formatdomain.html
page indicated the 'type' field could be 'ceph' or 'iscsi', when in fact it
should have been 'chap'.
---
docs/schemas/storagepool.rng | 42 ++++++++++++++++++++++--------------------
1 file changed, 22 insertions(+), 20 deletions(-)
diff --git a/docs/schemas/storagepool.rng b/docs/schemas/storagepool.rng
index 3c2158a..2595e37 100644
--- a/docs/schemas/storagepool.rng
+++ b/docs/schemas/storagepool.rng
@@ -280,28 +280,30 @@
<define name='sourceinfoauth'>
<element name='auth'>
- <attribute name='type'>
- <choice>
- <value>chap</value>
- <value>ceph</value>
- </choice>
- </attribute>
<choice>
- <attribute name='login'>
- <text/>
- </attribute>
- <attribute name='username'>
- <text/>
- </attribute>
+ <group>
+ <attribute name='type'>
+ <value>chap</value>
+ </attribute>
+ <interleave>
+ <attribute name='login'>
+ <text/>
+ </attribute>
+ <attribute name='passwd'>
+ <text/>
+ </attribute>
+ </interleave>
+ </group>
+ <group>
+ <attribute name='type'>
+ <value>ceph</value>
+ </attribute>
+ <attribute name='username'>
+ <text/>
+ </attribute>
+ <ref name='sourceinfoauthsecret'/>
+ </group>
</choice>
- <optional>
- <attribute name='passwd'>
- <text/>
- </attribute>
- </optional>
- <optional>
- <ref name='sourceinfoauthsecret'/>
- </optional>
</element>
</define>
--
1.8.1.4
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list