Add fuzz to the bounce and restart durations in the dependency engine #6539

Merged
merged 1 commit into from Nov 4, 2016

Conversation

Projects
None yet
5 participants
Contributor

dooferlad commented Nov 3, 2016

There was a TODO...

TODO(fwereade): we should add fuzz to the bounce and restart durations to more vigorously shake out the bugs...

It has been done. Fixed docs to match and also render correctly with godoc.

There was a TODO:
TODO(fwereade): we should add fuzz to the bounce and restart durations to more vigorously shake out the bugs...

It has been done. Fixed docs to match and also render correctly with godoc.

@dooferlad dooferlad changed the base branch from staging to develop Nov 3, 2016

Contributor

bz2 commented Nov 3, 2016

!!retry!!

Contributor

bz2 commented Nov 3, 2016

!!reretry!!

Owner

nskaggs commented Nov 3, 2016

mgz add to whitelist

Contributor

bz2 commented Nov 3, 2016

!!rereretry!!

Contributor

bz2 commented Nov 3, 2016

!!rerereretry!!

Contributor

bz2 commented Nov 3, 2016

!!rererereretry!!

Contributor

dooferlad commented Nov 4, 2016

$$merge$$

Contributor

jujubot commented Nov 4, 2016

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

Contributor

jujubot commented Nov 4, 2016

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

Contributor

dooferlad commented Nov 4, 2016

$$merge$$

Contributor

jujubot commented Nov 4, 2016

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

@jujubot jujubot merged commit d78e69a into juju:develop Nov 4, 2016

1 check passed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment