Bug fix on the update_ipv6addr_slivertag.py plugin. Just avoided to use

[nodemanager.git] / plugins / update_ipv6addr_slivertag.py

diff --git a/plugins/update_ipv6addr_slivertag.py b/plugins/update_ipv6addr_slivertag.py
index ad72875..3f63c2d 100644 (file)
--- a/plugins/update_ipv6addr_slivertag.py
+++ b/plugins/update_ipv6addr_slivertag.py
@@ -1,5 +1,3 @@
-# -*- python-indent: 4 -*-
-
 """
 Description: Update the IPv6 Address sliver tag accordingly to the IPv6 address set
 update_ipv6addr_slivertag nodemanager plugin
@@ -26,35 +24,36 @@ def start():
     logger.log("update_ipv6addr_slivertag: plugin starting up...")
 
 
-def getSliverTagId(slivertags):
+def get_sliver_tag_id_value(slivertags):
     for slivertag in slivertags:
         if slivertag['tagname']==ipv6addrtag:
-            return slivertag['slice_tag_id']
-    return None
+            return slivertag['slice_tag_id'], slivertag['value']
 
 def SetSliverTag(plc, data, tagname):
 
     for slice in data['slivers']:
         logger.log("update_ipv6addr_slivertag: starting with slice=%s" % (slice['name']) )
 
-           # TODO: what about the prefixlen? should we add on it as well?
-           # here, I'm just taking the ipv6addr (value)
+        # TODO: what about the prefixlen? should we add on it as well?
+        # here, I'm just taking the ipv6addr (value)
         value,prefixlen = tools.get_sliver_ipv6(slice['name'])
 
         node_id = tools.node_id()
         slivertags = plc.GetSliceTags({"name":slice['name'],"node_id":node_id,"tagname":tagname})
-           #logger.log(repr(str(slivertags)))
-           #for tag in slivertags:
-           #   logger.log(repr(str(tag)))
-
-        ipv6addr = plc.GetSliceIPv6Address(slice['name'])
+        #logger.log(repr(str(slivertags)))
+        #for tag in slivertags:
+            #    logger.log(repr(str(tag)))
+
+        try:
+            slivertag_id,ipv6addr = get_sliver_tag_id_value(slivertags)
+        except:
+            slivertag_id,ipv6addr = None,None
         logger.log("update_ipv6addr_slivertag: slice=%s getSliceIPv6Address=%s" % \
                (slice['name'],ipv6addr) )
         # if the value to set is null...
         if value is None:
             if ipv6addr is not None:
                 # then, let's remove the slice tag
-                slivertag_id = getSliverTagId(slivertags)
                 if slivertag_id:
                     try:
                         plc.DeleteSliceTag(slivertag_id)
@@ -78,7 +77,6 @@ def SetSliverTag(plc, data, tagname):
                            "slice=%s tag=%s node_id=%d" % (slice['name'],tagname,node_id) )
             # if the ipv6 addr set on the slice is different on the value provided, let's update it
             if (ipv6addr is not None) and (len(value)>0) and (ipv6addr!=value):
-                slivertag_id = getSliverTagId(slivertags)
                 plc.UpdateSliceTag(slivertag_id,value)
             # ipv6 entry on /etc/hosts of each slice
             result = tools.search_ipv6addr_hosts(slice['name'], value)
@@ -89,7 +87,5 @@ def SetSliverTag(plc, data, tagname):
                "slice=%s" % (slice['name']) )
 
 def GetSlivers(data, config, plc):
-
     SetSliverTag(plc, data, ipv6addrtag)
-
     logger.log("update_ipv6addr_slivertag: all done!")