The docs are pretty clear, this is an option for the main config file only, and
given the age of Nagios 3 it's unlikely to be something that'll change any time
soon.
Since we're in the middle of a release, I'll postpone this till after the
release and target for the next one.
--
Thanks for the contribution.
I'm concerned that we have the code to enable the module, but nothing to
actually deploy the module itself, meaning that this change alone doesn't
achieve much. Adding the module would be ideally put into a subordinate charm,
along with dropping a fresh config
The proposal to merge ~giulio.cervera/nagios-charm:bug/1860669 into
nagios-charm:master has been updated.
Commit message changed to:
Add support for enabling broker module
For more details, see:
https://code.launchpad.net/~giulio.cervera/nagios-charm/+git/nagios-charm/+merge/378449
--
Your
Giulio Cervera has proposed merging ~giulio.cervera/nagios-charm:bug/1860669
into nagios-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~giulio.cervera/nagios-charm/+git/nagios-charm/+merge/378449
--
Your team
This merge proposal is being monitored by mergebot. Change the status to
Approved to merge.
--
https://code.launchpad.net/~giulio.cervera/nagios-charm/+git/nagios-charm/+merge/378449
Your team Nagios Charm developers is requested to review the proposed merge of
5 matches
Mail list logo
