X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=southbound%2Fsouthbound-impl%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fovsdb%2Fsouthbound%2Fovsdb%2Ftransact%2FTerminationPointCreateCommand.java;h=e043b764355902823bb7154707bc9d4e7a089950;hb=48096e085b70127b9358d9ec33e772a6d94f569d;hp=ce3a1464376fac82265bd62b89080aa8cf4efd4b;hpb=1c2ffcc0b97fda099d0e3fc9ad2e14856feeb4c5;p=ovsdb.git diff --git a/southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/TerminationPointCreateCommand.java b/southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/TerminationPointCreateCommand.java index ce3a14643..e043b7643 100644 --- a/southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/TerminationPointCreateCommand.java +++ b/southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/TerminationPointCreateCommand.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2015 Brocade Communications Systems, Inc. and others. All rights reserved. + * Copyright © 2015, 2017 Brocade Communications Systems, Inc. and others. All rights reserved. * * This program and the accompanying materials are made available under the * terms of the Eclipse Public License v1.0 which accompanies this distribution, @@ -8,36 +8,37 @@ package org.opendaylight.ovsdb.southbound.ovsdb.transact; import static org.opendaylight.ovsdb.lib.operations.Operations.op; +import static org.opendaylight.ovsdb.southbound.SouthboundUtil.schemaMismatchLog; +import java.util.ArrayList; +import java.util.Arrays; +import java.util.Collection; import java.util.Collections; import java.util.HashMap; import java.util.HashSet; import java.util.List; import java.util.Map; import java.util.Map.Entry; -import java.util.concurrent.ExecutionException; +import java.util.Optional; import java.util.Set; - -import org.opendaylight.controller.md.sal.binding.api.ReadOnlyTransaction; -import org.opendaylight.controller.md.sal.common.api.data.AsyncDataChangeEvent; -import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType; -import org.opendaylight.controller.md.sal.common.api.data.ReadFailedException; +import org.opendaylight.mdsal.binding.api.DataTreeModification; import org.opendaylight.ovsdb.lib.error.SchemaVersionMismatchException; import org.opendaylight.ovsdb.lib.notation.Mutator; import org.opendaylight.ovsdb.lib.notation.UUID; import org.opendaylight.ovsdb.lib.operations.Mutate; import org.opendaylight.ovsdb.lib.operations.TransactionBuilder; -import org.opendaylight.ovsdb.lib.schema.typed.TyperUtils; import org.opendaylight.ovsdb.schema.openvswitch.Bridge; import org.opendaylight.ovsdb.schema.openvswitch.Interface; import org.opendaylight.ovsdb.schema.openvswitch.Port; +import org.opendaylight.ovsdb.southbound.InstanceIdentifierCodec; import org.opendaylight.ovsdb.southbound.SouthboundConstants; import org.opendaylight.ovsdb.southbound.SouthboundMapper; -import org.opendaylight.ovsdb.southbound.SouthboundProvider; +import org.opendaylight.ovsdb.southbound.SouthboundUtil; +import org.opendaylight.ovsdb.utils.yang.YangUtils; import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.InterfaceTypeBase; -import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbBridgeAugmentation; import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbPortInterfaceAttributes.VlanMode; import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbTerminationPointAugmentation; +import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.ovsdb.port._interface.attributes.InterfaceBfd; import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.ovsdb.port._interface.attributes.InterfaceExternalIds; import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.ovsdb.port._interface.attributes.InterfaceLldp; import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.ovsdb.port._interface.attributes.InterfaceOtherConfigs; @@ -47,65 +48,78 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.re import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.ovsdb.port._interface.attributes.Trunks; import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node; import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.node.TerminationPoint; -import org.opendaylight.yangtools.yang.binding.DataObject; import org.opendaylight.yangtools.yang.binding.InstanceIdentifier; +import org.opendaylight.yangtools.yang.common.Uint16; +import org.opendaylight.yangtools.yang.common.Uint32; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import com.google.common.base.Optional; -import com.google.common.collect.ImmutableMap; -import com.google.common.collect.Sets; -import com.google.common.util.concurrent.CheckedFuture; - -public class TerminationPointCreateCommand extends AbstractTransactCommand { +public class TerminationPointCreateCommand implements TransactCommand { private static final Logger LOG = LoggerFactory.getLogger(TerminationPointCreateCommand.class); - public TerminationPointCreateCommand(BridgeOperationalState state, - AsyncDataChangeEvent, DataObject> changes) { - super(state, changes); + @Override + public void execute(final TransactionBuilder transaction, final BridgeOperationalState state, + final DataChangeEvent events, final InstanceIdentifierCodec instanceIdentifierCodec) { + execute(transaction, state, TransactUtils.extractCreated(events, OvsdbTerminationPointAugmentation.class), + TransactUtils.extractCreatedOrUpdated(events, Node.class), instanceIdentifierCodec); } @Override - public void execute(TransactionBuilder transaction) { - for (Entry, DataObject> entry: getChanges().getCreatedData().entrySet()) { - DataObject dataObject = entry.getValue(); - if (dataObject instanceof OvsdbTerminationPointAugmentation) { - OvsdbTerminationPointAugmentation terminationPoint = (OvsdbTerminationPointAugmentation) dataObject; - LOG.debug("Received request to create termination point {}", - terminationPoint.getName()); - InstanceIdentifier terminationPointIid = entry.getKey(); - Optional terminationPointOptional = - getOperationalState().getBridgeTerminationPoint(terminationPointIid); - if (!terminationPointOptional.isPresent()) { - // Configure interface - String interfaceUuid = "Interface_" + SouthboundMapper.getRandomUUID(); - Interface ovsInterface = - TyperUtils.getTypedRowWrapper(transaction.getDatabaseSchema(), Interface.class); - createInterface(terminationPoint, ovsInterface); - transaction.add(op.insert(ovsInterface).withId(interfaceUuid)); - - stampInstanceIdentifier(transaction, (InstanceIdentifier) entry.getKey(), - ovsInterface.getName()); - - // Configure port with the above interface details - String portUuid = "Port_" + SouthboundMapper.getRandomUUID(); - Port port = TyperUtils.getTypedRowWrapper(transaction.getDatabaseSchema(), Port.class); - createPort(terminationPoint, port, interfaceUuid); - transaction.add(op.insert(port).withId(portUuid)); - - //Configure bridge with the above port details - Bridge bridge = TyperUtils.getTypedRowWrapper(transaction.getDatabaseSchema(), Bridge.class); - if (getBridge(entry.getKey()) != null) { - bridge.setName(getBridge(entry.getKey()).getBridgeName().getValue()); - bridge.setPorts(Sets.newHashSet(new UUID(portUuid))); - - transaction.add(op.mutate(bridge) - .addMutation(bridge.getPortsColumn().getSchema(), - Mutator.INSERT,bridge.getPortsColumn().getData()) - .where(bridge.getNameColumn().getSchema() - .opEqual(bridge.getNameColumn().getData())).build()); - } + public void execute(final TransactionBuilder transaction, final BridgeOperationalState state, + final Collection> modifications, + final InstanceIdentifierCodec instanceIdentifierCodec) { + execute(transaction, state, + TransactUtils.extractCreated(modifications, OvsdbTerminationPointAugmentation.class), + TransactUtils.extractCreatedOrUpdated(modifications, Node.class), instanceIdentifierCodec); + } + + private void execute(final TransactionBuilder transaction, final BridgeOperationalState state, + final Map, OvsdbTerminationPointAugmentation> + createdTerminationPoints, + final Map, Node> nodes, final InstanceIdentifierCodec instanceIdentifierCodec) { + for (Entry, OvsdbTerminationPointAugmentation> entry : + createdTerminationPoints.entrySet()) { + OvsdbTerminationPointAugmentation terminationPoint = entry.getValue(); + LOG.debug("Received request to create termination point {}", + terminationPoint.getName()); + InstanceIdentifier terminationPointIid = entry.getKey(); + Optional terminationPointOptional = + state.getBridgeTerminationPoint(terminationPointIid); + if (!terminationPointOptional.isPresent()) { + // Configure interface + String interfaceUuid = "Interface_" + SouthboundMapper.getRandomUuid(); + Interface ovsInterface = transaction.getTypedRowWrapper(Interface.class); + createInterface(terminationPoint, ovsInterface); + transaction.add(op.insert(ovsInterface).withId(interfaceUuid)); + + stampInstanceIdentifier(transaction, entry.getKey(), ovsInterface.getName(), instanceIdentifierCodec); + + // Configure port with the above interface details + String portUuid = "Port_" + SouthboundMapper.getRandomUuid(); + Port port = transaction.getTypedRowWrapper(Port.class); + final String opendaylightIid = instanceIdentifierCodec.serialize(terminationPointIid); + createPort(terminationPoint, port, interfaceUuid, opendaylightIid); + transaction.add(op.insert(port).withId(portUuid)); + LOG.info("Created Termination Point : {} with Uuid : {}", + terminationPoint.getName(),portUuid); + //Configure bridge with the above port details + Bridge bridge = transaction.getTypedRowWrapper(Bridge.class); + String bridgeName = SouthboundUtil + .getBridgeNameFromOvsdbNodeId(entry.getKey().firstIdentifierOf(Node.class)); + if (bridgeName != null) { + LOG.trace("Updating bridge {} for newly added port {}", bridgeName, terminationPoint.getName()); + bridge.setName(bridgeName); + bridge.setPorts(Collections.singleton(new UUID(portUuid))); + + transaction.add(op.mutate(bridge) + .addMutation(bridge.getPortsColumn().getSchema(), + Mutator.INSERT, bridge.getPortsColumn().getData()) + .where(bridge.getNameColumn().getSchema() + .opEqual(bridge.getNameColumn().getData())).build()); + } else { + LOG.error("Missing BridgeName for Node {} during creation of port {}", + entry.getKey().firstIdentifierOf(Node.class), terminationPoint.getName()); } } } @@ -124,11 +138,11 @@ public class TerminationPointCreateCommand extends AbstractTransactCommand { createInterfaceOtherConfig(terminationPoint, ovsInterface); createInterfaceExternalIds(terminationPoint, ovsInterface); createInterfaceLldp(terminationPoint, ovsInterface); + createInterfaceBfd(terminationPoint, ovsInterface); } - private void createInterfaceType(final OvsdbTerminationPointAugmentation terminationPoint, - final Interface ovsInterface) { - + private static void createInterfaceType(final OvsdbTerminationPointAugmentation terminationPoint, + final Interface ovsInterface) { Class mdsaltype = terminationPoint.getInterfaceType(); if (mdsaltype != null) { ovsInterface.setType(SouthboundMapper.createOvsdbInterfaceType(mdsaltype)); @@ -137,24 +151,24 @@ public class TerminationPointCreateCommand extends AbstractTransactCommand { private void createPort( final OvsdbTerminationPointAugmentation terminationPoint, - final Port port, final String interfaceUuid) { + final Port port, final String interfaceUuid, final String opendaylightIid) { port.setName(terminationPoint.getName()); - port.setInterfaces(Sets.newHashSet(new UUID(interfaceUuid))); + port.setInterfaces(Collections.singleton(new UUID(interfaceUuid))); createPortOtherConfig(terminationPoint, port); createPortVlanTag(terminationPoint, port); createPortVlanTrunk(terminationPoint, port); createPortVlanMode(terminationPoint, port); - createPortExternalIds(terminationPoint, port); + createPortExternalIds(terminationPoint, port, opendaylightIid); } private void createOfPort( final OvsdbTerminationPointAugmentation terminationPoint, final Interface ovsInterface) { - Long ofPort = terminationPoint.getOfport(); + Uint32 ofPort = terminationPoint.getOfport(); if (ofPort != null) { - ovsInterface.setOpenFlowPort(Sets.newHashSet(ofPort)); + ovsInterface.setOpenFlowPort(Collections.singleton(ofPort.toJava())); } } @@ -162,9 +176,9 @@ public class TerminationPointCreateCommand extends AbstractTransactCommand { final OvsdbTerminationPointAugmentation terminationPoint, final Interface ovsInterface) { - Integer ofPortRequest = terminationPoint.getOfportRequest(); + Uint16 ofPortRequest = terminationPoint.getOfportRequest(); if (ofPortRequest != null) { - ovsInterface.setOpenFlowPortRequest(Sets.newHashSet(ofPortRequest.longValue())); + ovsInterface.setOpenFlowPortRequest(Collections.singleton(ofPortRequest.longValue())); } } @@ -174,12 +188,9 @@ public class TerminationPointCreateCommand extends AbstractTransactCommand { //Configure optional input if (terminationPoint.getOptions() != null) { - Map optionsMap = new HashMap<>(); - for (Options option : terminationPoint.getOptions()) { - optionsMap.put(option.getOption(), option.getValue()); - } try { - ovsInterface.setOptions(ImmutableMap.copyOf(optionsMap)); + ovsInterface.setOptions(YangUtils.convertYangKeyValueListToMap(terminationPoint.getOptions(), + Options::getOption, Options::getValue)); } catch (NullPointerException e) { LOG.warn("Incomplete OVSDB interface options", e); } @@ -193,15 +204,17 @@ public class TerminationPointCreateCommand extends AbstractTransactCommand { List interfaceExternalIds = terminationPoint.getInterfaceExternalIds(); if (interfaceExternalIds != null && !interfaceExternalIds.isEmpty()) { - Map externalIdsMap = new HashMap<>(); - for (InterfaceExternalIds externalId: interfaceExternalIds) { - externalIdsMap.put(externalId.getExternalIdKey(), externalId.getExternalIdValue()); - } - try { - ovsInterface.setExternalIds(ImmutableMap.copyOf(externalIdsMap)); - } catch (NullPointerException e) { - LOG.warn("Incomplete OVSDB interface external_ids", e); - } + interfaceExternalIds.add(SouthboundUtil.createExternalIdsForInterface( + SouthboundConstants.CREATED_BY, SouthboundConstants.ODL)); + } else { + interfaceExternalIds = Arrays.asList(SouthboundUtil.createExternalIdsForInterface( + SouthboundConstants.CREATED_BY, SouthboundConstants.ODL)); + } + try { + ovsInterface.setExternalIds(YangUtils.convertYangKeyValueListToMap(interfaceExternalIds, + InterfaceExternalIds::getExternalIdKey, InterfaceExternalIds::getExternalIdValue)); + } catch (NullPointerException e) { + LOG.warn("Incomplete OVSDB interface external_ids", e); } } @@ -233,36 +246,61 @@ public class TerminationPointCreateCommand extends AbstractTransactCommand { List interfaceLldpList = terminationPoint.getInterfaceLldp(); if (interfaceLldpList != null && !interfaceLldpList.isEmpty()) { - Map interfaceLldpMap = new HashMap<>(); - for (InterfaceLldp interfaceLldp : interfaceLldpList) { - interfaceLldpMap.put(interfaceLldp.getLldpKey(), interfaceLldp.getLldpValue()); - } try { - ovsInterface.setLldp(ImmutableMap.copyOf(interfaceLldpMap)); + ovsInterface.setLldp(YangUtils.convertYangKeyValueListToMap(interfaceLldpList, + InterfaceLldp::getLldpKey, InterfaceLldp::getLldpValue)); } catch (NullPointerException e) { LOG.warn("Incomplete OVSDB interface lldp", e); } } } catch (SchemaVersionMismatchException e) { - LOG.debug("lldp column for Interface Table unsupported for this version of ovsdb schema", e); + schemaMismatchLog("lldp", "Interface", e); + } + } + + private void createInterfaceBfd(final OvsdbTerminationPointAugmentation terminationPoint, + final Interface ovsInterface) { + + try { + List interfaceBfdList = terminationPoint.getInterfaceBfd(); + if (interfaceBfdList != null && !interfaceBfdList.isEmpty()) { + try { + ovsInterface.setBfd(YangUtils.convertYangKeyValueListToMap(interfaceBfdList, + InterfaceBfd::getBfdKey, InterfaceBfd::getBfdValue)); + } catch (NullPointerException e) { + LOG.warn("Incomplete OVSDB interface bfd", e); + } + } + } catch (SchemaVersionMismatchException e) { + schemaMismatchLog("bfd", "Interface", e); } } private void createPortExternalIds( final OvsdbTerminationPointAugmentation terminationPoint, - final Port port) { + final Port port, final String opendaylightIid) { + // Set the iid external_id List portExternalIds = terminationPoint.getPortExternalIds(); if (portExternalIds != null && !portExternalIds.isEmpty()) { - Map externalIdsMap = new HashMap<>(); - for (PortExternalIds externalId: portExternalIds) { - externalIdsMap.put(externalId.getExternalIdKey(), externalId.getExternalIdValue()); - } - try { - port.setExternalIds(ImmutableMap.copyOf(externalIdsMap)); - } catch (NullPointerException e) { - LOG.warn("Incomplete OVSDB port external_ids", e); - } + portExternalIds.add(SouthboundUtil.createExternalIdsForPort( + SouthboundConstants.CREATED_BY, SouthboundConstants.ODL)); + portExternalIds.add(SouthboundUtil.createExternalIdsForPort( + SouthboundConstants.IID_EXTERNAL_ID_KEY, opendaylightIid)); + } else { + portExternalIds = new ArrayList<>(); + portExternalIds.add(SouthboundUtil.createExternalIdsForPort( + SouthboundConstants.CREATED_BY, SouthboundConstants.ODL)); + portExternalIds.add(SouthboundUtil.createExternalIdsForPort( + SouthboundConstants.IID_EXTERNAL_ID_KEY, opendaylightIid)); + } + try { + port.setExternalIds(YangUtils.convertYangKeyValueListToMap(portExternalIds, + PortExternalIds::getExternalIdKey, PortExternalIds::getExternalIdValue)); + //YangUtils.copyYangKeyValueListToMap(externalIdMap, terminationPoint.getPortExternalIds(), + // PortExternalIds::getExternalIdKey, PortExternalIds::getExternalIdValue); + } catch (NullPointerException e) { + LOG.warn("Incomplete OVSDB port external_ids", e); } } @@ -299,7 +337,7 @@ public class TerminationPointCreateCommand extends AbstractTransactCommand { if (terminationPoint.getVlanMode() != null) { Set portVlanMode = new HashSet<>(); VlanMode modelVlanMode = terminationPoint.getVlanMode(); - portVlanMode.add(SouthboundConstants.VLANMODES.values()[modelVlanMode.getIntValue() - 1].getMode()); + portVlanMode.add(SouthboundConstants.VlanModes.values()[modelVlanMode.getIntValue() - 1].getMode()); port.setVlanMode(portVlanMode); } } @@ -310,57 +348,25 @@ public class TerminationPointCreateCommand extends AbstractTransactCommand { List portOtherConfigs = terminationPoint.getPortOtherConfigs(); if (portOtherConfigs != null && !portOtherConfigs.isEmpty()) { - Map otherConfigsMap = new HashMap<>(); - for (PortOtherConfigs portOtherConfig : portOtherConfigs) { - otherConfigsMap.put(portOtherConfig.getOtherConfigKey(), - portOtherConfig.getOtherConfigValue()); - } try { - ovsPort.setOtherConfig(ImmutableMap.copyOf(otherConfigsMap)); + ovsPort.setOtherConfig(YangUtils.convertYangKeyValueListToMap(portOtherConfigs, + PortOtherConfigs::getOtherConfigKey, PortOtherConfigs::getOtherConfigValue)); } catch (NullPointerException e) { LOG.warn("Incomplete OVSDB port other_config", e); } } } - private OvsdbBridgeAugmentation getBridge(InstanceIdentifier key) { - OvsdbBridgeAugmentation bridge = null; - InstanceIdentifier nodeIid = key.firstIdentifierOf(Node.class); - Map, Node> nodes = - TransactUtils.extractCreatedOrUpdated(getChanges(),Node.class); - if (nodes != null && nodes.get(nodeIid) != null) { - Node node = nodes.get(nodeIid); - bridge = node.getAugmentation(OvsdbBridgeAugmentation.class); - if (bridge == null) { - ReadOnlyTransaction transaction = SouthboundProvider.getDb().newReadOnlyTransaction(); - CheckedFuture, ReadFailedException> future = - transaction.read(LogicalDatastoreType.OPERATIONAL, nodeIid); - try { - Optional nodeOptional = future.get(); - if (nodeOptional.isPresent()) { - bridge = nodeOptional.get().getAugmentation(OvsdbBridgeAugmentation.class); - } - } catch (InterruptedException | ExecutionException e) { - LOG.warn("Error reading from datastore",e); - } - transaction.close(); - } - } - return bridge; - } - - public static void stampInstanceIdentifier(TransactionBuilder transaction,InstanceIdentifier iid, - String interfaceName) { - Port port = TyperUtils.getTypedRowWrapper(transaction.getDatabaseSchema(), Port.class); + public static void stampInstanceIdentifier(final TransactionBuilder transaction, + final InstanceIdentifier iid, final String interfaceName, + final InstanceIdentifierCodec instanceIdentifierCodec) { + Port port = transaction.getTypedRowWrapper(Port.class); port.setName(interfaceName); - port.setExternalIds(Collections.emptyMap()); - Mutate mutate = TransactUtils.stampInstanceIdentifierMutation(transaction, - iid, - port.getSchema(), - port.getExternalIdsColumn().getSchema()); + port.setExternalIds(Collections.emptyMap()); + Mutate mutate = TransactUtils.stampInstanceIdentifierMutation(transaction, iid, port.getSchema(), + port.getExternalIdsColumn().getSchema(), instanceIdentifierCodec); transaction.add(mutate .where(port.getNameColumn().getSchema().opEqual(interfaceName)) .build()); } - }