fix data race noted in lp:1579057 #5356

Merged
merged 2 commits into from May 6, 2016

Conversation

Projects
None yet
3 participants
Contributor

fwereade commented May 6, 2016

We were leaking references to internal relation member dicts out of Snapshot.

(Review request: http://reviews.vapour.ws/r/4786/)

Owner

cmars commented May 6, 2016

LGTM

Contributor

fwereade commented May 6, 2016

$$merge$$

Contributor

jujubot commented May 6, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented May 6, 2016

Build failed: Does not match ['fixes-1564511', 'fixes-1578286', 'fixes-1579059', 'fixes-1579127', 'fixes-1579062', 'fixes-1576874', 'fixes-1579057', 'fixes-1578906', 'fixes-1579051']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7671

Contributor

fwereade commented May 6, 2016

$$fixes-1579057$$

Contributor

jujubot commented May 6, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 03981e4 into juju:master May 6, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment