lp://staging/~huwshimi/charms/trusty/juju-gui/config-version

Branch merges

Propose for merging

No branches dependent on this one.

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

223. By Huw Wilkins on 2014-12-10

WIP write the juju version to the config.

222. By Jeff Pihach on 2014-11-27

Merged back the new charm release.

221. By Jeff Pihach on 2014-11-27

Updated to the newest juju-gui release.

220. By Francesco Banconi on 2014-11-18

Merged back the new charm release.

219. By Richard Harding on 2014-11-18

Update README to reflect recent changes.

218. By Francesco Banconi on 2014-11-18

Merged back the new charm release.

217. By Francesco Banconi on 2014-11-18

Updated to the newest juju-gui release.

216. By Francesco Banconi on 2014-11-18

Merged changes from the released charm.

215. By Francesco Banconi on 2014-11-18

Merged changes from the released charm.

214. By Francesco Banconi on 2014-11-14

Support user defined port.

Allow specifying a port on which the
GUI server starts listening for requests.

Tests: `make unittest`.

QA:
- bootstrap an environment;
- `make deploy`;
- wait for the unit to be deployed;
- visit the GUI URL, ensure port 80 redirects to 443 as usual
  and the GUI works as usual;
- switch to a customized port: `juju set juju-gui port=8080`;
- ensure the GUI is no longer reachable at port 80 or 443,
  instead it works correctly if you visit `https://<GUI-URL>:8080/`;
- switch to a another customized port: `juju set juju-gui port=443`;
- this time the GUI is still listening on default port for HTTPS,
  but port 80 no longer redirects to HTTPS;
- switch to an invalid port: `juju set juju-gui port=80000`;
- the GUI refuses to do that, and it falls back to using
  the default configuration: ensure port 80 redirects to 443;
- switch again to a customized valid port: `juju set juju-gui port=12345`;
- ensure the GUI server is listening on that port;
- finally, unset the port value: `juju unset juju-gui port`;
- back to defaults: ensure port 80 redirects to 443;
- destroy the environment.

Done, thank you.

R=rharding, jeff.pihach
CC=
https://codereview.appspot.com/174170043