lp://staging/~frankban/charms/precise/juju-gui/install-yaml
- Get this branch:
- bzr branch lp://staging/~frankban/charms/precise/juju-gui/install-yaml
Branch merges
- charmers: Pending requested
-
Diff: 145 lines (+28/-25)6 files modifiedhooks/backend.py (+1/-2)
hooks/install (+10/-6)
hooks/utils.py (+12/-12)
revision (+1/-1)
server-requirements.pip (+2/-0)
tests/test_backends.py (+2/-4)
Branch information
- Owner:
- Francesco Banconi
- Status:
- Development
Recent revisions
- 126. By Francesco Banconi
-
Update requirements and improve their handling.
Updated the juju-deployer and jujucleint
dependencies to the latest versions, which
include our recent fixes. This way we avoid
using our own forks of the projects.Updated the code that installs the builtin
server dependencies: now a pip requirement
file is used, and the test requirement file
includes the former. The overall dependency
infrastructure should now be less confusing.Also added documentation about how to update
the builtin server requirements.Removed the no longer required --upload-tools
from the functional tests runner.The deployer functional tests now also use
bundles including numunits > 1 and constraints.QA:
- Bootstrap a Juju environment.
- Deploy and expose the GUI (make deploy).
- Wait for the GUI to be ready/started.
- Deploy this bundle: http://pastebin. ubuntu. com/6411548/
- Check everything is ok, xy annotations work (
the services are vertically aligned), wordpress has
customized constraints, mysql customized options and
two units.
- No try to deploy the same bundle again, you
will see a "services already there" kind of error.R=gary.poster
CC=
https://codereview. appspot. com/26130043 - 125. By Brad Crittenden
-
Ensure JUJU_ENV is set when required.
R=jeff.pihach
CC=
https://codereview. appspot. com/24790045 - 122. By Francesco Banconi
-
Improve bundle deployment error handling.
Workaround for http://
bugs.python. org/issue169233 5. Include a customized jujuclient tarball in the
charm, The diff with the original is here:
http://pastebin. ubuntu. com/6377430/ Add a first fix to the testing environment creation
so that the local dependencies are installed
when available.Also fixed the headers sent by the info handler.
Tests: make unittest
QA:
I used this bundle: http://pastebin. ubuntu. com/6377441/
to live test the branch.
Bootstrap a juju environment, run `make deploy` and then
switch the gui source to lp:juju-gui.
When everything is ready, try to deploy the bundle above:
it will fail because it includes local charms, but the
server will not explode, and it will be possible
to deploy other valid bundles after the first one.
The GUI user does not receive notifications, and that's
normal since deployments are not yet watched by the GUI.
Go to https://[GUI URL]/gui-server- info and you should
find the deployment status to be completed with the
expected error.R=rharding, bac
CC=
https://codereview. appspot. com/23000043 - 121. By Francesco Banconi
-
Fix deployer integration.
Fix the deployer error when the bundle includes
services with constraints.Updated the deployer dependency. The new tarball
is generated from lp:~frankban/juju-deployer/guienv-fixes
I will take care of proposing that branch later, however,
the diff can be found here:
http://pastebin. ubuntu. com/6376113/ This branch includes the work Benji did for fixing
this issue.TODO: but these are not release blockers IMHO:
- improve the way we handle the set up of
testing/production environments (as Rick suggested);
- improve the GUI server bundle validation, e.g. disallow
the deployment of local charms, or better check that the
YAML structure is what we expect;
- investigate and fix the bundle deployment error handling:
why errors in the deployer process are not propagated?
Why concurrent.futures explodes with an error while trying
to set up the exception to be propagated in the Future?
This is important because right now an error in the
deployer is not notified, so the deployment is forever
considered in progress and all the other deployments
will be just queued... Actually this is "almost" a release
blocker.Tests: make unittest
QA:
I used this bundle: http://pastebin. ubuntu. com/6376098/
to live test the branch.
Bootstrap a juju environment, run `make deploy` and then
switch the gui source to lp:juju-gui.
When everything is ready, try to deploy a bundle
which includes num_units != 1, customized config and
constraints. Check the bundle is deployed correctly
and the resulting services have the defined
number of units, options and constraints.
Now try to deploy a bundle with invalid constraints: right
after the drag and drop, the GUI should notify an
invalid constraints error.R=rharding, benji
CC=
https://codereview. appspot. com/22810043
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp://staging/charms/juju-gui