Adding -use properties for discoveryrules

[6jo6jo]

Hi all,

I write a patch to add '-use' feature on discovery rules like this :

define discoveryrule { discoveryrule_name RouterOS creation_type host macvendor routerboard.com openports ^8291$ +use http,router-os -use linux }

diff --git a/shinken/discovery/discoverymanager.py b/shinken/discovery/discoverymanager.py index 1396d3d..9ea9884 100755 --- a/shinken/discovery/discoverymanager.py +++ b/shinken/discovery/discoverymanager.py @@ -32,6 +32,7 @@ import re  import time  import copy  import random +import string  # Always initialize random...  random.seed(time.time())  try: @@ -112,22 +113,52 @@ class DiscoveredHost(object):              d['address'] = self.data['ip']

         self.matched_rules.sort(by_order)

+          for r in self.matched_rules:              for k,v in r.writing_properties.iteritems():

•                # If it's a + (add) property, add with a ,
•                # If it's a + (add) property, append                  if k.startswith('+'):
•                    prop = k[1:]
•                    kprop = k[1:]                      # If the d do not already have this prop,
•                    # just push it
•                    if not prop in d:
•                        d[prop] = v
•                    # oh, must add with a , so
•                    # create list
•                    if not kprop in d:
•                        print 'New prop',kprop
•                        d[kprop]=[] +
•                elif not k.startswith('-'):
•                    kprop = k
•                    if not kprop in d:
•                        print 'New prop',kprop                      else:
•                        print 'Already got', d[prop], 'add', v
•                        d[prop] = d[prop] + ',' + v
•                else:
•                    d[k] = v
•                        print 'Prop',kprop,'reset with new value'
•                    d[kprop]=[] +
•                for prop in string.split(v,','):
•                    prop=prop.strip()
•                    #checks that prop does not already exist and adds
•                    if not prop in d[kprop]:
•                    if not prop in d[kprop]:
•                        if len(d[kprop]) > 0:
•                            print 'Already got', ','.join(d[kprop]), 'add', prop
•                        else:
•                            print 'Add',prop
•                        d[kprop].append(prop) +
•            # Now look for - (rem) property
•            for k,v in r.writing_properties.iteritems():
•                if k.startswith('-'):
•                    kprop = k[1:]
•                    if kprop in d:
•                        for prop in string.split(v,','):
•                            prop = prop.strip()
•                            if prop in d[kprop]:
•                                print 'Already got', ','.join(d[kprop]), 'rem', prop
•                                d[kprop].remove(prop) +
•        # Change join prop list in string with a ',' separator
•        for (k,v) in d.iteritems():
•            if type(d[k]).name=='list':
•                d[k]=','.join(d[k]) +          self.properties = d          print 'Update our properties', self.name, d

diff --git a/shinken/objects/discoveryrule.py b/shinken/objects/discoveryrule.py index 45624ab..26c63ee 100755 --- a/shinken/objects/discoveryrule.py +++ b/shinken/objects/discoveryrule.py @@ -88,11 +88,12 @@ class Discoveryrule(MatchingItem):          # with a !, it's a not rule)          # -> in self.matches or self.not_matches          # in writing properties if start with + (means 'add this')

•        # in writing properties if start with - (means 'del this')          for key in params:              # Some key are quite special              if key in cls.properties:                  setattr(self, key, params[key])
•            elif key in ['use'] or key.startswith('+') or key in tcls.properties or key.startswith('_'):
•            elif key in ['use'] or key.startswith('+') or key.startswith('-') or key in tcls.properties or key.startswith('_'):                  self.writing_properties[key] = params[key]              else:                  if key.startswith('!'):

[6jo6jo]

Sorry, i'm not sure it's the good way (6 - Open an issue and paste your diff ...)

[naparuba]

Github is not happy with the +, can you send me the patch by mail? I'll integer it :)

[6jo6jo]

Bonsoir (je vie à Noumea GMT +11)

J'ai envoyé le patch sur la liste avec le wikicode pour la doc. Tu préfères peut être le patch en pièce jointe?

Joël

Envoyé de mon iPhone

Le 26 oct. 2012 à 18:37, "Gabès Jean" notifications@github.com a écrit :

Github is not happy with the +, can you send me the patch by mail? I'll integer it :)

— Reply to this email directly or view it on GitHubhttps://github.com/naparuba/shinken/issues/646#issuecomment-9804980.

[naparuba]

Was included. Close.