Merge lp:~hazmat/charms/trusty/juju-gui/trunk into lp:charms/trusty/juju-gui

Proposed by Kapil Thangavelu
Status: Merged
Merge reported by: Kapil Thangavelu
Merged at revision: not available
Proposed branch: lp:~hazmat/charms/trusty/juju-gui/trunk
Merge into: lp:charms/trusty/juju-gui
Diff against target: 51 lines (+16/-1)
3 files modified
config/guiserver.conf.template (+12/-0)
hooks/utils.py (+3/-0)
server-requirements.pip (+1/-1)
To merge this branch: bzr merge lp:~hazmat/charms/trusty/juju-gui/trunk
Reviewer Review Type Date Requested Status
Francesco Banconi Approve
Review via email: mp+232639@code.launchpad.net

Description of the change

adds proxy environment variable support and updates embedded deployer to 0.3.9 (needed for some ssl proxies).

To post a comment you must log in.
Revision history for this message
Francesco Banconi (frankban) wrote :

Thanks for this fix Kapil!
The branch looks good (with a minor suggestion).
Please note that this merge proposal should be against lp:~juju-gui/charms/trusty/juju-gui/trunk:
that's our development branch. Periodically, the dev branch is then pushed to precise and trusty release branches.
Lbox can be used to propose and submit the charm.
Thank you!

review: Approve
Revision history for this message
Kapil Thangavelu (hazmat) wrote :

sounds good, i'll retarget to dev, also added inline comment re no_proxy.

95. By Kapil Thangavelu

fix upstart job, suggestion from review

Revision history for this message
Kapil Thangavelu (hazmat) wrote :

Added suggestion, also fixed an issue from field testing.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'config/guiserver.conf.template'
2--- config/guiserver.conf.template 2013-11-14 13:06:22 +0000
3+++ config/guiserver.conf.template 2014-08-29 15:19:56 +0000
4@@ -4,6 +4,18 @@
5 start on (filesystem and net-device-up IFACE=lo)
6 stop on runlevel [!2345]
7
8+{{if https_proxy}}
9+env https_proxy="{{https_proxy}}"
10+{{endif}}
11+
12+{{if http_proxy}}
13+env http_proxy="{{http_proxy}}"
14+{{endif}}
15+
16+{{if no_proxy}}
17+env no_proxy="{{no_proxy}}"
18+{{endif}}
19+
20 exec /usr/bin/python /usr/local/bin/runserver.py \
21 --logging="{{builtin_server_logging}}" \
22 --guiroot="{{gui_root}}" \
23
24=== removed file 'deps/juju-deployer-0.3.6.tar.gz'
25Binary files deps/juju-deployer-0.3.6.tar.gz 2014-04-10 22:41:09 +0000 and deps/juju-deployer-0.3.6.tar.gz 1970-01-01 00:00:00 +0000 differ
26=== added file 'deps/juju-deployer-0.3.9.tar.gz'
27Binary files deps/juju-deployer-0.3.9.tar.gz 1970-01-01 00:00:00 +0000 and deps/juju-deployer-0.3.9.tar.gz 2014-08-29 15:19:56 +0000 differ
28=== modified file 'hooks/utils.py'
29--- hooks/utils.py 2014-07-08 15:52:13 +0000
30+++ hooks/utils.py 2014-08-29 15:19:56 +0000
31@@ -524,6 +524,9 @@
32 'serve_tests': serve_tests,
33 'ssl_cert_path': ssl_cert_path,
34 'charmworld_url': charmworld_url,
35+ 'http_proxy': os.environ.get('http_proxy'),
36+ 'https_proxy': os.environ.get('https_proxy'),
37+ 'no_proxy': os.environ.get('no_proxy', os.environ.get('NO_PROXY'))
38 }
39 if not sandbox:
40 api_url = 'wss://{}'.format(get_api_address())
41
42=== modified file 'server-requirements.pip'
43--- server-requirements.pip 2014-04-10 22:41:09 +0000
44+++ server-requirements.pip 2014-08-29 15:19:56 +0000
45@@ -27,5 +27,5 @@
46 tornado==3.1.1
47 websocket-client==0.12.0
48 jujuclient==0.17.5
49-juju-deployer==0.3.6
50+juju-deployer==0.3.9
51 charmworldlib==0.3.0

Subscribers

People subscribed via source and target branches