Code review comment for lp:~frankban/charms/precise/juju-gui/bug-1088618-serve-releases

Revision history for this message
Gary Poster (gary) wrote :

Good news and bad news. The good news is that the deployment took three
minutes instead of 20 for me. Rock! The bad news is that the tests did
not pass for me. I have to run now, but this is what I see so far.

$ time JUJU_REPOSITORY=~/dev/repo/ ~/bin/jitsu test juju-gui --logdir
/tmp --timeout 40m --no-bootstrap
2012-12-19 19:30:10,417 jitsu.test:INFO Running unit test: deploy.test
test_api_agent (__main__.DeployTest) ... ok
test_branch_source (__main__.DeployTest) ... ERROR
test_customized_api_port (__main__.DeployTest) ... ERROR
test_staging (__main__.DeployTest) ... ERROR
test_api_agent (__main__.DeployToTest) ...

If I see any helpful log messages I'll pass them on later. Anyway,
"Land with changes": please verify that the tests pass for you before
landing.

Thanks

Gary

https://codereview.appspot.com/6977043/

« Back to merge proposal