Merge lp:~mthaddon/mojo/cassandra-charm-testing into lp:mojo/mojo-specs

Proposed by Tom Haddon
Status: Merged
Merged at revision: 42
Proposed branch: lp:~mthaddon/mojo/cassandra-charm-testing
Merge into: lp:mojo/mojo-specs
Diff against target: 25 lines (+15/-0)
2 files modified
charm-testing/cassandra/collect (+2/-0)
charm-testing/cassandra/deploy (+13/-0)
To merge this branch: bzr merge lp:~mthaddon/mojo/cassandra-charm-testing
Reviewer Review Type Date Requested Status
Stuart Bishop (community) Approve
Review via email: mp+322290@code.launchpad.net

Description of the change

Add basic charm testing for cassandra.

To post a comment you must log in.
Revision history for this message
Tom Haddon (mthaddon) wrote :

Hi Stuart,

I'm looking for a review here of whether this is a sufficient basic test of "does the Cassandra charm install correctly". Over time we can flesh it out to be exercising more complicated use cases.

Revision history for this message
Stuart Bishop (stub) wrote :

This is a basic test installing Apache Cassandra 3.0. It will test that the units deploy without error.

It does not test that it actually works, which takes more effort. For this and testing alternative versions, it is worth wrapping the existing amulet tests with enough mojo to go through the CI systems.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added directory 'charm-testing/cassandra'
2=== added file 'charm-testing/cassandra/collect'
3--- charm-testing/cassandra/collect 1970-01-01 00:00:00 +0000
4+++ charm-testing/cassandra/collect 2017-04-10 14:16:15 +0000
5@@ -0,0 +1,2 @@
6+cassandra cs:{{ series }}/cassandra
7+nrpe cs:{{ series }}/nrpe
8
9=== added file 'charm-testing/cassandra/deploy'
10--- charm-testing/cassandra/deploy 1970-01-01 00:00:00 +0000
11+++ charm-testing/cassandra/deploy 2017-04-10 14:16:15 +0000
12@@ -0,0 +1,13 @@
13+cassandra:
14+ series: {{series}}
15+ services:
16+ cassandra:
17+ charm: cassandra
18+ num_units: 3
19+ nrpe:
20+ charm: nrpe
21+ {% if series == "xenial" %}options:
22+ procs: '-w 200 -c 250'
23+ swap: '-w 90% -c 75% -n ok'{% endif %}
24+ relations:
25+ - ["cassandra", "nrpe:nrpe-external-master"]

Subscribers

People subscribed via source and target branches

to all changes: