new package: ssh, from github.com/juju/juju/utils/ssh #167

Merged
merged 1 commit into from Nov 4, 2015

Conversation

Projects
None yet
3 participants
Contributor

davecheney commented Oct 27, 2015

new package: ssh, from github.com/juju/juju/utils/ssh

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

Contributor

davecheney commented Oct 27, 2015

$$merge$$

Contributor

jujubot commented Oct 27, 2015

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

Contributor

jujubot commented Oct 27, 2015

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

Contributor

davecheney commented Oct 27, 2015

$$merge$$

Contributor

jujubot commented Oct 27, 2015

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

Contributor

jujubot commented Oct 27, 2015

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

Contributor

davecheney commented Oct 27, 2015

sscyowts3dl4wfi6e72b

Contributor

davecheney commented Nov 3, 2015

$$$$merge$$

Contributor

jujubot commented Nov 3, 2015

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

Contributor

jujubot commented Nov 3, 2015

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

Contributor

bz2 commented Nov 3, 2015

Its the same issue all build attempts:

# golang.org/x/crypto/ssh
../../../golang.org/x/crypto/ssh/keys.go:492: undefined: crypto.Signer

Getting tip of this doesn't work, the interfaces changed in new go versions. If this is moving to utils, there needs to be a comprehensive dependencies.tsv (not just one line as was added), and the job needs to be updated to use it.

Contributor

davecheney commented Nov 3, 2015

$$merge$$

Contributor

jujubot commented Nov 3, 2015

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

Contributor

jujubot commented Nov 3, 2015

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

Contributor

davecheney commented Nov 3, 2015

$$merge$$

Contributor

jujubot commented Nov 3, 2015

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

Contributor

jujubot commented Nov 4, 2015

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

Contributor

davecheney commented Nov 4, 2015

$$merge$$

This time for sure.

Contributor

jujubot commented Nov 4, 2015

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

Contributor

jujubot commented Nov 4, 2015

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

Contributor

davecheney commented Nov 4, 2015

$$merge$$

Contributor

jujubot commented Nov 4, 2015

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

jujubot added a commit that referenced this pull request Nov 4, 2015

Merge pull request #167 from davecheney/move-ssh
new package: ssh, from github.com/juju/juju/utils/ssh

new package: ssh, from github.com/juju/juju/utils/ssh

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

@jujubot jujubot merged commit 21077b1 into juju:master Nov 4, 2015

@davecheney davecheney deleted the davecheney:move-ssh branch Nov 4, 2015

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