all: disable mongo SSL where not necessary #7297

Merged
merged 1 commit into from May 1, 2017

Conversation

Projects
None yet
3 participants
Member

axw commented May 1, 2017

Description of change

For tests that require Mongo to be running,
disable SSL except integration and end-to-end
tests (e.g. in the jujud/agent and feature
tests). Disabling SSL speeds up the tests
significantly. In my testing, the state
package takes ~500 seconds with SSL, and
~170 seconds without.

This branch updates to a newer juju/testing,
which defaults the storage engine to mmapv1.

QA steps

Run the tests.

Documentation changes

None.

Bug reference

None.

Looks great!

Member

axw commented May 1, 2017

$$merge$$

Contributor

jujubot commented May 1, 2017

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

all: disable mongo SSL where not necessary
For tests that require Mongo to be running,
disable SSL except integration and end-to-end
tests (e.g. in the jujud/agent and feature
tests). Disabling SSL speeds up the tests
significantly. In my testing, the state
package takes ~500 seconds with SSL, and
~170 seconds without.
Contributor

jujubot commented May 1, 2017

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10796

Member

axw commented May 1, 2017

$$merge$$

Contributor

jujubot commented May 1, 2017

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

@jujubot jujubot merged commit aab1e89 into juju:develop May 1, 2017

1 check passed

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