Commit 993f9903 authored by Lluis Gifre Renom's avatar Lluis Gifre Renom
Browse files

Service Handler L3NM OpenConfig

- skipped endpoint configuration when no config rules are speicfied for an endpoint
parent 9a905796
Loading
Loading
Loading
Loading
+5 −2
Original line number Diff line number Diff line
@@ -21,8 +21,11 @@ def setup_config_rules(
    service_settings : TreeNode, endpoint_settings : TreeNode
) -> List[Dict]:

    json_settings          : Dict = {} if service_settings  is None else service_settings.value
    json_endpoint_settings : Dict = {} if endpoint_settings is None else endpoint_settings.value
    if service_settings  is None: return []
    if endpoint_settings is None: return []

    json_settings          : Dict = service_settings.value
    json_endpoint_settings : Dict = endpoint_settings.value

    service_short_uuid        = service_uuid.split('-')[-1]
    network_instance_name     = '{:s}-NetInst'.format(service_short_uuid)
+12 −12
Original line number Diff line number Diff line
@@ -86,7 +86,7 @@ class L3NMOpenConfigServiceHandler(_ServiceHandler):

                json_config_rules = setup_config_rules(
                    service_uuid, connection_uuid, device_uuid, endpoint_uuid, settings, endpoint_settings)

                if len(json_config_rules) > 0:
                    device = self.__task_executor.get_device(DeviceId(**json_device_id(device_uuid)))
                    del device.device_config.config_rules[:]
                    for json_config_rule in json_config_rules:
@@ -121,7 +121,7 @@ class L3NMOpenConfigServiceHandler(_ServiceHandler):

                json_config_rules = teardown_config_rules(
                    service_uuid, connection_uuid, device_uuid, endpoint_uuid, settings, endpoint_settings)

                if len(json_config_rules) > 0:
                    device = self.__task_executor.get_device(DeviceId(**json_device_id(device_uuid)))
                    del device.device_config.config_rules[:]
                    for json_config_rule in json_config_rules: