Use ConnSuite in state package instead of JujuConnSuite. #6317

Merged
merged 1 commit into from Sep 26, 2016

Conversation

Projects
None yet
3 participants
Member

anastasiamac commented Sep 26, 2016

No description provided.

LGTM

I did double check that ConnSuite wasn't ultimately just a wrapper around JujuConnSuite but it appears to be distinct.

Member

anastasiamac commented Sep 26, 2016

$$merge$$

Contributor

jujubot commented Sep 26, 2016

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

Contributor

jujubot commented Sep 26, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/9322

Member

anastasiamac commented Sep 26, 2016

unrelated failure in github.com/juju/juju/cmd/jujud/agent:
FAIL: machine_test.go:1034: MachineSuite.TestCertificateUpdateWorkerUpdatesCertificate
...
machine_test.go:1063:
s.assertChannelActive(c, updated, "certificate to be updated")
util_test.go:104:
c.Fatalf("timeout while waiting for %v", intent)
... Error: timeout while waiting for certificate to be updated

$$merge$$

Contributor

jujubot commented Sep 26, 2016

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

@jujubot jujubot merged commit b5857c4 into juju:master Sep 26, 2016

@anastasiamac anastasiamac deleted the anastasiamac:remove-jujuconnsuite-state branch Sep 26, 2016

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