32
|
|
|
Gary Poster |
11 years ago
|
|
|
31
|
|
|
Francesco Banconi |
11 years ago
|
|
|
30
|
|
|
Francesco Banconi |
11 years ago
|
|
|
29
|
|
|
Francesco Banconi |
11 years ago
|
|
|
28
|
|
|
Francesco Banconi |
11 years ago
|
|
|
27
|
|
|
Francesco Banconi |
11 years ago
|
|
|
26
|
|
|
Francesco Banconi |
11 years ago
|
|
|
25
|
|
Juju GUI source as env var + Charm test branch
This branch introduces the possibility to provide the Juju GUI source (used in tests) by setting an environment variable. This way the charm tests can be run as part of the release qa process, e.g.:
$ JUJU_GUI_SOURCE=lp:mybranch jitsu test juju-gui ...
or, to test a new trunk release:
$ JUJU_GUI_SOURCE=trunk jitsu test juju-gui ...
Also updated the test exercising the process of making a release from a branch, i.e. when the juju-gui-source option points to a Bazaar branch.
Now the branch used is a customized one, present in `lp:~frankban/juju-gui/charm-tests-branch` (please feel free to suggest another, more sane, location, and please take a look at that as part of your reviews). This is a test branch containing only a minimal Makefile and a release tarball. There, `make distfile` is almost a no-op, and for this reason the charm test speed is slightly increased (~4 minutes on ec2). Source: http://bazaar.launchpad.net/~frankban/juju-gui/charm-tests-branch/files
R=gary.poster, teknico CC= https://codereview.appspot.com/7102044
|
Francesco Banconi |
11 years ago
|
|
|
24
|
|
|
Nicola Larosa |
11 years ago
|
|
|
23
|
|
|
Francesco Banconi |
11 years ago
|
|
|
22
|
|
|
Gary Poster |
11 years ago
|
|
|
21
|
|
|
Nicola Larosa |
11 years ago
|
|
|
20
|
|
|
Francesco Banconi |
11 years ago
|
|
|
19
|
|
Allow using an own SSL cert and private key.
Added two new config options, one for the certificate, one for the private key. If they are both provided, they are used by nginx, otherwise, a new certificate is automatically generated.
Fixed a pre-existent bug: even if you can specify the directory where to store the certificates, this path was not used by nginx, because an hardcoded one was present in the configuration file.
Improved how ssl options are handled in config-changes. If the SSL path is changed using 'juju set', now that change is reflected in the nginx config file, and the service correctly restarted.
Added tests for the process of saving or generating SSL certificates.
Some code clean up.
Please note that all the SSL stuff is still disabled/commented.
R=teknico, matthew.scott, gary.poster CC= https://codereview.appspot.com/6976046
|
Francesco Banconi |
11 years ago
|
|
|
18
|
|
Updated the charm to serve Juju GUI releases.
I had a pre-implementation call with Gary and some really useful help from Brad.
Details:
Replaced the juju-gui-branch config option with a new juju-gui-source one: here you can specify where to deploy the GUI from (stable, trunk, a branch, a specific version of stable/trunk).
Added a bunch of functions in utils.py: they help parsing the juju-gui-source option, retrieving the URL of a release, etc.
Added tests for the utility functions above.
Changed the fetch/build function in utils.py, so that now a release tarball is always used to install the GUI. It can be downloaded from Launchpad or created using "make distfile" from a juju-gui checkout. Also reorganized the fetch/build code: separated each of the fetch/build functions into two different functions (one for the GUI, one for the API).
Updated the install and the config-changed hooks to conform to the new fetch/setup functions (mentioned above).
Updated the config-changed hook to reflect changes to the options, install hook and utils.py.
Added one functional test specific to deploying from a branch. The other tests now default to the latest stable release.
Created and uploaded Juju GUI trunk/stable releases.
Updated the stop hook: now the function in utils.py is called with the argument it expects.
Overall code lint and clean up.
All those changes result in a big diff (~820 lines), sorry about that.
R=gary.poster, teknico CC= https://codereview.appspot.com/6977043
|
Francesco Banconi |
11 years ago
|
|
|
17
|
|
|
Nicola Larosa |
11 years ago
|
|
|
16
|
|
|
Brad Crittenden |
11 years ago
|
|
|
15
|
|
|
Gary Poster |
11 years ago
|
|
|
14
|
|
|
Gary Poster |
11 years ago
|
|
|
13
|
|
|
Gary Poster |
11 years ago
|
|
|