juju-core; default configuration value of ""

[David Britton]

Hi James --

I just filed this bug yesterday:
https://bugs.launchpad.net/juju-core/+bug/1192728  -- It's similar but for
the bulk get operation (and a subtle difference in the way it differs from
pyjuju compared to your case).

Even though I could see advantages to both approaches, I really didn't like
the change from pyjuju, since there are a *lot* of charms already, and this
introduces very subtle and hard to find errors.  Personally, I think in
regard to relation-get, config-get, we should be sticking with what pyjuju
did as closely as possible, especially when there are advantages on both
sides.

But, those are just my thoughts, take it or leave it. :)



On Fri, Jun 21, 2013 at 11:56 AM, James Page <james.page at ubuntu.com> wrote:

> Hi All
>
> A couple of days ago I noticed a change in behaviour for default string
> configuration values of "" in juju-core when compared to pyjuju.
>
> Example config:
>
> options:
>   database:
>     default: ""
>     type: string
>     description: empty string
>   test_two:
>     type: string
>     description: no default string
>
> Under pyjuju:
>
> config-get --format=json database
> ""
> config-get --format=json test_two
> null
>
> Under juju-core:
>
> config-get --format=json database
> null
> config-get --format=json test_two
> null
>
> Specifically this breaks the postgresql charm on juju-core; I've raised a
> bug (see [0]) to which the juju-core team have responded; I prefer the
> proposed new behaviour (its in effect what I do in the charmn helpers I
> have written) but I appreciate that this will probably have impact on a
> large number of charms; so I'd like to throw this open to the list for
> further discussion.
>
> Thoughts?
>
> Cheers
>
> James
>
> [0] https://bugs.launchpad.net/**juju-core/+bug/1192706<https://bugs.launchpad.net/juju-core/+bug/1192706>
>
>
> --
> James Page
> Ubuntu Core Developer
> Debian Maintainer
> james.page at ubuntu.com
>
> --
> Juju mailing list
> Juju at lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/**
> mailman/listinfo/juju <https://lists.ubuntu.com/mailman/listinfo/juju>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/juju/attachments/20130621/b842f506/attachment.html>