/juju

api: implement CACert entry point #7327

Merged
merged 1 commit into from May 11, 2017

Conversation

Reviewers

@axw axw

@mhilton mhilton

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@rogpeppe @jujubot @axw @mhilton
@rogpeppe
Owner

rogpeppe commented May 10, 2017

This will enable a client to find out the CA certificate from a controller
even when it has connected without a CA cert (for example when connecting
to a public controller). This means that when migrating models,
the source controller can find out the correct CA certificate to install
in the agent configuration files.
@mhilton

mhilton approved these changes May 11, 2017
View changes

LGTM

@axw

axw approved these changes May 11, 2017
View changes

@rogpeppe
Owner

rogpeppe commented May 11, 2017

$$merge$$
@jujubot
Contributor

jujubot commented May 11, 2017

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

jujubot commented May 11, 2017

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10867
@rogpeppe
Owner

rogpeppe commented May 11, 2017

$$merge$$
@jujubot
Contributor

jujubot commented May 11, 2017

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

rogpeppe commented May 11, 2017

$$merge$$

@jujubot jujubot merged commit f949826 into juju:develop May 11, 2017
1 check failed

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

rogpeppe added a commit to rogpeppe/juju that referenced this pull request May 12, 2017

@rogpeppe
api, apiserver: implement CACert on MigrationTarget 
A previous PR (juju#7327)
added the CA cert entry point to Client, but the Client
facade isn't available in the controller API connection
that the migration process makes, so make the
call available in the MigrationTarget API too.
2a5c03c

@rogpeppe rogpeppe referenced this pull request May 12, 2017

Merged

api, apiserver: implement CACert on MigrationTarget #7337

rogpeppe added a commit to rogpeppe/juju that referenced this pull request May 12, 2017

@rogpeppe
api, apiserver: implement CACert on MigrationTarget 
A previous PR (juju#7327)
added the CA cert entry point to Client, but the Client
facade isn't available in the controller API connection
that the migration process makes, so make the
call available in the MigrationTarget API too.
ccc547a

rogpeppe added a commit to rogpeppe/juju that referenced this pull request May 12, 2017

@rogpeppe
api, apiserver: implement CACert on MigrationTarget 
A previous PR (juju#7327)
added the CA cert entry point to Client, but the Client
facade isn't available in the controller API connection
that the migration process makes, so make the
call available in the MigrationTarget API too.
517395f

jujubot added a commit that referenced this pull request May 12, 2017

@jujubot
Merge pull request #7337 from rogpeppe/160-migration-target-cacert 
api, apiserver: implement CACert on MigrationTarget

A previous PR (#7327)
added the CA cert entry point to Client, but the Client
facade isn't available in the controller API connection
that the migration process makes, so make the
call available in the MigrationTarget API too.
7ca69b4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment