Merge lp:~chris-gondolin/charm-helpers/add-nagios-servicegroups-support into lp:charm-helpers
Proposed by
Chris Stratford
Status: | Superseded |
---|---|
Proposed branch: | lp:~chris-gondolin/charm-helpers/add-nagios-servicegroups-support |
Merge into: | lp:charm-helpers |
Diff against target: |
142 lines (+28/-12) 2 files modified
charmhelpers/contrib/charmsupport/nrpe.py (+15/-5) tests/contrib/charmsupport/test_nrpe.py (+13/-7) |
To merge this branch: | bzr merge lp:~chris-gondolin/charm-helpers/add-nagios-servicegroups-support |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Chris Glass (community) | Needs Fixing | ||
charmers | Pending | ||
Review via email:
|
This proposal has been superseded by a proposal from 2014-08-29.
Description of the change
Adds nagios_
To post a comment you must log in.
This branch breaks unit tests unfortunately:
$ make test charmsupport. test_nrpe. NRPECheckTestCa se.test_ check_write_ nrpe_exportdir_ not_accessible ------- ------- ------- ------- ------- ------- ------- ------- ------- tribaal/ workspace/ charm-helpers/ trunk/tests/ contrib/ charmsupport/ test_nrpe. py", line 193, in test_check_ write_nrpe_ exportdir_ not_accessible assertEqual( None, check.write( 'testctx' , 'hostname'))
[...]
ERROR: tests.contrib.
-------
_StringException: Traceback (most recent call last):
File "/home/
self.
TypeError: write() takes exactly 4 arguments (3 given)
======= ======= ======= ======= ======= ======= ======= ======= ======= ======= charmsupport. test_nrpe. NRPECheckTestCa se.test_ write_removes_ existing_ config ------- ------- ------- ------- ------- ------- ------- ------- ------- tribaal/ workspace/ charm-helpers/ trunk/tests/ contrib/ charmsupport/ test_nrpe. py", line 183, in test_write_ removes_ existing_ config assertEqual( None, check.write( 'testctx' , 'hostname'))
ERROR: tests.contrib.
-------
_StringException: Traceback (most recent call last):
File "/home/
self.
TypeError: write() takes exactly 4 arguments (3 given)
======= ======= ======= ======= ======= ======= ======= ======= ======= ======= charmsupport. test_nrpe. NRPETestCase. test_init_ gets_config ------- ------- ------- ------- ------- ------- ------- ------- ------- tribaal/ workspace/ charm-helpers/ trunk/tests/ contrib/ charmsupport/ test_nrpe. py", line 55, in test_init_ gets_config tribaal/ workspace/ charm-helpers/ trunk/charmhelp ers/contrib/ charmsupport/ nrpe.py" , line 195, in __init__ nagios_ servicegroups = self.config[ 'nagios_ servicegroups' ] servicegroups'
ERROR: tests.contrib.
-------
_StringException: Traceback (most recent call last):
File "/home/
checker = nrpe.NRPE()
File "/home/
self.
KeyError: 'nagios_
======= ======= ======= ======= ======= ======= ======= ======= ======= ======= charmsupport. test_nrpe. NRPETestCase. test_no_ nagios_ installed_ bails ------- ------- ------- ------- ------- ------- ------- ------- ------- tribaal/ workspace/ charm-helpers/ trunk/tests/ contrib/ charmsupport/ test_nrpe. py", line 75, in test_no_ nagios_ installed_ bails tribaal/ workspace/ charm-helpers/ trunk/charmhelp ers/contrib/ charmsupport/ nrpe.py" , line 195, in __init__ nagios_ servicegroups = self.config[ 'nagios_ servicegroups' ] servicegroups'
ERROR: tests.contrib.
-------
_StringException: Traceback (most recent call last):
File "/home/
checker = nrpe.NRPE()
File "/home/
self.
KeyError: 'nagios_
======= ======= ======= ======= ======= ======= ======= ======= ======= ======= charmsupport. test_nrpe. NRPETestCase. test_update_ nrpe ------- ------- ------- ------- ------- ------- ------- ------- ------- tribaal/ workspace/ charm-helpers/ trunk/tests/ contrib/ charmsupport/ test_nrpe. py", line 109, in test_update_nrpe tribaal/ workspace/ charm-helpers/ trunk/charmhelp ers/contrib/ charmsupport/ nrpe.py" , line 195, in __init__ nagios_ servicegroups = self.config[ 'nagios_ servicegroups' ] servicegroups'
ERROR: tests.contrib.
-------
_StringException: Traceback (most recent call last):
File "/home/
checker = nrpe.NRPE()
File "/home/
self.
KeyError: 'nagios_
======= ======= ======= ======= ======= ======= ======= ======= ======= ======= charmsupport. test_nrpe. NRPE...
ERROR: tests.contrib.