Merge ~peter-sabaini/charm-nagios:robustify-install into ~nagios-charmers/charm-nagios:master

Proposed by Peter Sabaini
Status: Merged
Approved by: Peter Sabaini
Approved revision: 8568ba9581ff8281dc11e9eaefedf389a88b9970
Merged at revision: fa015564e63c77d88357e81d8be4d121297ec9a3
Proposed branch: ~peter-sabaini/charm-nagios:robustify-install
Merge into: ~nagios-charmers/charm-nagios:master
Diff against target: 27 lines (+3/-3)
2 files modified
hooks/install (+2/-2)
hooks/templates/localhost_nagios2.cfg.tmpl (+1/-1)
Reviewer Review Type Date Requested Status
Stuart Bishop (community) Approve
Canonical IS Reviewers Pending
Review via email: mp+364101@code.launchpad.net

Commit message

Make install more robust

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Stuart Bishop (stub) wrote :

LTGM

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision fa015564e63c77d88357e81d8be4d121297ec9a3

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/hooks/install b/hooks/install
index db3e40f..4d1ff11 100755
--- a/hooks/install
+++ b/hooks/install
@@ -35,9 +35,9 @@ if [ -f $CHARM_DIR/files/hostgroups_nagios2.cfg ]; then
35 # Write the new hostgroups_nagios2.cfg file to prevent servers being classified as Debian.35 # Write the new hostgroups_nagios2.cfg file to prevent servers being classified as Debian.
36 cp -v $CHARM_DIR/files/hostgroups_nagios2.cfg /etc/nagios3/conf.d/hostgroups_nagios2.cfg36 cp -v $CHARM_DIR/files/hostgroups_nagios2.cfg /etc/nagios3/conf.d/hostgroups_nagios2.cfg
37 # Remove the services configuration file to eliminiate the need for ssh and localhost groups.37 # Remove the services configuration file to eliminiate the need for ssh and localhost groups.
38 rm -v /etc/nagios3/conf.d/services_nagios2.cfg38 rm -vf /etc/nagios3/conf.d/services_nagios2.cfg
39 # Remove the ext file to eliminate the need for ssh and localhost groups.39 # Remove the ext file to eliminate the need for ssh and localhost groups.
40 rm -v /etc/nagios3/conf.d/extinfo_nagios2.cfg40 rm -vf /etc/nagios3/conf.d/extinfo_nagios2.cfg
41fi41fi
4242
43enable_livestatus=$(config-get enable_livestatus)43enable_livestatus=$(config-get enable_livestatus)
diff --git a/hooks/templates/localhost_nagios2.cfg.tmpl b/hooks/templates/localhost_nagios2.cfg.tmpl
index 85c756f..d70b915 100644
--- a/hooks/templates/localhost_nagios2.cfg.tmpl
+++ b/hooks/templates/localhost_nagios2.cfg.tmpl
@@ -53,7 +53,7 @@ define service{
53 use generic-service ; Name of service template to use53 use generic-service ; Name of service template to use
54 host_name {{ nagios_hostname }}54 host_name {{ nagios_hostname }}
55 service_description Total Processes55 service_description Total Processes
56 check_command check_procs!250!40056 check_command check_procs!600!900
57 }57 }
5858
5959

Subscribers

People subscribed via source and target branches