Skip to content
Snippets Groups Projects
Commit 5ff8c113 authored by Erik Reid's avatar Erik Reid
Browse files

replace repeated string by const, in a few places

parent 2bc37d6d
No related branches found
No related tags found
No related merge requests found
from brian_polling_manager import sensu from brian_polling_manager import sensu
_CHECK_PREFIX = 'eumetmc'
def load_eumetsat_multicast_checks(sensu_params): def load_eumetsat_multicast_checks(sensu_params):
def _is_eumetsat_multicast_check(check): def _is_eumetsat_multicast_check(check):
name = check['metadata']['name'] name = check['metadata']['name']
return name.startswith('eumetmc') return name.startswith(_CHECK_PREFIX)
ifc_checks = filter( ifc_checks = filter(
_is_eumetsat_multicast_check, sensu.load_all_checks(sensu_params)) _is_eumetsat_multicast_check, sensu.load_all_checks(sensu_params))
return {c['metadata']['name']: c for c in ifc_checks} return {c['metadata']['name']: c for c in ifc_checks}
...@@ -19,7 +20,7 @@ class EUMETSATMulticastHostCheck(sensu.AbstractCheck): ...@@ -19,7 +20,7 @@ class EUMETSATMulticastHostCheck(sensu.AbstractCheck):
@sensu.AbstractCheck.name.getter @sensu.AbstractCheck.name.getter
def name(self): def name(self):
return f'eumetmc-{self.hostname}' return f'{_CHECK_PREFIX}-{self.hostname}'
@sensu.AbstractCheck.command.getter @sensu.AbstractCheck.command.getter
def command(self): def command(self):
......
from brian_polling_manager import sensu from brian_polling_manager import sensu
_CHECK_PREFIX = 'gwsd'
def load_gws_direct_checks(sensu_params): def load_gws_direct_checks(sensu_params):
def _is_gws_direct_check(check): def _is_gws_direct_check(check):
name = check['metadata']['name'] name = check['metadata']['name']
return name.startswith('gwsd') return name.startswith(_CHECK_PREFIX)
ifc_checks = filter( ifc_checks = filter(
_is_gws_direct_check, sensu.load_all_checks(sensu_params)) _is_gws_direct_check, sensu.load_all_checks(sensu_params))
return {c['metadata']['name']: c for c in ifc_checks} return {c['metadata']['name']: c for c in ifc_checks}
...@@ -23,7 +25,7 @@ class GwSDirectInterfaceCheck(sensu.AbstractCheck): ...@@ -23,7 +25,7 @@ class GwSDirectInterfaceCheck(sensu.AbstractCheck):
isp = isp.replace(' ', '_') isp = isp.replace(' ', '_')
tag = self.interface['tag'] tag = self.interface['tag']
tag = tag.replace(' ', '_') tag = tag.replace(' ', '_')
return f'gwsd-{self.interface["nren"]}-{isp}-{tag}' return f'{_CHECK_PREFIX}-{self.interface["nren"]}-{isp}-{tag}'
@sensu.AbstractCheck.command.getter @sensu.AbstractCheck.command.getter
def command(self): def command(self):
......
import re import re
from brian_polling_manager import sensu from brian_polling_manager import sensu
_CHECK_PREFIX = 'dscp32'
def load_dscp32_checks(sensu_params): def load_dscp32_checks(sensu_params):
def _is_dscp32_check(check): def _is_dscp32_check(check):
name = check['metadata']['name'] name = check['metadata']['name']
return name.startswith('dscp32') return name.startswith(_CHECK_PREFIX)
ifc_checks = filter( ifc_checks = filter(
_is_dscp32_check, sensu.load_all_checks(sensu_params)) _is_dscp32_check, sensu.load_all_checks(sensu_params))
return {c['metadata']['name']: c for c in ifc_checks} return {c['metadata']['name']: c for c in ifc_checks}
...@@ -21,7 +23,7 @@ class DSCP32CountersCheck(sensu.AbstractCheck): ...@@ -21,7 +23,7 @@ class DSCP32CountersCheck(sensu.AbstractCheck):
@sensu.AbstractCheck.name.getter @sensu.AbstractCheck.name.getter
def name(self): def name(self):
name = re.sub(r'[\s_-]+', '_', self.service['name']) name = re.sub(r'[\s_-]+', '_', self.service['name'])
return f'dscp32-{name}' return f'{_CHECK_PREFIX}-{name}'
@sensu.AbstractCheck.command.getter @sensu.AbstractCheck.command.getter
def command(self): def command(self):
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment