~gary/charms/precise/juju-gui/download-cache

Viewing all changes in revision 127.

  • Committer: Francesco Banconi
  • Date: 2013-11-13 19:03:23 UTC
  • mfrom: (126.1.3 install-yaml)
  • Revision ID: francesco.banconi@canonical.com-20131113190323-lrzibwcp9xs30nsl
Fix base Python dependencies.

The install hook imported charmhelpers
before installing python-yaml, and 
charmhelpers imports from yaml.

This very old bug has been hidden by the 
fact that python-yaml is automatically 
installed in ec2/lxc instances, but showed
itself when using the manual provider.

Copy/pasting and integrating the QA from 
my last branch, a double check can help
before the release.
QA:
- Bootstrap a Juju environment with --debug.
- Deploy and expose the GUI (make deploy).
- Check juju debug-log (or
  less ~/.juju/local/log/unit-juju-gui-0.log if you
  used lxc): you should see the following message
  at the beginning of the install hook log:
  "Installing base Python dependencies: python-apt, 
  python-launchpadlib, python-tempita, python-yaml."
- 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/26190043

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: