/juju

state: remove APIAddressesFromMachines #7326

Merged
merged 1 commit into from May 11, 2017

Conversation

Reviewers

@axw axw

@mhilton mhilton

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@rogpeppe @jujubot @axw @mhilton
@rogpeppe
Owner

rogpeppe commented May 10, 2017

This method is not used, so remove it.

QA no regressions
@rogpeppe
state: remove APIAddressesFromMachines 
This method is not used, so remove it.
b2118bc
@mhilton

mhilton approved these changes May 10, 2017
View changes

LGTM

@axw

axw approved these changes May 11, 2017
View changes

@rogpeppe
Owner

rogpeppe commented May 11, 2017

$$merge$$
@jujubot
Contributor

jujubot commented May 11, 2017

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

rogpeppe commented May 11, 2017

$$merge$$
@jujubot
Contributor

jujubot commented May 11, 2017

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10865
@rogpeppe
Owner

rogpeppe commented May 11, 2017

$$merge$$
@jujubot
Contributor

jujubot commented May 11, 2017

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

rogpeppe commented May 11, 2017

$$merge$$

@jujubot jujubot merged commit 40541af into juju:develop May 11, 2017
1 check failed

1 check failed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment