Merge lp:~gnuoy/charms/trusty/keystone/next-haproxy-always into lp:~openstack-charmers-archive/charms/trusty/keystone/next
- Trusty Tahr (14.04)
- next-haproxy-always
- Merge into next
Status: | Merged |
---|---|
Merged at revision: | 99 |
Proposed branch: | lp:~gnuoy/charms/trusty/keystone/next-haproxy-always |
Merge into: | lp:~openstack-charmers-archive/charms/trusty/keystone/next |
Diff against target: |
85 lines (+18/-11) 3 files modified
hooks/keystone_context.py (+8/-4) hooks/keystone_utils.py (+8/-5) tests/basic_deployment.py (+2/-2) |
To merge this branch: | bzr merge lp:~gnuoy/charms/trusty/keystone/next-haproxy-always |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
James Page | Approve | ||
Review via email: mp+242900@code.launchpad.net |
Commit message
Description of the change
uosci-testing-bot (uosci-testing-bot) wrote : | # |
uosci-testing-bot (uosci-testing-bot) wrote : | # |
UOSCI bot says:
charm_unit_test #1065 keystone-next for gnuoy mp242900
UNIT OK: passed
UNIT Results (max last 5 lines):
hooks/
hooks/
TOTAL 797 390 51%
Ran 38 tests in 3.278s
OK
Full unit test output: http://
Build: http://
uosci-testing-bot (uosci-testing-bot) wrote : | # |
UOSCI bot says:
charm_amulet_test #534 keystone-next for gnuoy mp242900
AMULET FAIL: amulet-test failed
AMULET Results (max last 5 lines):
juju-
WARNING cannot delete security group "juju-osci-sv05-0". Used by another environment?
juju-test INFO : Results: 1 passed, 2 failed, 0 errored
ERROR subprocess encountered error code 2
make: *** [test] Error 2
Full amulet test output: http://
Build: http://
James Page (james-page) wrote : | # |
Needs some amulet test case fixes as well please
- 92. By Liam Young
-
Merged next in
- 93. By Liam Young
-
Sync charmhelpers
uosci-testing-bot (uosci-testing-bot) wrote : | # |
charm_lint_check #307 keystone-next for gnuoy mp242900
LINT OK: passed
uosci-testing-bot (uosci-testing-bot) wrote : | # |
charm_unit_test #335 keystone-next for gnuoy mp242900
UNIT OK: passed
uosci-testing-bot (uosci-testing-bot) wrote : | # |
charm_amulet_test #470 keystone-next for gnuoy mp242900
AMULET FAIL: amulet-test failed
AMULET Results (max last 2 lines):
ERROR subprocess encountered error code 1
make: *** [test] Error 1
Full amulet test output: http://
Build: http://
- 94. By Liam Young
-
charmhelper sync
uosci-testing-bot (uosci-testing-bot) wrote : | # |
charm_lint_check #525 keystone-next for gnuoy mp242900
LINT OK: passed
uosci-testing-bot (uosci-testing-bot) wrote : | # |
charm_unit_test #554 keystone-next for gnuoy mp242900
UNIT OK: passed
uosci-testing-bot (uosci-testing-bot) wrote : | # |
charm_amulet_test #681 keystone-next for gnuoy mp242900
AMULET FAIL: amulet-test failed
AMULET Results (max last 2 lines):
ERROR subprocess encountered error code 1
make: *** [test] Error 1
Full amulet test output: http://
Build: http://
- 95. By Liam Young
-
Fix amulet tests broken by my last fix
uosci-testing-bot (uosci-testing-bot) wrote : | # |
charm_lint_check #537 keystone-next for gnuoy mp242900
LINT OK: passed
uosci-testing-bot (uosci-testing-bot) wrote : | # |
charm_unit_test #566 keystone-next for gnuoy mp242900
UNIT OK: passed
uosci-testing-bot (uosci-testing-bot) wrote : | # |
charm_amulet_test #693 keystone-next for gnuoy mp242900
AMULET OK: passed
James Page (james-page) : | # |
Preview Diff
1 | === modified file 'hooks/keystone_context.py' | |||
2 | --- hooks/keystone_context.py 2014-11-14 02:16:38 +0000 | |||
3 | +++ hooks/keystone_context.py 2015-01-05 16:01:10 +0000 | |||
4 | @@ -72,8 +72,10 @@ | |||
5 | 72 | listen_ports['public_port'] = api_port('keystone-public') | 72 | listen_ports['public_port'] = api_port('keystone-public') |
6 | 73 | 73 | ||
7 | 74 | # Apache ports | 74 | # Apache ports |
10 | 75 | a_admin_port = determine_apache_port(api_port('keystone-admin')) | 75 | a_admin_port = determine_apache_port(api_port('keystone-admin'), |
11 | 76 | a_public_port = determine_apache_port(api_port('keystone-public')) | 76 | singlenode_mode=True) |
12 | 77 | a_public_port = determine_apache_port(api_port('keystone-public'), | ||
13 | 78 | singlenode_mode=True) | ||
14 | 77 | 79 | ||
15 | 78 | port_mapping = { | 80 | port_mapping = { |
16 | 79 | 'admin-port': [ | 81 | 'admin-port': [ |
17 | @@ -100,8 +102,10 @@ | |||
18 | 100 | ) | 102 | ) |
19 | 101 | ctxt = {} | 103 | ctxt = {} |
20 | 102 | ctxt['token'] = set_admin_token(config('admin-token')) | 104 | ctxt['token'] = set_admin_token(config('admin-token')) |
23 | 103 | ctxt['admin_port'] = determine_api_port(api_port('keystone-admin')) | 105 | ctxt['admin_port'] = determine_api_port(api_port('keystone-admin'), |
24 | 104 | ctxt['public_port'] = determine_api_port(api_port('keystone-public')) | 106 | singlenode_mode=True) |
25 | 107 | ctxt['public_port'] = determine_api_port(api_port('keystone-public'), | ||
26 | 108 | singlenode_mode=True) | ||
27 | 105 | ctxt['debug'] = config('debug') in ['yes', 'true', 'True'] | 109 | ctxt['debug'] = config('debug') in ['yes', 'true', 'True'] |
28 | 106 | ctxt['verbose'] = config('verbose') in ['yes', 'true', 'True'] | 110 | ctxt['verbose'] = config('verbose') in ['yes', 'true', 'True'] |
29 | 107 | ctxt['identity_backend'] = config('identity-backend') | 111 | ctxt['identity_backend'] = config('identity-backend') |
30 | 108 | 112 | ||
31 | === modified file 'hooks/keystone_utils.py' | |||
32 | --- hooks/keystone_utils.py 2014-12-17 17:54:26 +0000 | |||
33 | +++ hooks/keystone_utils.py 2015-01-05 16:01:10 +0000 | |||
34 | @@ -125,7 +125,7 @@ | |||
35 | 125 | context.WorkerConfigContext()], | 125 | context.WorkerConfigContext()], |
36 | 126 | }), | 126 | }), |
37 | 127 | (HAPROXY_CONF, { | 127 | (HAPROXY_CONF, { |
39 | 128 | 'contexts': [context.HAProxyContext(), | 128 | 'contexts': [context.HAProxyContext(singlenode_mode=True), |
40 | 129 | keystone_context.HAProxyContext()], | 129 | keystone_context.HAProxyContext()], |
41 | 130 | 'services': ['haproxy'], | 130 | 'services': ['haproxy'], |
42 | 131 | }), | 131 | }), |
43 | @@ -247,9 +247,10 @@ | |||
44 | 247 | def save_script_rc(): | 247 | def save_script_rc(): |
45 | 248 | env_vars = {'OPENSTACK_SERVICE_KEYSTONE': 'keystone', | 248 | env_vars = {'OPENSTACK_SERVICE_KEYSTONE': 'keystone', |
46 | 249 | 'OPENSTACK_PORT_ADMIN': determine_api_port( | 249 | 'OPENSTACK_PORT_ADMIN': determine_api_port( |
48 | 250 | api_port('keystone-admin')), | 250 | api_port('keystone-admin'), singlenode_mode=True), |
49 | 251 | 'OPENSTACK_PORT_PUBLIC': determine_api_port( | 251 | 'OPENSTACK_PORT_PUBLIC': determine_api_port( |
51 | 252 | api_port('keystone-public'))} | 252 | api_port('keystone-public'), |
52 | 253 | singlenode_mode=True)} | ||
53 | 253 | _save_script_rc(**env_vars) | 254 | _save_script_rc(**env_vars) |
54 | 254 | 255 | ||
55 | 255 | 256 | ||
56 | @@ -297,10 +298,12 @@ | |||
57 | 297 | ipv6_addr = get_ipv6_addr(exc_list=[config('vip')])[0] | 298 | ipv6_addr = get_ipv6_addr(exc_list=[config('vip')])[0] |
58 | 298 | endpoint_url = 'http://[%s]:{}/v2.0/' % ipv6_addr | 299 | endpoint_url = 'http://[%s]:{}/v2.0/' % ipv6_addr |
59 | 299 | local_endpoint = endpoint_url.format( | 300 | local_endpoint = endpoint_url.format( |
61 | 300 | determine_api_port(api_port('keystone-admin'))) | 301 | determine_api_port(api_port('keystone-admin'), |
62 | 302 | singlenode_mode=True)) | ||
63 | 301 | else: | 303 | else: |
64 | 302 | local_endpoint = 'http://localhost:{}/v2.0/'.format( | 304 | local_endpoint = 'http://localhost:{}/v2.0/'.format( |
66 | 303 | determine_api_port(api_port('keystone-admin'))) | 305 | determine_api_port(api_port('keystone-admin'), |
67 | 306 | singlenode_mode=True)) | ||
68 | 304 | 307 | ||
69 | 305 | return local_endpoint | 308 | return local_endpoint |
70 | 306 | 309 | ||
71 | 307 | 310 | ||
72 | === modified file 'tests/basic_deployment.py' | |||
73 | --- tests/basic_deployment.py 2015-01-05 14:44:10 +0000 | |||
74 | +++ tests/basic_deployment.py 2015-01-05 16:01:10 +0000 | |||
75 | @@ -304,8 +304,8 @@ | |||
76 | 304 | conf = '/etc/keystone/keystone.conf' | 304 | conf = '/etc/keystone/keystone.conf' |
77 | 305 | relation = unit.relation('identity-service', 'cinder:identity-service') | 305 | relation = unit.relation('identity-service', 'cinder:identity-service') |
78 | 306 | expected = {'admin_token': relation['admin_token'], | 306 | expected = {'admin_token': relation['admin_token'], |
81 | 307 | 'admin_port': relation['auth_port'], | 307 | 'admin_port': '35347', |
82 | 308 | 'public_port': relation['service_port'], | 308 | 'public_port': '4990', |
83 | 309 | 'use_syslog': 'False', | 309 | 'use_syslog': 'False', |
84 | 310 | 'log_config': '/etc/keystone/logging.conf', | 310 | 'log_config': '/etc/keystone/logging.conf', |
85 | 311 | 'debug': 'False', | 311 | 'debug': 'False', |
UOSCI bot says:
charm_lint_check #1231 keystone-next for gnuoy mp242900
LINT OK: passed
LINT Results (max last 5 lines):
I: config.yaml: option ssl_ca has no default value
I: config.yaml: option os-public-network has no default value
I: config.yaml: option os-admin-network has no default value
I: config.yaml: option ssl_cert has no default value
I: config.yaml: option os-internal-network has no default value
Full lint test output: http:// paste.ubuntu. com/9252840/ 10.98.191. 181:8080/ job/charm_ lint_check/ 1231/
Build: http://