/juju

cmd/jujud/reboot: fix reboot worker and test #4749

Merged
merged 2 commits into from Mar 17, 2016

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 Mar 15, 2016

@bogdanteleaga spotted that I had accidentally neutered the timeout in
the reboot worked.

Fix, and fix the test to actually test that the test fails.

(Review request: http://reviews.vapour.ws/r/4186/)
@davecheney
cmd/jujud/reboot: fix reboot worker and test 
@bogdanteleaga spotted that I had accidentally neutered the timeout in
the reboot worked.

Fix, and fix the test to actually test that the test fails.
68bf450
@davecheney
Contributor

davecheney commented Mar 16, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Mar 16, 2016

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

jujubot commented Mar 16, 2016

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

davecheney commented Mar 17, 2016

$$JFDI$$
@jujubot
Contributor

jujubot commented Mar 17, 2016

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

jujubot added a commit that referenced this pull request Mar 17, 2016

@jujubot
Merge pull request #4749 from davecheney/x 
cmd/jujud/reboot: fix reboot worker and test

@bogdanteleaga spotted that I had accidentally neutered the timeout in
the reboot worked.

Fix, and fix the test to actually test that the test fails.

(Review request: http://reviews.vapour.ws/r/4186/)
00e499e

@jujubot jujubot merged commit 00e499e into juju:master Mar 17, 2016

@davecheney davecheney deleted the davecheney:x branch Mar 17, 2016

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