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

