Remove the precedence attribute from the policy communication maps

This attribute is not supported by the nSX (and not used by the plugin)

Change-Id: I21edfa640f8760dbf2574a3b4b52713577c0d483

vmware_nsxlib/tests/unit/v3/test_policy_resources.py

@@ -840,7 +840,6 @@ class TestPolicyCommunicationMap(NsxPolicyLibTestCase):
                 name=name,
                 description=description,
                 category=policy_constants.CATEGORY_APPLICATION,
-                precedence=0,
                 tenant=TEST_TENANT)
 
             entry_def = policy_defs.CommunicationMapEntryDef(
@@ -889,7 +888,6 @@ class TestPolicyCommunicationMap(NsxPolicyLibTestCase):
                 name=name,
                 description=description,
                 category=category,
-                precedence=0,
                 tenant=TEST_TENANT)
 
             entry_def = policy_defs.CommunicationMapEntryDef(
@@ -931,7 +929,6 @@ class TestPolicyCommunicationMap(NsxPolicyLibTestCase):
                 name=name,
                 description=description,
                 category=policy_constants.CATEGORY_APPLICATION,
-                precedence=0,
                 tenant=TEST_TENANT)
 
             expected_entry_def = policy_defs.CommunicationMapEntryDef(
@@ -967,7 +964,6 @@ class TestPolicyCommunicationMap(NsxPolicyLibTestCase):
                 name=name,
                 description=description,
                 category=policy_constants.CATEGORY_APPLICATION,
-                precedence=0,
                 tenant=TEST_TENANT)
 
             self.assert_called_with_def(
@@ -1127,7 +1123,6 @@ class TestPolicyCommunicationMap(NsxPolicyLibTestCase):
                 name=name,
                 description=description,
                 category=category,
-                precedence=0,
                 tenant=TEST_TENANT)
 
             self.assert_called_with_defs(api_call,

vmware_nsxlib/v3/policy_defs.py

@@ -650,7 +650,7 @@ class CommunicationMapDef(ResourceDef):
 
     def get_obj_dict(self):
         body = super(CommunicationMapDef, self).get_obj_dict()
-        for attr in ('category', 'precedence'):
+        for attr in ('category', ):
             if self.get_attr(attr):
                 body[attr] = self.get_attr(attr)
 

vmware_nsxlib/v3/policy_resources.py

@@ -1019,7 +1019,7 @@ class NsxPolicyCommunicationMapApi(NsxPolicyResourceBase):
         return last_sequence + 1
 
     def create_or_overwrite(self, name, domain_id, map_id=None,
-                            description=None, precedence=0,
+                            description=None,
                             category=policy_constants.CATEGORY_APPLICATION,
                             sequence_number=None, service_ids=None,
                             action=policy_constants.ACTION_ALLOW,
@@ -1066,13 +1066,13 @@ class NsxPolicyCommunicationMapApi(NsxPolicyResourceBase):
         map_def = policy_defs.CommunicationMapDef(
             domain_id=domain_id, map_id=map_id,
             tenant=tenant, name=name, description=description,
-            precedence=precedence, category=category, tags=tags)
+            category=category, tags=tags)
 
         self._create_or_store(map_def, entry_def)
         return map_id
 
     def create_or_overwrite_map_only(
-        self, name, domain_id, map_id=None, description=None, precedence=0,
+        self, name, domain_id, map_id=None, description=None,
         category=policy_constants.CATEGORY_APPLICATION,
         tags=None, tenant=policy_constants.POLICY_INFRA_TENANT):
         """Create or update a CommunicationMap
@@ -1084,7 +1084,7 @@ class NsxPolicyCommunicationMapApi(NsxPolicyResourceBase):
         map_def = policy_defs.CommunicationMapDef(
             domain_id=domain_id, map_id=map_id,
             tenant=tenant, name=name, description=description,
-            precedence=precedence, category=category, tags=tags)
+            category=category, tags=tags)
 
         self.policy_api.create_or_update(map_def)
         return map_id
@@ -1114,7 +1114,7 @@ class NsxPolicyCommunicationMapApi(NsxPolicyResourceBase):
 
     def create_with_entries(
         self, name, domain_id, map_id=None,
-        description=None, precedence=0,
+        description=None,
         category=policy_constants.CATEGORY_APPLICATION,
         entries=None, tags=None,
         tenant=policy_constants.POLICY_INFRA_TENANT):
@@ -1125,7 +1125,7 @@ class NsxPolicyCommunicationMapApi(NsxPolicyResourceBase):
         map_def = policy_defs.CommunicationMapDef(
             domain_id=domain_id, map_id=map_id,
             tenant=tenant, name=name, description=description,
-            precedence=precedence, category=category, tags=tags)
+            category=category, tags=tags)
 
         # TODO(annak): support transactional create
         self.policy_api.create_with_parent(map_def, entries)
@@ -1209,7 +1209,7 @@ class NsxPolicyCommunicationMapApi(NsxPolicyResourceBase):
 
     def update(self, domain_id, map_id, name=None, description=None,
                sequence_number=None, service_ids=None, action=None,
-               source_groups=None, dest_groups=None, precedence=None,
+               source_groups=None, dest_groups=None,
                category=None, direction=None, logged=False, tags=None,
                tenant=policy_constants.POLICY_INFRA_TENANT):
 
@@ -1219,7 +1219,6 @@ class NsxPolicyCommunicationMapApi(NsxPolicyResourceBase):
             name=name,
             description=description,
             category=category,
-            precedence=precedence,
             tags=tags,
             tenant=tenant)