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.

-- 
Mailing list: https://launchpad.net/~nagios-charmers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~nagios-charmers
More help   : https://help.launchpad.net/ListHelp

Reply via email to