juju: add support for redirection when connecting to a model #5684

Merged
merged 1 commit into from Jun 23, 2016

Conversation

Projects
None yet
3 participants
Owner

rogpeppe commented Jun 21, 2016

We are going to have controllers which multiplex other
controllers. In such a case, we don't want to require
all the network traffic for the models that the sub-controllers
control to pass through the top level hub, so allow
a controller to return a "redirect required" error to cause
a client to connect to the model API in the sub-controller
(or some other server) directly.

There is more work to be done after this lands. In particular,
we should cache the API addresses that we were redirected
to and associate them with the model, so we can connect
directly on subsequent API opens.

(Review request: http://reviews.vapour.ws/r/5127/)

juju: add support for redirection when connecting to a model
We are going to have controllers which multiplex other
controllers. In such a case, we don't want to require
all the network traffic for the models that the sub-controllers
control to pass through the top level hub, so allow
a controller to return a "redirect required" error to cause
a client to connect to the model API in the sub-controller
(or some other server) directly.

There is more work to be done after this lands. In particular,
we should cache the API addresses that we were redirected
to and associate them with the model, so we can connect
directly on subsequent API opens.
Owner

rogpeppe commented Jun 22, 2016

$$merge$$

Contributor

jujubot commented Jun 22, 2016

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

Contributor

jujubot commented Jun 22, 2016

Build failed: Does not match ['fixes-1593299', 'fixes-1594580', 'fixes-1593761', 'fixes-1591290', 'fixes-1581157', 'fixes-1593838', 'fixes-1586890']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8232

Owner

rogpeppe commented Jun 22, 2016

$$JFDI$$ (signed off courtesy of cherylj)

Contributor

jujubot commented Jun 22, 2016

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

Contributor

jujubot commented Jun 22, 2016

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

cherylj commented Jun 22, 2016

$$JFDI$$

Contributor

jujubot commented Jun 22, 2016

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

Contributor

jujubot commented Jun 22, 2016

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

Owner

rogpeppe commented Jun 23, 2016

$$JFDI$$

Contributor

jujubot commented Jun 23, 2016

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

@jujubot jujubot merged commit 12cbfa9 into juju:master Jun 23, 2016

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