← Back to team overview

cf-charmers team mailing list archive

Re: [Charmhelpers] Adding service with multimple units using RelationContext.

 

Hey, Cory and Ben.

Could you took a look on my changes in this MP:
https://code.launchpad.net/~lomov-as/charm-helpers/cloud-foundry-multiple-units-in-relation-context/+merge/220508
.

I wanted code from old versions of charms to work with and to add new
functionality.

I had an error creating MP with lbox during posting diffs to
codereview.appspot.com: error: Failed to send patch set to codereview:
empty user. This is second time I have it, if you know how to solve it
would be great.

Thanks you,
Alex L.


On 21 May 2014 15:18, Alexander Lomov <lomov.as@xxxxxxxxx> wrote:

> Is it okey that every unit rewrites data for the whole service relation?
>
> What if we have several units in service ?
>
> I am talking about this code (
> http://bazaar.launchpad.net/~cf-charmers/charm-helpers/cloud-foundry/view/head:/charmhelpers/core/templating.py#L75
> ).
>
> As far as we moved relation context to we need to take care of it. Also
> taking information from several units will be useful (sometime required)
> for cf-loggregator charm.
>
> Thank you,
> Alex L.
>

References