From 88c9074e0a094474bc407f0d1aa5de3a428dcb00 Mon Sep 17 00:00:00 2001 From: Balagangadhar Bathula Date: Thu, 26 May 2022 07:35:57 -0400 Subject: [PATCH] Fix bug in 200G 31.6 GBaud renderer - Instance ID for 200G OTSi group should be 2 not 4. - Fix the issue with interface naming when the source-tp is not null and has network-token Signed-off-by: Balagangadhar Bathula Change-Id: Iad341cb0f4357feb8288141d87b3348fb03f3f6d --- .../renderer/openroadminterface/OpenRoadmInterface710.java | 3 +-- .../renderer/provisiondevice/DeviceRendererServiceImpl.java | 6 ++++++ 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/renderer/src/main/java/org/opendaylight/transportpce/renderer/openroadminterface/OpenRoadmInterface710.java b/renderer/src/main/java/org/opendaylight/transportpce/renderer/openroadminterface/OpenRoadmInterface710.java index 01ed89c37..51fdf9626 100644 --- a/renderer/src/main/java/org/opendaylight/transportpce/renderer/openroadminterface/OpenRoadmInterface710.java +++ b/renderer/src/main/java/org/opendaylight/transportpce/renderer/openroadminterface/OpenRoadmInterface710.java @@ -224,8 +224,7 @@ public class OpenRoadmInterface710 { LOG.info("FOIC is 2.8 for 31.6 Gbaud and rate is 200"); // FOIC rate is different flexoBuilder.setFoicType(Foic28.class) - .setIid(new ArrayList<>(Arrays.asList(Uint8.valueOf(1), Uint8.valueOf(2), - Uint8.valueOf(3), Uint8.valueOf(4)))); + .setIid(new ArrayList<>(Arrays.asList(Uint8.valueOf(1), Uint8.valueOf(2)))); } else { // default is dp-qpsk for 200G under 63.1 GBaud flexoBuilder.setFoicType(Foic24.class) diff --git a/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/DeviceRendererServiceImpl.java b/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/DeviceRendererServiceImpl.java index bf2e12eaa..a861d59e1 100644 --- a/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/DeviceRendererServiceImpl.java +++ b/renderer/src/main/java/org/opendaylight/transportpce/renderer/provisiondevice/DeviceRendererServiceImpl.java @@ -188,6 +188,12 @@ public class DeviceRendererServiceImpl implements DeviceRendererService { String supportingOchInterface = this.openRoadmInterfaceFactory.createOpenRoadmOchInterface( nodeId, srcTp, spectrumInformation); createdOchInterfaces.add(supportingOchInterface); + // Split the string based on # pass the last element as the supported Interface + // This is needed for 7.1 device models with B100G, we have OTSI, OTSI-group combined as OCH + String[] listOfSuppOchInf = supportingOchInterface.split("#"); + createdOchInterfaces = Arrays.asList(listOfSuppOchInf); + // Taking the last element + supportingOchInterface = createdOchInterfaces.get(createdOchInterfaces.size() - 1); String supportingOtuInterface = this.openRoadmInterfaceFactory.createOpenRoadmOtu4Interface( nodeId, srcTp, supportingOchInterface, apiInfoA, apiInfoZ); createdOtuInterfaces.add(supportingOtuInterface); -- 2.36.6