/juju

juju gui command: retrieve the API host from the controller connection. #6482

Merged
merged 1 commit into from Oct 25, 2016

Conversation

Reviewers

@mhilton mhilton

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@frankban @dimitern @jujubot @mhilton
@frankban
Member

frankban commented Oct 20, 2016

This will allow us to properly handle HTTP requests for the GUI from alternative controllers.
@mhilton

mhilton approved these changes Oct 20, 2016
View changes

LGTM, thanks

@frankban
Member

frankban commented Oct 20, 2016

To QA this, bootstrap controllers on lxd and a public cloud, then run "juju gui", log into the GUI and ensure it works.
@dimitern
Contributor

dimitern commented Oct 25, 2016

LGTM and QA OK - tested with LXD and AWS.
@frankban
Member

frankban commented Oct 25, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Oct 25, 2016

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

@jujubot jujubot merged commit 125eb3b into juju:develop Oct 25, 2016
1 check passed

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