nagios-charmers team mailing list archive

[Aggelos Kolaitis <akolaitis@xxxxxxxxxxxxxx>]

Thanks for the initial review, the code should be OK now.

Diff comments:

> diff --git a/config.yaml b/config.yaml
> index 4ce0f18..03c270e 100644
> --- a/config.yaml
> +++ b/config.yaml
> @@ -105,7 +105,7 @@ options:
>          type: string
>          default: pageroot@localhost
>          description: |

Changed as suggested

> -            Email address used for the admin pager, used by $ADMINPAGER$ in 
> +            Email address used for the admin pager, used by $ADMINPAGER$ in
>              notification commands.
>      enable_pagerduty:
>          type: boolean
> @@ -168,7 +173,7 @@ options:
>          type: string
>          description: |

Changed as suggested

>              A string to pass to the Nagios load monitoring command.  Default is
> -            to report warning at 5.0, 4.0 and 3.0 averages, critical at 10.0, 
> +            to report warning at 5.0, 4.0 and 3.0 averages, critical at 10.0,
>              6.0 and 4.0.
>      pagerduty_notification_levels:
>          default: u,c,r
> diff --git a/hooks/upgrade-charm b/hooks/upgrade-charm
> index 19e736c..e154b85 100755
> --- a/hooks/upgrade-charm
> +++ b/hooks/upgrade-charm
> @@ -284,7 +284,8 @@ def update_config():
>  
>  
>  def update_cgi_config():
> -    template_values = {'ro_password': ro_password}
> +    template_values = {'nagiosadmin': hookenv.config('nagiosadmin'),

Changed so that the configuration value falls back to 'nagiosadmin' when empty.

> +                       'ro_password': ro_password}
>      with open('hooks/templates/nagios-cgi.tmpl', 'r') as f:
>          templateDef = f.read()
>  


-- 
https://code.launchpad.net/~aggkolaitis/nagios-charm/+git/nagios-charm/+merge/374089
Your team Nagios Charm developers is requested to review the proposed merge of ~aggkolaitis/nagios-charm:change_nagios_gui_username into nagios-charm:master.