api/controller: allow empty CACert in migration #7315

Merged
merged 1 commit into from May 11, 2017

Conversation

Projects
None yet
5 participants
Owner

rogpeppe commented May 8, 2017

This allows a model to be migrated to a controller with a
global CA certificate, the common case when using the
JIMM controller.

QA add a controller with an autocert-dns-name configuration,
configure the appropriate DNS entry to point to the controller
instance, remove the ca-cert entry from its controllers.yaml entry,
and try to migrate a model from another controller to that one.

Looks good thanks!

mhilton approved these changes May 9, 2017

LGTM

Owner

rogpeppe commented May 9, 2017

$$merge$$

Contributor

jujubot commented May 9, 2017

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

Owner

rogpeppe commented May 9, 2017

$$merge$$

Contributor

jujubot commented May 9, 2017

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

mjs approved these changes May 10, 2017

Contributor

mjs commented May 10, 2017

$$merge$$

Contributor

jujubot commented May 10, 2017

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

Contributor

jujubot commented May 10, 2017

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

Owner

rogpeppe commented May 11, 2017

$$merge$$

Contributor

jujubot commented May 11, 2017

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

@jujubot jujubot merged commit 633d641 into juju:develop May 11, 2017

1 check failed

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