cf-charmers team mailing list archive
-
cf-charmers team
-
Mailing list archive
-
Message #00227
Re: [Merge] lp:~lomov-as/charm-helpers/cloud-foundry-multiple-units-in-relation-context into lp:~cf-charmers/charm-helpers/cloud-foundry
The idea here is fine. I'd suggest that this is a different context type as it implies a different relationship to the template is manages however. Any consumer/template needs to be prepared to iterate a list.
Maybe a subclass called RelationListContext which sets multiple units to True and the base class doesn't expose a way to change this directly, set to False in the class. That way we share the implmentation but it is explicit to the template what they will be consuming.
--
https://code.launchpad.net/~lomov-as/charm-helpers/cloud-foundry-multiple-units-in-relation-context/+merge/220508
Your team Cloud Foundry Charmers is requested to review the proposed merge of lp:~lomov-as/charm-helpers/cloud-foundry-multiple-units-in-relation-context into lp:~cf-charmers/charm-helpers/cloud-foundry.
Follow ups
References