/juju

provider/local: fix syscall.ECONNREFUSED detection #2304

Merged
merged 3 commits into from May 13, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@davecheney @jujubot
@davecheney
Contributor

davecheney commented May 13, 2015

Go 1.5 has changed the type of the error message returned, basically
added another level of indirection.

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

davecheney added some commits May 13, 2015

@davecheney
provider/local: fix syscall.ECONNREFUSED detection 
Go 1.5 has changed the type of the error message returned, basically
added another level of indirection.
088571c
@davecheney
provider/local: fix syscall.ECONNREFUSED detection 
Go 1.5 has changed the type of the error message returned, basically
added another level of indirection.
c58c0e4
@davecheney
Merge branch '224-detect-econnrefused-in-go15' of github.com:davechen… 
…ey/juju into 224-detect-econnrefused-in-go15

Conflicts:
	provider/local/environprovider.go
0a0bf8c
@davecheney
Contributor

davecheney commented May 13, 2015

$$merge$$
@jujubot
Contributor

jujubot commented May 13, 2015

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

jujubot commented May 13, 2015

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/3219
@davecheney
Contributor

davecheney commented May 13, 2015

$$comeinspinner$$
@jujubot
Contributor

jujubot commented May 13, 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 13, 2015

@jujubot
Merge pull request #2304 from davecheney/224-detect-econnrefused-in-go15 
provider/local: fix syscall.ECONNREFUSED detection

Go 1.5 has changed the type of the error message returned, basically
added another level of indirection.

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

@jujubot jujubot merged commit db5080a into juju:master May 13, 2015

@ericsnowcurrently ericsnowcurrently referenced this pull request Sep 10, 2015

Merged

Adjust to a change in Go 1.5. #3245

jujubot added a commit that referenced this pull request Sep 10, 2015

@jujubot
Merge pull request #3245 from ericsnowcurrently/fix-conn-refused-1493503 
Adjust to a change in Go 1.5.

This is a backport of PR #2304.  Essentially Go 1.5 introduced a backward-incompatible change for which we need to compensate.

Fixes https://bugs.launchpad.net/juju-core/+bug/1493503.

(Review request: http://reviews.vapour.ws/r/2624/)
a925c8e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment