Merge lp:~stub/charms/trusty/cassandra/rsync-port into lp:charms/trusty/cassandra
Proposed by
Stuart Bishop
Status: | Merged |
---|---|
Merged at revision: | 369 |
Proposed branch: | lp:~stub/charms/trusty/cassandra/rsync-port |
Merge into: | lp:charms/trusty/cassandra |
Diff against target: |
60 lines (+8/-5) 3 files modified
hooks/actions.py (+1/-0) tests/test_actions.py (+1/-0) tests/test_integration.py (+6/-5) |
To merge this branch: | bzr merge lp:~stub/charms/trusty/cassandra/rsync-port |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Konstantinos Tsakalozos (community) | Approve | ||
Review Queue (community) | automated testing | Approve | |
charmers | Pending | ||
Review via email: mp+278671@code.launchpad.net |
Description of the change
Open access to the rsync port.
This does nothing unless a subordinate is installing and configuring rsyncd, at which point it is a requirement. Since charms using ufw are in the minority, it seems better to open the point here in the Cassandra charm than require subordinates to somehow detect if ufw has been enabled and react appropriately.
(The nrpe port is already open for the same reasons)
To post a comment you must log in.
This item has failed automated testing! Results available here http:// juju-ci. vapour. ws:8080/ job/charm- bundle- test-lxc/ 1562/