/juju

api: fix racy test #7440

Merged
merged 1 commit into from Jun 1, 2017

Conversation

Reviewers

@wallyworld wallyworld

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@rogpeppe @wallyworld @jujubot
@rogpeppe
Owner

rogpeppe commented Jun 1, 2017

The DNS fallback functionality meant that we could
make two dials when only one was expected,
giving rise to a race (for example http://reports.vapour.ws/releases/5331/job/run-unit-tests-race/attempt/2831)
@rogpeppe
api: fix racy test 
The DNS fallback functionality meant that we could
make two dials when only one was expected,
giving rise to a race.
3b459d5
@wallyworld
Owner

wallyworld commented Jun 1, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Jun 1, 2017

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

@jujubot jujubot merged commit 67e8f99 into juju:develop Jun 1, 2017
1 check passed

1 check passed

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