/juju

Fixed lp:1437266 - Bootstrap node occasionally panicing #2320

Merged
merged 2 commits into from May 14, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@themue @jujubot @dimitern
@themue
Contributor

themue commented May 13, 2015

First implementation of backingOpenedPorts.removed() only handled unit IDs after the direct closing of ports. Now the eventing of machine IDs, e.g. after destroying services and machines, correctly too.

(Review request: http://reviews.vapour.ws/r/1681/)
@themue
Contributor

themue commented May 14, 2015

$$merge$$
@jujubot
Contributor

jujubot commented May 14, 2015

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

jujubot commented May 14, 2015

Build failed: Does not match ['fixes-1454658']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/3258
@dimitern
Contributor

dimitern commented May 14, 2015

$$merge$$
@jujubot
Contributor

jujubot commented May 14, 2015

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

jujubot commented May 14, 2015

Build failed: Does not match ['fixes-1454658']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/3259
@themue
Contributor

themue commented May 14, 2015

$$merge$$
@jujubot
Contributor

jujubot commented May 14, 2015

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

jujubot added a commit that referenced this pull request May 14, 2015

@jujubot
Merge pull request #2320 from TheMue/fix-1437266-for-1.24 
Fixed lp:1437266 - Bootstrap node occasionally panicing

First implementation of `backingOpenedPorts.removed()` only handled unit IDs after the direct closing of ports. Now the eventing of machine IDs, e.g. after destroying services and machines, correctly too.

(Review request: http://reviews.vapour.ws/r/1681/)
6475bf4

@jujubot jujubot merged commit 6475bf4 into juju:1.24 May 14, 2015

@themue themue deleted the themue:fix-1437266-for-1.24 branch Jul 9, 2015

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