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

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

Hi Brad,

The problem I see is by promoting it to the charm store, we're expecting users to know that they need to manually fork the precise version of the nrpe-external-master charm to trusty and deploy from local. It's not an intuitive process.

If you could expand the README to demystify the process, I'd be happy to approve this in the interim.

« Back to merge proposal