nagios-charmers team mailing list archive

[Edward Hope-Morley <edward.hope-morley@xxxxxxxxxxxxx>]

Diff comments:

> diff --git a/hooks/upgrade-charm b/hooks/upgrade-charm
> index 19e736c..6afea93 100755
> --- a/hooks/upgrade-charm
> +++ b/hooks/upgrade-charm
> @@ -161,8 +161,13 @@ def enable_pagerduty_config():
>              os.remove(pagerduty_cron)
>  
>      # Update contacts for admin
> +    contactgroup_members = hookenv.config("contactgroup-members")
> +    if enable_pagerduty:
> +        contactgroup_members += ", pagerduty" 

fair point, will do

> +
>      template_values = {'enable_pagerduty': enable_pagerduty,
> -                       'admin_email': hookenv.config('admin_email')}
> +                       'admin_email': hookenv.config('admin_email'),
> +                       'contactgroup_members': contactgroup_members}
>  
>      with open('hooks/templates/contacts-cfg.tmpl', 'r') as f:
>          templateDef = f.read()


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