Merge lp:~celebdor/charms/trusty/midonet-api/cassandra into lp:charms/trusty/midonet-api

Proposed by Antoni Segura Puimedon
Status: Merged
Merged at revision: 43
Proposed branch: lp:~celebdor/charms/trusty/midonet-api/cassandra
Merge into: lp:charms/trusty/midonet-api
Diff against target: 20 lines (+7/-2)
1 file modified
tests/basic_deployment.py (+7/-2)
To merge this branch: bzr merge lp:~celebdor/charms/trusty/midonet-api/cassandra
Reviewer Review Type Date Requested Status
Cory Johns (community) Approve
Review via email: mp+287334@code.launchpad.net
To post a comment you must log in.
44. By Antoni Segura Puimedon <email address hidden>

fix lack of null keys so that extra keys and extra sources match

45. By Antoni Segura Puimedon <email address hidden>

amulet: add missing bracket

Revision history for this message
Cory Johns (johnsca) wrote :

Thanks for this. It has been merged with the small addition of quotes around the strings in the list.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tests/basic_deployment.py'
--- tests/basic_deployment.py 2016-02-16 14:38:52 +0000
+++ tests/basic_deployment.py 2016-02-26 19:53:21 +0000
@@ -98,9 +98,14 @@
98 self.d.configure('cassandra', {98 self.d.configure('cassandra', {
99 'authenticator': 'AllowAllAuthenticator',99 'authenticator': 'AllowAllAuthenticator',
100 'cluster_name': 'midonet',100 'cluster_name': 'midonet',
101 'install_keys': '7E41C00F85BFC1706C4FFFB3350200F2B999A372',101 'install_keys':
102 '[null, '
103 'null, '
104 '7E41C00F85BFC1706C4FFFB3350200F2B999A372]',
102 'install_sources':105 'install_sources':
103 'deb http://debian.datastax.com/community 2.0 main',106 '[ppa:openjdk-r/ppa, '
107 'ppa:stub/cassandra, '
108 'deb http://debian.datastax.com/community 2.0 main]',
104 'extra_packages': 'dsc20'})109 'extra_packages': 'dsc20'})
105110
106 self.d.configure('mysql', {'max-connections': 500})111 self.d.configure('mysql', {'max-connections': 500})

Subscribers

People subscribed via source and target branches