Merge lp:~brad-marshall/charm-helpers/charmhelpers-fix-nagios-servicegroups into lp:charm-helpers

Proposed by Brad Marshall
Status: Merged
Merged at revision: 313
Proposed branch: lp:~brad-marshall/charm-helpers/charmhelpers-fix-nagios-servicegroups
Merge into: lp:charm-helpers
Diff against target: 37 lines (+3/-6)
1 file modified
charmhelpers/contrib/charmsupport/nrpe.py (+3/-6)
To merge this branch: bzr merge lp:~brad-marshall/charm-helpers/charmhelpers-fix-nagios-servicegroups
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
Review via email: mp+250096@code.launchpad.net

Description of the change

Fix nrpe class to default to nagios_context if nagios_servicegroup is not set

To post a comment you must log in.
Revision history for this message
Liam Young (gnuoy) wrote :

Approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'charmhelpers/contrib/charmsupport/nrpe.py'
2--- charmhelpers/contrib/charmsupport/nrpe.py 2015-01-22 06:11:15 +0000
3+++ charmhelpers/contrib/charmsupport/nrpe.py 2015-02-18 07:53:08 +0000
4@@ -161,7 +161,7 @@
5 log('Check command not found: {}'.format(parts[0]))
6 return ''
7
8- def write(self, nagios_context, hostname, nagios_servicegroups=None):
9+ def write(self, nagios_context, hostname, nagios_servicegroups):
10 nrpe_check_file = '/etc/nagios/nrpe.d/{}.cfg'.format(
11 self.command)
12 with open(nrpe_check_file, 'w') as nrpe_check_config:
13@@ -177,14 +177,11 @@
14 nagios_servicegroups)
15
16 def write_service_config(self, nagios_context, hostname,
17- nagios_servicegroups=None):
18+ nagios_servicegroups):
19 for f in os.listdir(NRPE.nagios_exportdir):
20 if re.search('.*{}.cfg'.format(self.command), f):
21 os.remove(os.path.join(NRPE.nagios_exportdir, f))
22
23- if not nagios_servicegroups:
24- nagios_servicegroups = nagios_context
25-
26 templ_vars = {
27 'nagios_hostname': hostname,
28 'nagios_servicegroup': nagios_servicegroups,
29@@ -214,7 +211,7 @@
30 if 'nagios_servicegroups' in self.config:
31 self.nagios_servicegroups = self.config['nagios_servicegroups']
32 else:
33- self.nagios_servicegroups = 'juju'
34+ self.nagios_servicegroups = self.nagios_context
35 self.unit_name = local_unit().replace('/', '-')
36 if hostname:
37 self.hostname = hostname

Subscribers

People subscribed via source and target branches