~hatch/charms/trusty/juju-gui/proxy-ws-request

Viewing all changes in revision 226.

  • Committer: Francesco Banconi
  • Date: 2015-01-05 18:12:24 UTC
  • mfrom: (225.1.4 test-fixes)
  • Revision ID: francesco.banconi@canonical.com-20150105181224-mq1y9loqmbdr29so
Update to Tornado 3.2.2.

This is done as an attempt to fix the
Tornado race condition described in [1]

Also use the newest version of the
futures package.

Add the missing firefox dependency to the 
sysdeps make target.

Simplify the GUI server WebSocket client now that
Tornado supports close() out of the box.

Update the utils.clone_request function so that
the ProxyHandler does not fail if a GET request
is performed with an empty string body.

Other tests fixes.

Tests: `make lint unittest`

QA:
- bootstrap an environment;
- run `make deploy` and wait for the GUI to be ready;
- use the GUI as usual to deploy charms and bundles:
  ensure everything works as usual, with events correctly
  reported by the GUI and no disconnections from core
  happen;
- use `juju ssh juju-gui/0 sudo tailf /var/log/upstart/guiserver.log`
  to check no errors are reported by the GUI server while
  working with the GUI;
- ensure the juju-gui icon is correctly displayed in the service view
  (meaning local charm icons are still properly retrieved from core);
- destroy the environment.

[1] https://bugs.launchpad.net/juju-gui/+bug/1402061

R=rharding, martin.hilton
CC=
https://codereview.appspot.com/195720043

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: