[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH v2 2/7] storage: Support "username" for "chap" type "auth"

On 11/07/13 00:04, Daniel P. Berrange wrote:
On Wed, Jul 10, 2013 at 11:51:42AM -0400, John Ferlan wrote:
On 07/10/2013 10:49 AM, Daniel P. Berrange wrote:
On Tue, Jul 09, 2013 at 03:10:46PM -0400, John Ferlan wrote:
To be consistent with "ceph" types for storage "auth" elements, allow
"username" to be used as an "auth" attribute name for "chap" types.
Continue to allow "login" for backwards compatibility when reading the XML,
but when writing the XML use "username".
Hmm, so the schema for 'chap' auth is utterly awful.

While we have parsed this schema for a while, nothing in the libvirt
codebase has ever used 'chap' auth.

As such I think we have reasonable grounds for just discarding the
existing code for parsing 'chap' auth and doing it right. ie use
the same terminology as 'ceph' and do not include the 'password'
value in the XML at all.

Thoughts ?


Don't have the historical perspective w/r/t when it was added - even
looking through gitk I can find that parsing in for quite a while (even
in the src/storage_conf.c).  I admit I found it quite confusing to read
and attempt to document.

Other than having to actually make the changes :-), I see no reason why
the storage_conf needs to be different and support numerous combinations
based on type...

To be sure we're on the same page, the storage_conf XML then becomes

<auth username='someuser'>
   <secret type='[ceph|iscsi]' [usage='mypassed'|uuid='someuuid']/>
Actually I think the schema is:

  <auth username='someuser' type='[ceph|chap]'>

as a habit, it should be

<auth type='[ceph|chap]' username='someuser'>

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]