]> review.fuel-infra Code Review - openstack-build/heat-build.git/commit
Get list properties working again
authorZane Bitter <zbitter@redhat.com>
Thu, 31 May 2012 09:17:30 +0000 (11:17 +0200)
committerZane Bitter <zbitter@redhat.com>
Thu, 31 May 2012 10:09:58 +0000 (12:09 +0200)
commitcfe391073a681ba8f1b806fc53428adb36486181
tree735a9e107c807549a392c6eb13b87a73b93ee4eb
parentd1f71f6d5d4dbbcc68d33f3779f09aaaf4aefd19
Get list properties working again

The CloudFormation documentation is very confusing on this point.
'CommaDelimitedList' is one of the data types that are valid for a
Parameter (the others are 'String' and 'Number'). A CommaDelimitedList
parameter takes the form of a string where the list members are delimted by
commas:

    "item1,item2,item3"

However the documentation also uses the phrase "Comma Delimited List" to
refer to the type of some Resource Properties that are, in fact, simply
lists:

    [ "item1" , "item2" , "item3" ]

...as if there were *another* way to represent lists. (Note that the
items here need not be strings, and in fact are usually objects of some
variety.)

So we need a different data type to represent the latter. (This patch
changes the name from 'TupleList' to just 'List', since the actual Python
sequence type is just an implementation detail.) In future, we should
probably also verify that only the 3 valid Parameter types are used, and
perhaps that list Properties contain only objects of the correct type.

Change-Id: I94054f588fc37f7d4ba245f2e92b86ac9c872c37
Signed-off-by: Zane Bitter <zbitter@redhat.com>
heat/engine/checkeddict.py
heat/engine/instance.py
heat/engine/security_group.py
heat/engine/user.py
heat/engine/volume.py