maas-devel team mailing list archive

[Gavin Panella <gavin.panella@xxxxxxxxxxxxx>]

On 10 October 2012 10:23, Martin Packman <martin.packman@xxxxxxxxxxxxx> wrote:
>>   - couldn't use tags to deploy with juju. Got an error "no such tag":
>> https://bugs.launchpad.net/maas/+bug/1064734
>
> This is painful as juju 0.6 was released yesterday, but we need the
> fix there. Short story, my initial code version involved a set and
> used ",".join when passing on to api, then simplified to just pass a
> string straight through, then in response to review re-added the set
> logic to pre-validate tags but missed adapting the api back again. So
> MaaS is passed the stringified set rather than the original tag
> string. The provider needs some better unit test coverage for
> constraint/api interactions.

It's ugly as hell, but is it worth adding a temporary workaround to
extract tags from a string of the form "set([...])"?