Code review comment for lp:~s-matyukevich/charms/trusty/elasticsearch/elasticsearch-dns-bug-fix

Revision history for this message
Sergey Matyukevich (s-matyukevich) wrote :

> I've done a separate MP which in the end is the same solution (socket), but
> fits in with ansible (ie. without having to patch charmhelpers). The branch
> also updates a few other things that needed doing:
>
> https://code.launchpad.net/~michael.nelson/charms/trusty/elasticsearch/use-
> new-charmhelpers/+merge/239935
>
> It'd be good to mark this MP as obsolete, assuming Sergey agrees and is happy
> with the above MP.

I don't find how I can mark my merge proposal as obsolete, but I am totally happy with your solution.

« Back to merge proposal