Juju Charms Collection
nrpe package

Code review comment for lp:~dbuliga/charms/trusty/nrpe/nrpe

  1. Trusty Tahr (14.04)
  2. nrpe
  3. Merge into trunk
Revision history for this message
Antonio Rosales (arosales) wrote :

@Denis,

Thanks for your contribution.

Given this is a CentoOS update this may actually be better as a separate CentOS charm. Specifically, the metadata stating it is a CentOS charm so Juju can deploy it as such and it can be discoverable in the Charm Store.

It looks like there is some feedback on the tests, but it would be great to see this in a stand-alone CentOS charm.

-thanks,
Antonio

Reply

« Back to merge proposal