Merge lp:~hopem/charms/trusty/ceilometer/fix-apachessl-port-selection into lp:~openstack-charmers-archive/charms/trusty/ceilometer/next

Proposed by Edward Hope-Morley
Status: Merged
Merged at revision: 63
Proposed branch: lp:~hopem/charms/trusty/ceilometer/fix-apachessl-port-selection
Merge into: lp:~openstack-charmers-archive/charms/trusty/ceilometer/next
Diff against target: 100 lines (+16/-29)
4 files modified
hooks/charmhelpers/contrib/openstack/context.py (+4/-2)
hooks/charmhelpers/contrib/openstack/neutron.py (+8/-2)
hooks/charmhelpers/contrib/openstack/templates/haproxy.cfg (+4/-2)
templates/openstack_https_frontend (+0/-23)
To merge this branch: bzr merge lp:~hopem/charms/trusty/ceilometer/fix-apachessl-port-selection
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
Review via email: mp+246129@code.launchpad.net
To post a comment you must log in.
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_lint_check #654 ceilometer-next for hopem mp246129
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/654/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_unit_test #683 ceilometer-next for hopem mp246129
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/683/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_amulet_test #839 ceilometer-next for hopem mp246129
    AMULET FAIL: amulet-test missing

AMULET Results (max last 2 lines):
INFO:root:Search string not found in makefile target commands.
ERROR:root:No make target was executed.

Full amulet test output: http://paste.ubuntu.com/9718210/
Build: http://10.245.162.77:8080/job/charm_amulet_test/839/

Revision history for this message
Liam Young (gnuoy) wrote :

Approve

review: Approve
64. By Edward Hope-Morley

removed stale apache2 templates

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_lint_check #659 ceilometer-next for hopem mp246129
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/659/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_unit_test #688 ceilometer-next for hopem mp246129
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/688/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_amulet_test #844 ceilometer-next for hopem mp246129
    AMULET FAIL: amulet-test missing

AMULET Results (max last 2 lines):
INFO:root:Search string not found in makefile target commands.
ERROR:root:No make target was executed.

Full amulet test output: http://paste.ubuntu.com/9718343/
Build: http://10.245.162.77:8080/job/charm_amulet_test/844/

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/charmhelpers/contrib/openstack/context.py'
2--- hooks/charmhelpers/contrib/openstack/context.py 2014-12-17 18:42:00 +0000
3+++ hooks/charmhelpers/contrib/openstack/context.py 2015-01-12 13:31:54 +0000
4@@ -491,6 +491,7 @@
5 ctxt['haproxy_client_timeout'] = config('haproxy-client-timeout')
6
7 if config('prefer-ipv6'):
8+ ctxt['ipv6'] = True
9 ctxt['local_host'] = 'ip6-localhost'
10 ctxt['haproxy_host'] = '::'
11 ctxt['stat_port'] = ':::8888'
12@@ -662,8 +663,9 @@
13 addresses = self.get_network_addresses()
14 for address, endpoint in sorted(set(addresses)):
15 for api_port in self.external_ports:
16- ext_port = determine_apache_port(api_port)
17- int_port = determine_api_port(api_port)
18+ ext_port = determine_apache_port(api_port,
19+ singlenode_mode=True)
20+ int_port = determine_api_port(api_port, singlenode_mode=True)
21 portmap = (address, endpoint, int(ext_port), int(int_port))
22 ctxt['endpoints'].append(portmap)
23 ctxt['ext_ports'].append(int(ext_port))
24
25=== modified file 'hooks/charmhelpers/contrib/openstack/neutron.py'
26--- hooks/charmhelpers/contrib/openstack/neutron.py 2014-12-17 18:42:00 +0000
27+++ hooks/charmhelpers/contrib/openstack/neutron.py 2015-01-12 13:31:54 +0000
28@@ -152,9 +152,15 @@
29 database=config('neutron-database'),
30 relation_prefix='neutron',
31 ssl_dir=NEUTRON_CONF_DIR)],
32- 'services': ['calico-compute', 'bird', 'neutron-dhcp-agent'],
33+ 'services': ['calico-felix',
34+ 'bird',
35+ 'neutron-dhcp-agent',
36+ 'nova-api-metadata'],
37 'packages': [[headers_package()] + determine_dkms_package(),
38- ['calico-compute', 'bird', 'neutron-dhcp-agent']],
39+ ['calico-compute',
40+ 'bird',
41+ 'neutron-dhcp-agent',
42+ 'nova-api-metadata']],
43 'server_packages': ['neutron-server', 'calico-control'],
44 'server_services': ['neutron-server']
45 }
46
47=== modified file 'hooks/charmhelpers/contrib/openstack/templates/haproxy.cfg'
48--- hooks/charmhelpers/contrib/openstack/templates/haproxy.cfg 2014-10-21 20:56:39 +0000
49+++ hooks/charmhelpers/contrib/openstack/templates/haproxy.cfg 2015-01-12 13:31:54 +0000
50@@ -35,10 +35,12 @@
51 stats auth admin:password
52
53 {% if frontends -%}
54-{% for service, ports in service_ports.iteritems() -%}
55+{% for service, ports in service_ports.items() -%}
56 frontend tcp-in_{{ service }}
57 bind *:{{ ports[0] }}
58+ {% if ipv6 -%}
59 bind :::{{ ports[0] }}
60+ {% endif -%}
61 {% for frontend in frontends -%}
62 acl net_{{ frontend }} dst {{ frontends[frontend]['network'] }}
63 use_backend {{ service }}_{{ frontend }} if net_{{ frontend }}
64@@ -46,7 +48,7 @@
65 {% for frontend in frontends -%}
66 backend {{ service }}_{{ frontend }}
67 balance leastconn
68- {% for unit, address in frontends[frontend]['backends'].iteritems() -%}
69+ {% for unit, address in frontends[frontend]['backends'].items() -%}
70 server {{ unit }} {{ address }}:{{ ports[1] }} check
71 {% endfor %}
72 {% endfor -%}
73
74=== removed file 'templates/openstack_https_frontend'
75--- templates/openstack_https_frontend 2014-03-05 13:53:56 +0000
76+++ templates/openstack_https_frontend 1970-01-01 00:00:00 +0000
77@@ -1,23 +0,0 @@
78-{% if endpoints %}
79-{% for ext, int in endpoints %}
80-Listen {{ ext }}
81-NameVirtualHost *:{{ ext }}
82-<VirtualHost *:{{ ext }}>
83- ServerName {{ private_address }}
84- SSLEngine on
85- SSLCertificateFile /etc/apache2/ssl/{{ namespace }}/cert
86- SSLCertificateKeyFile /etc/apache2/ssl/{{ namespace }}/key
87- ProxyPass / http://localhost:{{ int }}/
88- ProxyPassReverse / http://localhost:{{ int }}/
89- ProxyPreserveHost on
90-</VirtualHost>
91-<Proxy *>
92- Order deny,allow
93- Allow from all
94-</Proxy>
95-<Location />
96- Order allow,deny
97- Allow from all
98-</Location>
99-{% endfor %}
100-{% endif %}

Subscribers

People subscribed via source and target branches