Code review comment for lp:~paulgear/charms/trusty/memcached/allow-rsync-firewall

Revision history for this message
Adam Israel (aisrael) wrote :

Hi Brad,

There are two other related merge proposals from Paul that I tested, along with this one. I wasn't able to get any of the nrpe functionality to work. There's nothing in the README about its usage, nor any unit tests to verify functionality across substrates. At the very least, I'd like to see some instructions on using nrpe with the memcached charm, so that I can fully test that functionality.

« Back to merge proposal