From 136b9074bc3a49fbbb86699eec6458d7f4d7ba7f Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Sun, 31 Jan 2016 18:48:08 +0100 Subject: Fixup after vpp-japi change vppApi has now proper lifecycle, we must adhere to it. Change-Id: Ibf7eb5726f2751d017a3c28f823329ce4150cb01 Signed-off-by: Robert Varga --- .../io/fd/honeycomb/v3po/impl/V3poApiRequest.java | 226 ++++++++++----------- .../io/fd/honeycomb/v3po/impl/V3poProvider.java | 60 +++--- .../io/fd/honeycomb/v3po/impl/V3poRequest.java | 7 +- .../honeycomb/v3po/impl/VppPollOperDataImpl.java | 77 +++---- 4 files changed, 190 insertions(+), 180 deletions(-) diff --git a/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poApiRequest.java b/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poApiRequest.java index f466adb53..21541d96e 100644 --- a/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poApiRequest.java +++ b/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poApiRequest.java @@ -16,9 +16,11 @@ package io.fd.honeycomb.v3po.impl; +import com.google.common.primitives.Ints; +import com.google.common.util.concurrent.CheckedFuture; +import com.google.common.util.concurrent.Futures; import java.math.BigInteger; import java.net.InetAddress; - import org.opendaylight.controller.md.sal.binding.api.DataBroker; import org.opendaylight.controller.md.sal.binding.api.WriteTransaction; import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType; @@ -55,6 +57,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.v3po.rev import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.v3po.rev150105.interfaces.state._interface.l2.interconnection.BridgeBasedBuilder; import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.v3po.rev150105.interfaces.state._interface.l2.interconnection.XconnectBasedBuilder; import org.opendaylight.yangtools.yang.binding.InstanceIdentifier; +import org.openvpp.vppjapi.vppApi; import org.openvpp.vppjapi.vppBridgeDomainDetails; import org.openvpp.vppjapi.vppBridgeDomainInterfaceDetails; import org.openvpp.vppjapi.vppIPv4Address; @@ -65,48 +68,47 @@ import org.openvpp.vppjapi.vppVxlanTunnelDetails; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import com.google.common.primitives.Ints; -import com.google.common.util.concurrent.CheckedFuture; -import com.google.common.util.concurrent.Futures; /* * VPP API Class overriding interface details callback */ public class V3poApiRequest extends V3poRequest { private static final Logger LOG = LoggerFactory.getLogger(V3poApiRequest.class); public String ifNames = ""; - private VppPollOperDataImpl caller; + private final VppPollOperDataImpl caller; + private final vppApi api; - public V3poApiRequest(VppPollOperDataImpl vppPollOperData) { + public V3poApiRequest(final vppApi api, final VppPollOperDataImpl vppPollOperData) { + this.api = api; caller = vppPollOperData; } - private InstanceIdentifier getStateInterfaceIid(String interfaceName) { + private InstanceIdentifier getStateInterfaceIid(final String interfaceName) { return InstanceIdentifier.create(InterfacesState.class).child(Interface.class, new InterfaceKey(interfaceName)); } - - private InstanceIdentifier getStateInterfaceIpId(InstanceIdentifier iid) { + + private InstanceIdentifier getStateInterfaceIpId(final InstanceIdentifier iid) { return iid.augmentation(Interface2.class); } - - private InstanceIdentifier getStateInterfaceStatsId(InstanceIdentifier iid) { + + private InstanceIdentifier getStateInterfaceStatsId(final InstanceIdentifier iid) { return iid.child(Statistics.class); } - - private static Counter64 getCounter64(long num) { + + private static Counter64 getCounter64(final long num) { return new Counter64(BigInteger.valueOf(num)); } - - private static Counter32 getCounter32(long num) { + + private static Counter32 getCounter32(final long num) { return new Counter32(num); } - - private Statistics buildInterfaceStatistics(vppInterfaceCounters ifCounters) { + + private Statistics buildInterfaceStatistics(final vppInterfaceCounters ifCounters) { if (ifCounters == null) { return null; } StatisticsBuilder statsBuilder = new StatisticsBuilder(); - + statsBuilder.setInBroadcastPkts(getCounter64(ifCounters.rxBroadcast)); statsBuilder.setInDiscards(getCounter32(ifCounters.rxDiscard)); statsBuilder.setInErrors(getCounter32(ifCounters.rxError)); @@ -121,20 +123,20 @@ public class V3poApiRequest extends V3poRequest { statsBuilder.setOutMulticastPkts(getCounter64(ifCounters.txMulticast)); statsBuilder.setOutOctets(getCounter64(ifCounters.txOctets)); statsBuilder.setOutUnicastPkts(getCounter64(ifCounters.txUnicast)); - + VppInterfaceStatisticsAugmentationBuilder statsAugBuilder = new VppInterfaceStatisticsAugmentationBuilder(); statsAugBuilder.setInErrorsMiss(getCounter64(ifCounters.rxMiss)); statsAugBuilder.setInErrorsNoBuf(getCounter64(ifCounters.rxFifoFull)); // FIXME? Is this right? statsAugBuilder.setOutDiscardsFifoFull(getCounter64(ifCounters.txFifoFull)); - + statsBuilder.addAugmentation(VppInterfaceStatisticsAugmentation.class, statsAugBuilder.build()); - + return statsBuilder.build(); } - private static String getMacAddress(byte[] mac) { + private static String getMacAddress(final byte[] mac) { StringBuilder sb = new StringBuilder(18); for (byte b : mac) { if (sb.length() > 0) { @@ -144,16 +146,16 @@ public class V3poApiRequest extends V3poRequest { } return sb.toString(); } - + private static final Gauge64 vppSpeed0 = new Gauge64(BigInteger.ZERO); - private static final Gauge64 vppSpeed1 = new Gauge64(BigInteger.valueOf(10 * 1000000)); - private static final Gauge64 vppSpeed2 = new Gauge64(BigInteger.valueOf(100 * 1000000)); - private static final Gauge64 vppSpeed4 = new Gauge64(BigInteger.valueOf(1000 * 1000000)); - private static final Gauge64 vppSpeed8 = new Gauge64(BigInteger.valueOf(10000L * 1000000)); - private static final Gauge64 vppSpeed16 = new Gauge64(BigInteger.valueOf(40000L * 1000000)); - private static final Gauge64 vppSpeed32 = new Gauge64(BigInteger.valueOf(100000L * 1000000)); - - private static Gauge64 getSpeed(byte vppSpeed) { + private static final Gauge64 vppSpeed1 = new Gauge64(BigInteger.valueOf(10 * 1000000)); + private static final Gauge64 vppSpeed2 = new Gauge64(BigInteger.valueOf(100 * 1000000)); + private static final Gauge64 vppSpeed4 = new Gauge64(BigInteger.valueOf(1000 * 1000000)); + private static final Gauge64 vppSpeed8 = new Gauge64(BigInteger.valueOf(10000L * 1000000)); + private static final Gauge64 vppSpeed16 = new Gauge64(BigInteger.valueOf(40000L * 1000000)); + private static final Gauge64 vppSpeed32 = new Gauge64(BigInteger.valueOf(100000L * 1000000)); + + private static Gauge64 getSpeed(final byte vppSpeed) { switch (vppSpeed) { case 1: return vppSpeed1; case 2: return vppSpeed2; @@ -164,8 +166,8 @@ public class V3poApiRequest extends V3poRequest { default: return vppSpeed0; } } - - private static String ipv4IntToString(int ip) { + + private static String ipv4IntToString(final int ip) { InetAddress addr = null; byte[] bytes = Ints.toByteArray(ip); try { @@ -177,26 +179,26 @@ public class V3poApiRequest extends V3poRequest { return addr.getHostAddress(); } - private Interface buildStateInterface(int ifIndex, - String interfaceName, - int supIfIndex, - byte[] physAddr, - byte adminUp, byte linkUp, - byte linkDuplex, byte linkSpeed, - int subId, byte subDot1ad, - byte subNumberOfTags, - int subOuterVlanId, - int subInnerVlanId, - byte subExactMatch, - byte subDefault, - byte subOuterVlanIdAny, - byte subInnerVlanIdAny, - int vtrOp, int vtrPushDot1q, - int vtrTag1, int vtrTag2, - Statistics stats) { + private Interface buildStateInterface(final int ifIndex, + final String interfaceName, + final int supIfIndex, + final byte[] physAddr, + final byte adminUp, final byte linkUp, + final byte linkDuplex, final byte linkSpeed, + final int subId, final byte subDot1ad, + final byte subNumberOfTags, + final int subOuterVlanId, + final int subInnerVlanId, + final byte subExactMatch, + final byte subDefault, + final byte subOuterVlanIdAny, + final byte subInnerVlanIdAny, + final int vtrOp, final int vtrPushDot1q, + final int vtrTag1, final int vtrTag2, + final Statistics stats) { InterfaceBuilder ifBuilder = new InterfaceBuilder(); java.lang.Class ifType; - + // FIXME: missing types for virtualethernet, subinterface, tap interface etc if (interfaceName.startsWith("loop")) { ifType = SoftwareLoopback.class; @@ -209,50 +211,48 @@ public class V3poApiRequest extends V3poRequest { .setType(ifType) .setAdminStatus((adminUp == 0 ? AdminStatus.Down : AdminStatus.Up)) .setOperStatus((linkUp == 0 ? OperStatus.Down : OperStatus.Up)); -/* +/* DataContainerNodeBuilder builder = ImmutableNodes.mapEntryBuilder() .withNodeIdentifier(new NodeIdentifierWithPredicates(Interface.QNAME, NAME_QNAME, interfaceName)); builder.withChild(ImmutableNodes.leafNode(IF_TYPE, SoftwareLoopback.QNAME))*/ - + // subinterface? if (ifIndex != supIfIndex) { // TODO: get name and set } - + if (physAddr != null) { ifBuilder.setPhysAddress(new PhysAddress(getMacAddress(physAddr))); } ifBuilder.setSpeed(getSpeed(linkSpeed)); - + if (stats != null) { ifBuilder.setStatistics(stats); } - int bdId = this.bridgeDomainIdFromInterfaceName(interfaceName); - vppBridgeDomainDetails bd = (bdId != -1 ? this.getBridgeDomainDetails(bdId) : null); - + int bdId = api.bridgeDomainIdFromInterfaceName(interfaceName); + vppBridgeDomainDetails bd = (bdId != -1 ? api.getBridgeDomainDetails(bdId) : null); + String bdName = null; short splitHorizonGroup = 0; boolean bvi = false; - + if (bd != null) { bdName = bd.name; - for (int ifIdx = 0; ifIdx < bd.interfaces.length; ifIdx++) { - vppBridgeDomainInterfaceDetails bdIf = bd.interfaces[ifIdx]; - + for (vppBridgeDomainInterfaceDetails bdIf : bd.interfaces) { if (bdIf.interfaceName != interfaceName) { continue; } if (bd.bviInterfaceName == interfaceName) { bvi = true; } - splitHorizonGroup = (short)bdIf.splitHorizonGroup; + splitHorizonGroup = bdIf.splitHorizonGroup; } } VppInterfaceStateAugmentationBuilder vppIfStateAugBuilder = new VppInterfaceStateAugmentationBuilder(); - vppIfStateAugBuilder.setDescription(this.getInterfaceDescription(interfaceName)); + vppIfStateAugBuilder.setDescription(api.getInterfaceDescription(interfaceName)); setStateInterfaceL2(vppIfStateAugBuilder, bdId != -1, false, null, bdName, splitHorizonGroup, bvi); @@ -262,7 +262,7 @@ public class V3poApiRequest extends V3poRequest { "ACME Inc.", 1234); } - vppVxlanTunnelDetails[] vxlanDet = this.vxlanTunnelDump(ifIndex); + vppVxlanTunnelDetails[] vxlanDet = api.vxlanTunnelDump(ifIndex); if (null != vxlanDet && vxlanDet.length >= 1) { setStateInterfaceVxlan(vppIfStateAugBuilder, vxlanDet[0].srcAddress, vxlanDet[0].dstAddress, vxlanDet[0].vni, @@ -271,20 +271,20 @@ public class V3poApiRequest extends V3poRequest { ifBuilder.addAugmentation(VppInterfaceStateAugmentation.class, vppIfStateAugBuilder.build()); - + InterfaceStateIpv4Builder ipv4Builder = new InterfaceStateIpv4Builder(); // TODO ipv4Builder.setMtu(1234); - + InetAddress addr = null; - - vppIPv4Address[] ipv4Addrs = ipv4AddressDump(interfaceName); + + vppIPv4Address[] ipv4Addrs = api.ipv4AddressDump(interfaceName); if (ipv4Addrs != null) { for (vppIPv4Address vppAddr : ipv4Addrs) { if (null == vppAddr) { LOG.error("ipv4 address structure in null"); continue; } - + // FIXME: vppIPv4Address and vppIPv6 address can be the same if both will use // byte array for ip byte[] bytes = Ints.toByteArray(vppAddr.ip); @@ -294,22 +294,22 @@ public class V3poApiRequest extends V3poRequest { e.printStackTrace(); continue; } - + ipv4Builder.addAddress(addr.getHostAddress(), vppAddr.prefixLength, IpAddressOrigin.Static); } } - + InterfaceStateIpv6Builder ipv6Builder = new InterfaceStateIpv6Builder(); // TODO ipv6Builder.setMtu(1234); - - vppIPv6Address[] ipv6Addrs = ipv6AddressDump(interfaceName); + + vppIPv6Address[] ipv6Addrs = api.ipv6AddressDump(interfaceName); if (ipv6Addrs != null) { for (vppIPv6Address vppAddr : ipv6Addrs) { if (null == vppAddr) { LOG.error("ipv6 address structure in null"); continue; } - + byte[] bytes = vppAddr.ip; try { addr = InetAddress.getByAddress(bytes); @@ -317,28 +317,28 @@ public class V3poApiRequest extends V3poRequest { e.printStackTrace(); continue; } - + ipv6Builder.addAddress(addr.getHostAddress(), vppAddr.prefixLength, IpAddressOrigin.Static); } } Interface2Builder ipBuilder = new Interface2Builder(); - + ipBuilder.setIpv4(ipv4Builder.build()); ipBuilder.setIpv6(ipv6Builder.build()); - + ifBuilder.addAugmentation(Interface2.class, ipBuilder.build()); - + return ifBuilder.build(); } - + private void setStateInterfaceL2( - VppInterfaceStateAugmentationBuilder augBuilder, - boolean isL2BridgeBased, boolean isXconnect, - String xconnectOutgoingInterface, - String bdName, short splitHorizonGroup, boolean bvi) { + final VppInterfaceStateAugmentationBuilder augBuilder, + final boolean isL2BridgeBased, final boolean isXconnect, + final String xconnectOutgoingInterface, + final String bdName, final short splitHorizonGroup, final boolean bvi) { L2Builder l2Builder = new L2Builder(); - + if (isXconnect) { l2Builder.setInterconnection( new XconnectBasedBuilder() @@ -352,13 +352,13 @@ public class V3poApiRequest extends V3poRequest { .setBridgedVirtualInterface(bvi) .build()); } - + augBuilder.setL2(l2Builder.build()); } - + private void setStateInterfaceEthernet( - VppInterfaceStateAugmentationBuilder augBuilder, - boolean isFullDuplex, String manufacturerDesc, Integer mtu) { + final VppInterfaceStateAugmentationBuilder augBuilder, + final boolean isFullDuplex, final String manufacturerDesc, final Integer mtu) { EthernetBuilder ethBuilder = new EthernetBuilder(); ethBuilder.setDuplex((isFullDuplex ? Duplex.Full : Duplex.Half)) @@ -369,8 +369,8 @@ public class V3poApiRequest extends V3poRequest { } private void setStateInterfaceVxlan( - VppInterfaceStateAugmentationBuilder augBuilder, int srcAddress, - int dstAddress, int vni, int encapVrfId) { + final VppInterfaceStateAugmentationBuilder augBuilder, final int srcAddress, + final int dstAddress, final int vni, final int encapVrfId) { String srcAddressStr = ipv4IntToString(srcAddress); String dstAddressStr = ipv4IntToString(dstAddress); @@ -386,8 +386,8 @@ public class V3poApiRequest extends V3poRequest { augBuilder.setVxlan(vxlan); } - private void writeToIfState(InstanceIdentifier iid, - Interface intf) { + private void writeToIfState(final InstanceIdentifier iid, + final Interface intf) { DataBroker db = caller.getDataBroker(); WriteTransaction transaction = db.newWriteOnlyTransaction(); // TODO: how to delete existing interfaces that disappeared? (reset it before each dumpInterfaces call?) @@ -395,61 +395,61 @@ public class V3poApiRequest extends V3poRequest { /*LOG.info("VPPOPER-INFO: Adding interface " + intf.getName() + " to oper DataStore.");*/ transaction.put(LogicalDatastoreType.OPERATIONAL, iid, intf); - + CheckedFuture future = transaction.submit(); Futures.addCallback(future, new LoggingFuturesCallBack( "VPPOPER-WARNING: Failed to write " + "interface to ietf-interfaces state", LOG)); } - - private void processInterfaces(vppInterfaceDetails[] ifaces) { + + private void processInterfaces(final vppInterfaceDetails[] ifaces) { for (vppInterfaceDetails swIf : ifaces) { interfaceDetails(swIf); } } - + /** * TODO-ADD-JAVADOC. */ public void swInterfaceDumpAll() { vppInterfaceDetails[] ifaces; - - ifaces = swInterfaceDump((byte) 1, "Ether".getBytes()); + + ifaces = api.swInterfaceDump((byte) 1, "Ether".getBytes()); processInterfaces(ifaces); - - ifaces = swInterfaceDump((byte) 1, "lo".getBytes()); + + ifaces = api.swInterfaceDump((byte) 1, "lo".getBytes()); processInterfaces(ifaces); - - ifaces = swInterfaceDump((byte) 1, "vxlan".getBytes()); + + ifaces = api.swInterfaceDump((byte) 1, "vxlan".getBytes()); processInterfaces(ifaces); - - ifaces = swInterfaceDump((byte) 1, "l2tpv3_tunnel".getBytes()); + + ifaces = api.swInterfaceDump((byte) 1, "l2tpv3_tunnel".getBytes()); processInterfaces(ifaces); - - ifaces = swInterfaceDump((byte) 1, "tap".getBytes()); + + ifaces = api.swInterfaceDump((byte) 1, "tap".getBytes()); processInterfaces(ifaces); } - private void interfaceDetails(vppInterfaceDetails swIf) { + private void interfaceDetails(final vppInterfaceDetails swIf) { /*LOG.info("Got interface {} (idx: {}) adminUp: {} linkUp: {} duplex: {} speed: {} subId: {}", swIf.interfaceName, swIf.ifIndex, swIf.adminUp, swIf.linkUp, swIf.linkDuplex, swIf.linkSpeed, swIf.subId);*/ - vppInterfaceCounters ifCounters = getInterfaceCounters(swIf.ifIndex); + vppInterfaceCounters ifCounters = api.getInterfaceCounters(swIf.ifIndex); InstanceIdentifier iid = getStateInterfaceIid(swIf.interfaceName); Statistics stats = buildInterfaceStatistics(ifCounters); - + Interface intf = buildStateInterface(swIf.ifIndex, swIf.interfaceName, - swIf.supIfIndex, swIf.physAddr, + swIf.supIfIndex, swIf.physAddr, swIf.adminUp, swIf.linkUp, swIf.linkDuplex, swIf.linkSpeed, swIf.subId, swIf.subDot1ad, - swIf.subNumberOfTags, + swIf.subNumberOfTags, swIf.subOuterVlanId, swIf.subInnerVlanId, - swIf.subExactMatch, swIf.subDefault, + swIf.subExactMatch, swIf.subDefault, swIf.subOuterVlanIdAny, swIf.subInnerVlanIdAny, swIf.vtrOp, swIf.vtrPushDot1q, diff --git a/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poProvider.java b/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poProvider.java index 3b546f16f..263e255c2 100644 --- a/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poProvider.java +++ b/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poProvider.java @@ -16,11 +16,13 @@ package io.fd.honeycomb.v3po.impl; +import com.google.common.util.concurrent.CheckedFuture; +import com.google.common.util.concurrent.Futures; +import java.io.IOException; import java.util.ArrayList; import java.util.List; import java.util.Timer; import java.util.TimerTask; - import org.opendaylight.controller.md.sal.binding.api.DataBroker; import org.opendaylight.controller.md.sal.binding.api.WriteTransaction; import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType; @@ -64,23 +66,20 @@ import org.openvpp.vppjapi.vppApi; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import com.google.common.util.concurrent.CheckedFuture; -import com.google.common.util.concurrent.Futures; - public class V3poProvider implements BindingAwareProvider, AutoCloseable { private static final Logger LOG = LoggerFactory.getLogger(V3poProvider.class); private RpcRegistration v3poService; private VppIetfInterfaceListener vppInterfaceListener; private VppBridgeDomainListener vppBridgeDomainListener; - private static final vppApi api = new vppApi(); - private static DataBroker db; + private vppApi api; + private DataBroker db; VppPollOperDataImpl vppPollOperData; - private void writeToBridgeDomain(String bdName, Boolean flood, - Boolean forward, Boolean learn, - Boolean unknownUnicastFlood, - Boolean arpTermination) { + private void writeToBridgeDomain(final String bdName, final Boolean flood, + final Boolean forward, final Boolean learn, + final Boolean unknownUnicastFlood, + final Boolean arpTermination) { BridgeDomainBuilder bdBuilder = new BridgeDomainBuilder(); bdBuilder.setName(bdName); @@ -91,7 +90,7 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { bdBuilder.setArpTermination(arpTermination); LOG.info("VPPCFG-INFO: Adding Bridge Domain " + bdName + " to DataStore."); - InstanceIdentifier iid = + InstanceIdentifier iid = InstanceIdentifier.create(Vpp.class) .child(BridgeDomains.class) .child(BridgeDomain.class, new BridgeDomainKey(bdName)); @@ -102,7 +101,7 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { "VPPCFG-WARNING: Failed to write bridge domain " + bdName + " to Bridge Domains", LOG)); } - private void writeIpv4AddressToInterface(String name, Ipv4AddressNoZone ipv4Addr, short plen) { + private void writeIpv4AddressToInterface(final String name, final Ipv4AddressNoZone ipv4Addr, final short plen) { AddressKey addrKey = new AddressKey(ipv4Addr); AddressBuilder addrBuilder = new AddressBuilder(); PrefixLength prefixLen = new PrefixLengthBuilder().setPrefixLength(plen).build(); @@ -122,7 +121,7 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { ifBuilder.addAugmentation(Interface1.class, if1Builder.build()); LOG.info("VPPCFG-INFO: Adding ipv4 address {} to interface {} to DataStore.", ipv4Addr, name); - InstanceIdentifier iid = + InstanceIdentifier iid = InstanceIdentifier.create(Interfaces.class) .child(Interface.class, new InterfaceKey(name)); WriteTransaction transaction = db.newWriteOnlyTransaction(); @@ -132,8 +131,8 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { "VPPCFG-WARNING: Failed to write " + name + "interface to ietf-interfaces", LOG)); } - private void writeToInterface(String name, String description, - Boolean enabled, String bdName, int vrfId) { + private void writeToInterface(final String name, final String description, + final Boolean enabled, final String bdName, final int vrfId) { VppInterfaceAugmentationBuilder ifAugBuilder = new VppInterfaceAugmentationBuilder(); EthernetBuilder ethBuilder = new EthernetBuilder(); @@ -145,7 +144,7 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { bridgeBuilder.setBridgeDomain(bdName); bridgeBuilder.setSplitHorizonGroup((short)0); bridgeBuilder.setBridgedVirtualInterface(false); - + L2Builder l2Builder = new L2Builder(); l2Builder.setInterconnection(bridgeBuilder.build()); ifAugBuilder.setL2(l2Builder.build()); @@ -183,7 +182,7 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { Vpp vpp = new VppBuilder().build(); transaction.put(LogicalDatastoreType.CONFIGURATION, viid, vpp); CheckedFuture future = transaction.submit(); - Futures.addCallback(future, new + Futures.addCallback(future, new LoggingFuturesCallBack<>("VPPCFG-WARNING: Failed to create Vpp " + "configuration db.", LOG)); @@ -195,7 +194,7 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { Interfaces intf = new InterfacesBuilder().build(); transaction.put(LogicalDatastoreType.CONFIGURATION, iid, intf); future = transaction.submit(); - Futures.addCallback(future, new + Futures.addCallback(future, new LoggingFuturesCallBack<>("VPPCFG-WARNING: Failed to create IETF " + "Interface list configuration db.", LOG)); @@ -221,7 +220,7 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { false /*arpTermination*/); - writeToInterface("TenGigabitEthernet86/0/1", + writeToInterface("TenGigabitEthernet86/0/1", "Physical 10GbE Interface (Transport)", true, null, 7); writeToInterface("TenGigabitEthernet86/0/0", "Physical 10GbE Interface", @@ -284,10 +283,10 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { Futures.addCallback(tx.submit(), new LoggingFuturesCallBack<>( "VPPOPER-WARNING: Failed to create IETF " + "Interface state list operational db.", LOG)); } - + private void startOperationalUpdateTimer() { Timer timer = new Timer(); - + // fire task after 1 second and then repeat each 10 seconds timer.scheduleAtFixedRate(new TimerTask() { @Override @@ -298,19 +297,23 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { } @Override - public void onSessionInitiated(ProviderContext session) { + public void onSessionInitiated(final ProviderContext session) { LOG.info("VPP-INFO: V3poProvider Session Initiated"); - int rv = api.clientConnect("v3poODL"); - LOG.info("VPP-INFO: VPP api client connection return value = {}", rv); - if (rv != 0) { - LOG.error("VPP-ERROR: VPP api client connection failed: return value = {}", rv); + + try { + api = new vppApi("v3poODL"); + } catch (IOException e) { + LOG.error("VPP-ERROR: VPP api client connection failed", e); return; } + + LOG.info("VPP-INFO: VPP api client connection established"); + db = session.getSALService(DataBroker.class); initializeVppConfig(); initVppOperational(); - vppPollOperData = new VppPollOperDataImpl(db); + vppPollOperData = new VppPollOperDataImpl(api, db); v3poService = session.addRpcImplementation(V3poService.class, vppPollOperData); startOperationalUpdateTimer(); @@ -322,5 +325,8 @@ public class V3poProvider implements BindingAwareProvider, AutoCloseable { if (v3poService != null) { v3poService.close(); } + if (api != null) { + api.close(); + } } } diff --git a/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poRequest.java b/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poRequest.java index 900161577..43735ed1d 100644 --- a/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poRequest.java +++ b/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poRequest.java @@ -16,13 +16,16 @@ package io.fd.honeycomb.v3po.impl; -import org.openvpp.vppjapi.vppApi; import org.slf4j.Logger; import org.slf4j.LoggerFactory; /* * Abstract class overriding all callbacks with default error message */ -public abstract class V3poRequest extends vppApi { +public abstract class V3poRequest { private static final Logger LOG = LoggerFactory.getLogger(V3poRequest.class); + + protected V3poRequest() { + + } } diff --git a/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/VppPollOperDataImpl.java b/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/VppPollOperDataImpl.java index 71d4bea4b..6e2ba6e52 100644 --- a/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/VppPollOperDataImpl.java +++ b/v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/VppPollOperDataImpl.java @@ -16,11 +16,10 @@ package io.fd.honeycomb.v3po.impl; -import java.util.HashMap; -import java.util.concurrent.Future; - import com.google.common.util.concurrent.CheckedFuture; import com.google.common.util.concurrent.Futures; +import java.util.HashMap; +import java.util.concurrent.Future; import org.opendaylight.controller.md.sal.binding.api.DataBroker; import org.opendaylight.controller.md.sal.binding.api.WriteTransaction; import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType; @@ -32,6 +31,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.v3po.rev import org.opendaylight.yangtools.yang.binding.InstanceIdentifier; import org.opendaylight.yangtools.yang.common.RpcResult; import org.opendaylight.yangtools.yang.common.RpcResultBuilder; +import org.openvpp.vppjapi.vppApi; import org.openvpp.vppjapi.vppBridgeDomainDetails; import org.openvpp.vppjapi.vppBridgeDomainInterfaceDetails; import org.openvpp.vppjapi.vppL2Fib; @@ -42,14 +42,17 @@ import org.slf4j.LoggerFactory; public class VppPollOperDataImpl implements V3poService { private static final Logger LOG = LoggerFactory.getLogger(VppPollOperDataImpl.class); private vppVersion version; - private DataBroker db; + private final DataBroker db; private int[] bdIds; - private HashMap l2fibByBdId = new HashMap(); + private final HashMap l2fibByBdId = new HashMap(); + private final vppApi api; /** * TODO-ADD-JAVADOC. + * @param api */ - public VppPollOperDataImpl(DataBroker dataBroker) { + public VppPollOperDataImpl(final vppApi api, final DataBroker dataBroker) { + this.api = api; db = dataBroker; } @@ -66,79 +69,77 @@ public class VppPollOperDataImpl implements V3poService { * interfaces names */ public String updateOperational() { - V3poApiRequest api = new V3poApiRequest(this); + V3poApiRequest req = new V3poApiRequest(api, this); version = api.getVppVersion(); - + bdIds = api.bridgeDomainDump(-1); - + // TODO: we don't need to cache BDs now that we got rid of callbacks l2fibByBdId.clear(); if (bdIds != null) { - for (int idx = 0; idx < bdIds.length; idx++) { - l2fibByBdId.put(bdIds[idx], api.l2FibTableDump(bdIds[idx])); + for (int bdId : bdIds) { + l2fibByBdId.put(bdId, api.l2FibTableDump(bdId)); } } - api.swInterfaceDumpAll(); - + req.swInterfaceDumpAll(); + // build vpp-state VppStateCustomBuilder stateBuilder = new VppStateCustomBuilder(); - + // bridge domains - for (int i = 0; i < bdIds.length; i++) { - vppBridgeDomainDetails bd = api.getBridgeDomainDetails(bdIds[i]); - VppStateBridgeDomainBuilder bdBuilder = + for (int bdId : bdIds) { + vppBridgeDomainDetails bd = api.getBridgeDomainDetails(bdId); + VppStateBridgeDomainBuilder bdBuilder = new VppStateBridgeDomainBuilder( - bd.name, bd.flood, bd.uuFlood, + bd.name, bd.flood, bd.uuFlood, bd.arpTerm, bd.forward, bd.learn); - - for (int ifIdx = 0; ifIdx < bd.interfaces.length; ifIdx++) { - vppBridgeDomainInterfaceDetails bdIf = bd.interfaces[ifIdx]; - bdBuilder.addInterface(bdIf.interfaceName, - bd.bviInterfaceName == bdIf.interfaceName, + + for (vppBridgeDomainInterfaceDetails bdIf : bd.interfaces) { + bdBuilder.addInterface(bdIf.interfaceName, + bd.bviInterfaceName == bdIf.interfaceName, bdIf.splitHorizonGroup); } - - vppL2Fib[] bdFibs = l2fibByBdId.get(bdIds[i]); - for (int fibIdx = 0; fibIdx < bdFibs.length; fibIdx++) { - vppL2Fib fib = bdFibs[fibIdx]; + vppL2Fib[] bdFibs = l2fibByBdId.get(bdId); + + for (vppL2Fib fib : bdFibs) { bdBuilder.addL2Fib(fib.filter, fib.bridgedVirtualInterface, fib.outgoingInterface, fib.physAddress, fib.staticConfig); } - + stateBuilder.addBridgeDomain(bdBuilder.build()); } - + stateBuilder.setVersion(version); - + // write to oper writeVppState(getVppStateIid(), stateBuilder.build()); - - return api.ifNames; + + return req.ifNames; } - + @Override public Future> vppPollOperData() { String ifNames = updateOperational(); - + VppPollOperDataOutput output = new VppPollOperDataOutputBuilder() .setStatus(new Long(1)).build(); return RpcResultBuilder.success(output).buildFuture(); } - + private InstanceIdentifier getVppStateIid() { return (InstanceIdentifier.create(VppState.class)); } - - private void writeVppState(InstanceIdentifier iid, VppState vppState) { + + private void writeVppState(final InstanceIdentifier iid, final VppState vppState) { WriteTransaction transaction = db.newWriteOnlyTransaction(); //LOG.info("VPPOPER-INFO: Writing vpp-state to oper DataStore."); transaction.put(LogicalDatastoreType.OPERATIONAL, iid, vppState); - + CheckedFuture future = transaction.submit(); Futures.addCallback(future, new LoggingFuturesCallBack( "VPPOPER-WARNING: Failed to write vpp-state to oper datastore", LOG)); -- cgit 1.2.3-korg