Code review comment for lp:~bac/charms/precise/juju-gui/enverror

Revision history for this message
Brad Crittenden (bac) wrote :

To QA:

The charm needs to have the juju-gui trunk. I tried just setting
juju-gui-source but ran into all kinds of problems.
What I eventually did was make a new release in juju-gui and copy it
over to my charm '/releases' directory. That's what I describe here.

In juju-gui:
BRANCH_IS_GOOD=1 make distfile
Note the new tar ball created.

0) Grab the branch: lp:~bac/charms/precise/juju-gui/enverror
0.1) Copy tarball from juju-gui to the charm releases directory.
1) juju switch local
2) sudo juju bootstrap
3) make deploy

Now go to the GUI. Search for 'owner: bac'. Drag bac's Jenkins bundle
onto the canvas. After a short while you should get a notification
which says:

Updated status for deployment id: 0

The deployment errored: json: cannot unmarshal string into Go value of
type uint64

https://codereview.appspot.com/34270044/diff/1/server/guiserver/tests/bundles/test_base.py
File server/guiserver/tests/bundles/test_base.py (right):

https://codereview.appspot.com/34270044/diff/1/server/guiserver/tests/bundles/test_base.py#newcode253
server/guiserver/tests/bundles/test_base.py:253: 'Error': "bad wolf",
The returned error is now more concise.

https://codereview.appspot.com/34270044/

« Back to merge proposal