Re: [libvirt] [PATCH v6 1/5] domain: Add optional 'disableTLS' attribute for TCP chardev

2016-09-12 Thread John Ferlan
On 09/12/2016 04:26 AM, Daniel P. Berrange wrote: > On Fri, Sep 09, 2016 at 04:49:04PM -0400, John Ferlan wrote: >> Add an optional "disableTLS='yes'" option for a TCP chardev for the >> express purpose to not enable setting up TLS for the chardev > > Can we just call this "tls=yes|no" -

Re: [libvirt] [PATCH v6 1/5] domain: Add optional 'disableTLS' attribute for TCP chardev

2016-09-12 Thread Daniel P. Berrange
On Fri, Sep 09, 2016 at 04:49:04PM -0400, John Ferlan wrote: > Add an optional "disableTLS='yes'" option for a TCP chardev for the > express purpose to not enable setting up TLS for the chardev Can we just call this "tls=yes|no" - negative attributes (disableFOO) are kind of ugly IMHO, as they

[libvirt] [PATCH v6 1/5] domain: Add optional 'disableTLS' attribute for TCP chardev

2016-09-09 Thread John Ferlan
Add an optional "disableTLS='yes'" option for a TCP chardev for the express purpose to not enable setting up TLS for the chardev Signed-off-by: John Ferlan --- docs/formatdomain.html.in | 20 + docs/schemas/domaincommon.rng