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

Proposed by Antoni Segura Puimedon
Status: Merged
Merged at revision: 35
Proposed branch: lp:~celebdor/charms/trusty/midonet-agent/cassandra
Merge into: lp:charms/trusty/midonet-agent
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-agent/cassandra
Reviewer Review Type Date Requested Status
Cory Johns (community) Approve
Review via email: mp+287338@code.launchpad.net
To post a comment you must log in.
36. By Antoni Segura Puimedon <email address hidden>

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

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:28 +0000
+++ tests/basic_deployment.py 2016-02-26 19:55:23 +0000
@@ -100,9 +100,14 @@
100 self.d.configure('cassandra', {100 self.d.configure('cassandra', {
101 'authenticator': 'AllowAllAuthenticator',101 'authenticator': 'AllowAllAuthenticator',
102 'cluster_name': 'midonet',102 'cluster_name': 'midonet',
103 'install_keys': '7E41C00F85BFC1706C4FFFB3350200F2B999A372',103 'install_keys':
104 '[null, '
105 'null, '
106 '7E41C00F85BFC1706C4FFFB3350200F2B999A372]',
104 'install_sources':107 'install_sources':
105 'deb http://debian.datastax.com/community 2.0 main',108 '[ppa:openjdk-r/ppa, '
109 'ppa:stub/cassandra, '
110 'deb http://debian.datastax.com/community 2.0 main]',
106 'extra_packages': 'dsc20'})111 'extra_packages': 'dsc20'})
107112
108 self.d.configure('mysql', {'max-connections': 500})113 self.d.configure('mysql', {'max-connections': 500})

Subscribers

People subscribed via source and target branches