lp:~frankban/charms/precise/juju-gui/new-ws-client

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~juju-gui/charms/precise/juju-gui/trunk at revision 78

charmers: Pending requested 2013-07-22

Diff: 809 lines (+420/-184)

9 files modified

revision (+1/-1)
server/guiserver/clients.py (+65/-84)
server/guiserver/handlers.py (+89/-13)
server/guiserver/tests/helpers.py (+21/-29)
server/guiserver/tests/test_clients.py (+36/-28)
server/guiserver/tests/test_handlers.py (+98/-28)
server/guiserver/tests/test_utils.py (+66/-0)
server/guiserver/utils.py (+44/-0)
tests/requirements.pip (+0/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

84. By Francesco Banconi on 2013-07-23

Bumped revision up.

83. By Francesco Banconi on 2013-07-23

merged trunk.

82. By Francesco Banconi on 2013-07-23

Changes as per review.

81. By Francesco Banconi on 2013-07-22

Checkpoint.

80. By Francesco Banconi on 2013-07-22

Ready to propose.

79. By Francesco Banconi on 2013-07-22

Tests working.

78. By Francesco Banconi on 2013-07-20

Implemented client. Handler refactoring.

77. By Francesco Banconi on 2013-07-19

Update revision and test requirements.

76. By Francesco Banconi on 2013-07-19

Initial implementation of the Juju GUI server.

This first cut of the server replaces the
functionality provided in the charm by
haproxy and apache.

The server includes:
- static files serving;
- a WebSocket proxy handling the
  connections between the browser and
  the server and between the server and
  the Juju API;
- HTTP to HTTPS redirecting.

Further features like auth and deployer
management will be added in the future.

ws4py is used to implement the WebSocket
client: there is something to fix there and
it will be done in a future branch.

At this time there is no easy way to QA the
server, but tests can be run using
`make unittest`: the test/lint suite is already
integrated with the one used by the charm.

Unfortunately the diff is quite long, sorry!

R=gary.poster, bac
CC=
https://codereview.appspot.com/11530043

75. By Madison Scott-Clary on 2013-07-18

Charm dir perms in core #1202772

Apache was giving a 403 when charm was deployed using core due to perms on the charm dir.

R=benjamin.saller, gary.poster
CC=
https://codereview.appspot.com/11540043