Juju Charms Collection
midonet-api package

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

  1. Trusty Tahr (14.04)
  2. cassandra
  3. Merge into trunk
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.
lp:~celebdor/charms/trusty/midonet-api/cassandra updated
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
1=== modified file 'tests/basic_deployment.py'
2--- tests/basic_deployment.py 2016-02-16 14:38:52 +0000
3+++ tests/basic_deployment.py 2016-02-26 19:53:21 +0000
4@@ -98,9 +98,14 @@
5 self.d.configure('cassandra', {
6 'authenticator': 'AllowAllAuthenticator',
7 'cluster_name': 'midonet',
8- 'install_keys': '7E41C00F85BFC1706C4FFFB3350200F2B999A372',
9+ 'install_keys':
10+ '[null, '
11+ 'null, '
12+ '7E41C00F85BFC1706C4FFFB3350200F2B999A372]',
13 'install_sources':
14- 'deb http://debian.datastax.com/community 2.0 main',
15+ '[ppa:openjdk-r/ppa, '
16+ 'ppa:stub/cassandra, '
17+ 'deb http://debian.datastax.com/community 2.0 main]',
18 'extra_packages': 'dsc20'})
19
20 self.d.configure('mysql', {'max-connections': 500})

Subscribers

People subscribed via source and target branches