Serve static files using nginx

Serve static files using nginx

The charm now serves static files using nginx.  There is another branch to fix the websocket URL in config.js, but until then, the URL will default to localhost.

https://code.launchpad.net/~makyo/charms/precise/juju-gui/serve-static/+merge/138502

(do not edit description out of merge proposal)

[matthew.scott, 2012-12-06 16:31:03]

Please take a look.

[matthew.scott, 2012-12-06 16:59:52]

Please take a look.

[hazmat, 2012-12-06 17:32:37]

https://codereview.appspot.com/6898043/diff/3001/config/nginx.conf.template
File config/nginx.conf.template (right):

https://codereview.appspot.com/6898043/diff/3001/config/nginx.conf.template#n...
config/nginx.conf.template:2: listen 8888;
we should just put this on 80. also needs an update to the open-port call.

[teknico, 2012-12-07 15:11:53]

Nice branch.

https://codereview.appspot.com/6898043/diff/3001/hooks/start
File hooks/start (right):

https://codereview.appspot.com/6898043/diff/3001/hooks/start#newcode50
hooks/start:50: build_dir += staging and 'debug' or 'prod'
Man, it's time to drop the old tricks. :-)

build_dir += 'debug' if staging else 'prod'

[bac, 2012-12-10 13:43:51]

Hope you don't mind a third review.  Looks good with some small suggestions. 
Re-review not required.

https://codereview.appspot.com/6898043/diff/3001/hooks/install
File hooks/install (right):

https://codereview.appspot.com/6898043/diff/3001/hooks/install#newcode50
hooks/install:50: 
The run command returns stdout.  My branch adds a 'cmd_log' command which will
log the output if run is wrapped by it, i.e.

cmd_log(run(...))

If my branch lands before yours please consider adding it to any of the
run-based commands, which includes 'command' and derivatives.

https://codereview.appspot.com/6898043/diff/3001/revision
File revision (right):

https://codereview.appspot.com/6898043/diff/3001/revision#newcode1
revision:1: 16
I think it is pretty trivial, but shouldn't the revision number on trunk only
increment by 1 instead of leaving it at the number you used during development? 
Doing so will require an examination of trunk just before you land.

[matthew.scott, 2012-12-10 16:05:06]

*** Submitted:

Serve static files using nginx

The charm now serves static files using nginx.  There is another branch to fix
the websocket URL in config.js, but until then, the URL will default to
localhost.

R=hazmat, teknico, bac
CC=
https://codereview.appspot.com/6898043