← Back to team overview

nagios-charmers team mailing list archive

Re: [Merge] ~jhebden/nagios-charm:readonly-account into nagios-charm:master

 

Review: Needs Information

James, is nagios-cgi.tmpl directly from the nagios package? If so, should we instead parse and update that instead of shipping our own copy of it?

Diff comments:

> diff --git a/config.yaml b/config.yaml
> index 3eb834a..7ae04fb 100644
> --- a/config.yaml
> +++ b/config.yaml
> @@ -144,3 +144,8 @@ options:
>              Password to use for Nagios administrative access.  If not
>              provided, a password will be generated (see documentation for
>              instructions on retrieving the generated password.)
> +    ro-password:
> +        type: string
> +        default: ''
> +        description: |
> +            Password to use for read-only Nagios access.  If left blank, the nagiosro account will not be created.

Maybe just make the default 'nagiosro'?

Create no RO usre if it's blank?



-- 
https://code.launchpad.net/~jhebden/nagios-charm/+git/nagios-charm/+merge/318564
Your team Nagios Charm developers is subscribed to branch nagios-charm:master.


Follow ups

References