pyjuju

config.yaml should have boolean types

Bug #885551 reported by Marco Ceppi
18
This bug affects 4 people
Affects Status Importance Assigned to Milestone
pyjuju
Fix Released
Wishlist
Kapil Thangavelu
pyjuju florence

Bug Description

Currently only string is an acceptable type for data in the config.yaml file. Other types should be added, primarily: enum, integer, and boolean to help maintain some sanity in configuration options.

This bug covers boolean only. For enum, see bug #918386.

See original description

Related branches

lp:~hazmat/pyjuju/bool-and-validate-defaults
Benjamin Saller (community): Approve
Jim Baker (community): Approve
Diff: 141 lines (+68/-7)
2 files modified
juju/charm/config.py (+16/-5)
juju/charm/tests/test_config.py (+52/-2)
Revision history for this message
Kapil Thangavelu (hazmat) wrote : Re: [Bug 885551] [NEW] config.yaml should allow for different types

Excerpts from Marco Ceppi's message of Thu Nov 03 03:54:24 UTC 2011:
> Public bug reported:
>
> Currently only string is an acceptable type for data in the config.yaml
> file. Other types should be added, primarily: enum, integer, and boolean
> to help maintain some sanity in configuration options.
>
> ** Affects: juju
> Importance: Undecided
> Status: New
>

There are some additional types available, although we should definitely add
some more. boolean and enum both sound good.

re existing types

integer -> 'int'
float -> 'float'

Revision history for this message
Clint Byrum (clint-fewbar) wrote : Re: config.yaml should have enum and boolean types

I suspect this should be split into TWO bugs, one for enum, and one for boolean. But, bool will likely be very simple, so for now, just leaving this as one bug for somebody to tackle. It needs to be sent to the juju mailing list as a proposed API change first for discussion. Marking Triaged, Wishlist.

summary: - config.yaml should allow for different types
+ config.yaml should have enum and boolean types
Changed in juju:
status: New → Triaged
importance: Undecided → Wishlist
Revision history for this message
Kapil Thangavelu (hazmat) wrote :

assigning myself, boolean only for now.

Changed in juju:
milestone: none → florence
assignee: nobody → Kapil Thangavelu (hazmat)
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

Fixed summary/description to point to boolean only, and forked bug onto #918386 for enums.

summary: - config.yaml should have enum and boolean types
+ config.yaml should have boolean types
description: updated
William Reade (fwereade)
Changed in juju:
assignee: Kapil Thangavelu (hazmat) → William Reade (fwereade)
status: Triaged → In Progress
Revision history for this message
Kapil Thangavelu (hazmat) wrote :

taking this one back over for the python impl

Changed in juju:
assignee: William Reade (fwereade) → Kapil Thangavelu (hazmat)
Kapil Thangavelu (hazmat)
Changed in juju:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.