Modify a annotation and refactor Web UI.
Change-Id: I985733e8f4e7d0cda292ed4a240d1250db8e2316
diff --git a/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/DefaultFlowInfo.java b/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/DefaultFlowInfo.java
index 8560170..b0d3aad 100644
--- a/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/DefaultFlowInfo.java
+++ b/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/DefaultFlowInfo.java
@@ -27,6 +27,7 @@
import static com.google.common.base.MoreObjects.toStringHelper;
import static com.google.common.base.Preconditions.checkNotNull;
+import static org.onosproject.openstacktelemetry.util.OpenstackTelemetryUtil.getProtocolNameFromType;
/**
* Implementation class of FlowInfo.
@@ -200,7 +201,8 @@
((srcPort == null) ? "any" : srcPort.toString()) +
" -> " +
dstIp.toString() + ":" +
- ((dstPort == null) ? "any" : dstPort.toString());
+ ((dstPort == null) ? "any" : dstPort.toString()) +
+ " (" + getProtocolNameFromType(protocol) + ")";
}
@Override
diff --git a/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/OsgiPropertyConstants.java b/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/OsgiPropertyConstants.java
index 7be6973..325bce2 100644
--- a/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/OsgiPropertyConstants.java
+++ b/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/OsgiPropertyConstants.java
@@ -141,11 +141,11 @@
// Prometheus Telemetry config manager
static final String PROP_PROMETHEUS_ENABLE_SERVICE = "enableService";
- static final boolean PROMETHEUS_ENABLE_SERVICE_DEFAULT = false;
+ static final boolean PROP_PROMETHEUS_ENABLE_SERVICE_DEFAULT = true;
static final String PROP_PROMETHEUS_EXPORTER_ADDRESS = "address";
- public static final String PROMETHEUS_EXPORTER_ADDRESS_DEFAULT = "localhost";
+ public static final String PROP_PROMETHEUS_EXPORTER_ADDRESS_DEFAULT = "localhost";
static final String PROP_PROMETHEUS_EXPORTER_PORT = "port";
- public static final int PROMETHEUS_EXPORTER_PORT_DEFAULT = 50051;
+ public static final int PROP_PROMETHEUS_EXPORTER_PORT_DEFAULT = 9555;
}
diff --git a/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/PrometheusTelemetryConfigManager.java b/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/PrometheusTelemetryConfigManager.java
index 4405b97..dcc7023 100644
--- a/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/PrometheusTelemetryConfigManager.java
+++ b/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/PrometheusTelemetryConfigManager.java
@@ -33,9 +33,9 @@
import java.util.Dictionary;
-import static org.onosproject.openstacktelemetry.impl.OsgiPropertyConstants.PROMETHEUS_ENABLE_SERVICE_DEFAULT;
-import static org.onosproject.openstacktelemetry.impl.OsgiPropertyConstants.PROMETHEUS_EXPORTER_ADDRESS_DEFAULT;
-import static org.onosproject.openstacktelemetry.impl.OsgiPropertyConstants.PROMETHEUS_EXPORTER_PORT_DEFAULT;
+import static org.onosproject.openstacktelemetry.impl.OsgiPropertyConstants.PROP_PROMETHEUS_ENABLE_SERVICE_DEFAULT;
+import static org.onosproject.openstacktelemetry.impl.OsgiPropertyConstants.PROP_PROMETHEUS_EXPORTER_ADDRESS_DEFAULT;
+import static org.onosproject.openstacktelemetry.impl.OsgiPropertyConstants.PROP_PROMETHEUS_EXPORTER_PORT_DEFAULT;
import static org.onosproject.openstacktelemetry.impl.OsgiPropertyConstants.PROP_PROMETHEUS_ENABLE_SERVICE;
import static org.onosproject.openstacktelemetry.impl.OsgiPropertyConstants.PROP_PROMETHEUS_EXPORTER_ADDRESS;
import static org.onosproject.openstacktelemetry.impl.OsgiPropertyConstants.PROP_PROMETHEUS_EXPORTER_PORT;
@@ -49,9 +49,9 @@
immediate = true,
service = PrometheusTelemetryConfigService.class,
property = {
- PROP_PROMETHEUS_ENABLE_SERVICE + ":Boolean=" + PROMETHEUS_ENABLE_SERVICE_DEFAULT,
- PROP_PROMETHEUS_EXPORTER_ADDRESS + "=" + PROMETHEUS_EXPORTER_ADDRESS_DEFAULT,
- PROP_PROMETHEUS_EXPORTER_PORT + ":Integer=" + PROMETHEUS_EXPORTER_PORT_DEFAULT
+ PROP_PROMETHEUS_ENABLE_SERVICE + ":Boolean=" + PROP_PROMETHEUS_ENABLE_SERVICE_DEFAULT,
+ PROP_PROMETHEUS_EXPORTER_ADDRESS + "=" + PROP_PROMETHEUS_EXPORTER_ADDRESS_DEFAULT,
+ PROP_PROMETHEUS_EXPORTER_PORT + ":Integer=" + PROP_PROMETHEUS_EXPORTER_PORT_DEFAULT
}
)
public class PrometheusTelemetryConfigManager implements PrometheusTelemetryConfigService {
@@ -65,13 +65,13 @@
protected PrometheusTelemetryAdminService prometheusTelemetryAdminService;
/** Default IP address of prometheus exporter. */
- protected String address = PROMETHEUS_EXPORTER_ADDRESS_DEFAULT;
+ protected String address = PROP_PROMETHEUS_EXPORTER_ADDRESS_DEFAULT;
/** Default port number of prometheus exporter. */
- protected Integer port = PROMETHEUS_EXPORTER_PORT_DEFAULT;
+ protected Integer port = PROP_PROMETHEUS_EXPORTER_PORT_DEFAULT;
/** Specify the default behavior of telemetry service. */
- protected Boolean enableService = PROMETHEUS_ENABLE_SERVICE_DEFAULT;
+ protected Boolean enableService = PROP_PROMETHEUS_ENABLE_SERVICE_DEFAULT;
@Activate
protected void activate(ComponentContext context) {
@@ -115,12 +115,12 @@
Dictionary<?, ?> properties = context.getProperties();
String addressStr = Tools.get(properties, PROP_PROMETHEUS_EXPORTER_ADDRESS);
- address = addressStr != null ? addressStr : PROMETHEUS_EXPORTER_ADDRESS_DEFAULT;
+ address = addressStr != null ? addressStr : PROP_PROMETHEUS_EXPORTER_ADDRESS_DEFAULT;
log.info("Configured. Prometheus exporter address is {}", address);
Integer portConfigured = Tools.getIntegerProperty(properties, PROP_PROMETHEUS_EXPORTER_PORT);
if (portConfigured == null) {
- port = PROMETHEUS_EXPORTER_PORT_DEFAULT;
+ port = PROP_PROMETHEUS_EXPORTER_PORT_DEFAULT;
log.info("Prometheus exporter port is NOT configured, default value is {}", port);
} else {
port = portConfigured;
@@ -129,7 +129,7 @@
Boolean enableServiceConfigured = getBooleanProperty(properties, PROP_PROMETHEUS_ENABLE_SERVICE);
if (enableServiceConfigured == null) {
- enableService = PROMETHEUS_ENABLE_SERVICE_DEFAULT;
+ enableService = PROP_PROMETHEUS_ENABLE_SERVICE_DEFAULT;
log.info("Prometheus service enable flag is NOT " +
"configured, default value is {}", enableService);
} else {
diff --git a/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/PrometheusTelemetryManager.java b/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/PrometheusTelemetryManager.java
index f029fa4..69423ca 100644
--- a/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/PrometheusTelemetryManager.java
+++ b/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/PrometheusTelemetryManager.java
@@ -15,10 +15,11 @@
*/
package org.onosproject.openstacktelemetry.impl;
+import io.prometheus.client.Gauge;
+import org.onlab.packet.TpPort;
import org.onosproject.openstacktelemetry.api.FlowInfo;
import org.onosproject.openstacktelemetry.api.OpenstackTelemetryService;
import org.onosproject.openstacktelemetry.api.PrometheusTelemetryAdminService;
-import org.onosproject.openstacktelemetry.api.PrometheusTelemetryService;
import org.onosproject.openstacktelemetry.api.config.PrometheusTelemetryConfig;
import org.onosproject.openstacktelemetry.api.config.TelemetryConfig;
import org.osgi.service.component.annotations.Activate;
@@ -34,76 +35,101 @@
import org.eclipse.jetty.server.Server;
import org.eclipse.jetty.servlet.ServletContextHandler;
import org.eclipse.jetty.servlet.ServletHolder;
+
+import java.util.Arrays;
import java.util.Set;
/**
* Prometheus telemetry manager.
*/
-@Component(immediate = true, service = PrometheusTelemetryService.class)
+@Component(immediate = true, service = PrometheusTelemetryAdminService.class)
public class PrometheusTelemetryManager implements PrometheusTelemetryAdminService {
private final Logger log = LoggerFactory.getLogger(getClass());
private Server prometheusExporter;
- private static final String BYTE_VM2VM = "byte_vm2vm";
- private static final String BYTE_DEVICE = "byte_device";
- private static final String BYTE_SRC_IP = "byte_src_ip";
- private static final String BYTE_DST_IP = "byte_dst_ip";
+ private static final String FLOW_TYPE = "flowType";
+ private static final String DEVICE_ID = "deviceId";
+ private static final String INPUT_INTERFACE_ID = "inputInterfaceId";
+ private static final String OUTPUT_INTERFACE_ID = "outputInterfaceId";
+ private static final String VLAN_ID = "vlanId";
+ private static final String VXLAN_ID = "vxlanId";
+ private static final String SRC_IP = "srcIp";
+ private static final String DST_IP = "dstIp";
+ private static final String SRC_PORT = "srcPort";
+ private static final String DST_PORT = "dstPort";
+ private static final String PROTOCOL = "protocol";
- private static final String PKT_VM2VM = "pkt_vm2vm";
- private static final String PKT_DEVICE = "pkt_device";
- private static final String PKT_SRC_IP = "pkt_src_ip";
- private static final String PKT_DST_IP = "pkt_dst_ip";
+ private static final String[] LABEL_TAGS = {
+ FLOW_TYPE, DEVICE_ID, INPUT_INTERFACE_ID, OUTPUT_INTERFACE_ID,
+ VLAN_ID, VXLAN_ID, SRC_IP, DST_IP, SRC_PORT, DST_PORT, PROTOCOL
+ };
- private static final String PKT_ERROR = "pkt_error";
- private static final String PKT_DROP = "pkt_drop";
+ private static final String STAT_NAME_VM2VM_BYTE = "vm2vm_byte";
+ private static final String STAT_NAME_VM2VM_BYTE_PREV = "vm2vm_byte_prev";
+ private static final String STAT_NAME_VM2VM_BYTE_CURR = "vm2vm_byte_curr";
+ private static final String STAT_NAME_VM2VM_PKT = "vm2vm_pkt";
+ private static final String STAT_NAME_VM2VM_PKT_PREV = "vm2vm_pkt_prev";
+ private static final String STAT_NAME_VM2VM_PKT_CURR = "vm2vm_pkt_curr";
+ private static final String STAT_NAME_ERROR_PKT = "error_pkt";
+ private static final String STAT_NAME_DROP_PKT = "drop_pkt";
- private static final String LABEL_IP_5_TUPLE = "IP_5_TUPLE";
- private static final String LABEL_DEV_ID = "DEVICE_ID";
- private static final String LABEL_SRC_IP = "SOURCE_IP";
- private static final String LABEL_DST_IP = "DESTINATION_IP";
+ private static final String HELP_MSG_VM2VM_BYTE =
+ "SONA flow bytes statistics for VM to VM";
+ private static final String HELP_MSG_VM2VM_BYTE_PREV =
+ HELP_MSG_VM2VM_BYTE + " [Accumulated previous byte]";
+ private static final String HELP_MSG_VM2VM_BYTE_CURR =
+ HELP_MSG_VM2VM_BYTE + " [Accumulated current byte]";
+ private static final String HELP_MSG_VM2VM_PKT =
+ "SONA flow packets statistics for VM to VM";
+ private static final String HELP_MSG_VM2VM_PKT_PREV =
+ HELP_MSG_VM2VM_PKT + " [Accumulated previous pkt]";
+ private static final String HELP_MSG_VM2VM_PKT_CURR =
+ HELP_MSG_VM2VM_PKT + " [Accumulated current pkt]";
+ private static final String HELP_MSG_ERROR = "SONA error statistics";
+ private static final String HELP_MSG_DROP = "SONA drop statistics";
- private static final String HELP_MSG = "SONA Flow statistics";
-
- private static Counter byteVM2VM = Counter.build().name(BYTE_VM2VM)
- .help(HELP_MSG)
- .labelNames(LABEL_IP_5_TUPLE).register();
-
- private static Counter byteDevice = Counter.build().name(BYTE_DEVICE)
- .help(HELP_MSG)
- .labelNames(LABEL_DEV_ID).register();
-
- private static Counter byteSrcIp = Counter.build().name(BYTE_SRC_IP)
- .help(HELP_MSG)
- .labelNames(LABEL_SRC_IP).register();
-
- private static Counter byteDstIp = Counter.build().name(BYTE_DST_IP)
- .help(HELP_MSG)
- .labelNames(LABEL_DST_IP).register();
-
- private static Counter pktVM2VM = Counter.build().name(PKT_VM2VM)
- .help(HELP_MSG)
- .labelNames(LABEL_IP_5_TUPLE).register();
-
- private static Counter pktDevice = Counter.build().name(PKT_DEVICE)
- .help(HELP_MSG)
- .labelNames(LABEL_DEV_ID).register();
-
- private static Counter pktSrcIp = Counter.build().name(PKT_SRC_IP)
- .help(HELP_MSG)
- .labelNames(LABEL_SRC_IP).register();
-
- private static Counter pktDstIp = Counter.build().name(PKT_DST_IP)
- .help(HELP_MSG)
- .labelNames(LABEL_DST_IP).register();
-
- private static Counter pktError = Counter.build().name(PKT_ERROR)
- .help(HELP_MSG)
- .register();
- private static Counter pktDrop = Counter.build().name(PKT_DROP)
- .help(HELP_MSG)
- .register();
+ private static Gauge byteVM2VM = Gauge.build()
+ .name(STAT_NAME_VM2VM_BYTE)
+ .help(HELP_MSG_VM2VM_BYTE)
+ .labelNames(LABEL_TAGS)
+ .register();
+ private static Gauge byteVM2VMPrev = Gauge.build()
+ .name(STAT_NAME_VM2VM_BYTE_PREV)
+ .help(HELP_MSG_VM2VM_BYTE_PREV)
+ .labelNames(LABEL_TAGS)
+ .register();
+ private static Gauge byteVM2VMCurr = Gauge.build()
+ .name(STAT_NAME_VM2VM_BYTE_CURR)
+ .help(HELP_MSG_VM2VM_BYTE_CURR)
+ .labelNames(LABEL_TAGS)
+ .register();
+ private static Gauge pktVM2VM = Gauge.build()
+ .name(STAT_NAME_VM2VM_PKT)
+ .help(HELP_MSG_VM2VM_PKT)
+ .labelNames(LABEL_TAGS)
+ .register();
+ private static Gauge pktVM2VMPrev = Gauge.build()
+ .name(STAT_NAME_VM2VM_PKT_PREV)
+ .help(HELP_MSG_VM2VM_PKT_PREV)
+ .labelNames(LABEL_TAGS)
+ .register();
+ private static Gauge pktVM2VMCurr = Gauge.build()
+ .name(STAT_NAME_VM2VM_PKT_CURR)
+ .help(HELP_MSG_VM2VM_PKT_CURR)
+ .labelNames(LABEL_TAGS)
+ .register();
+ private static Counter pktError = Counter.build()
+ .name(STAT_NAME_ERROR_PKT)
+ .help(HELP_MSG_ERROR)
+ .labelNames(LABEL_TAGS)
+ .register();
+ private static Counter pktDrop = Counter.build()
+ .name(STAT_NAME_DROP_PKT)
+ .help(HELP_MSG_DROP)
+ .labelNames(LABEL_TAGS)
+ .register();
@Reference(cardinality = ReferenceCardinality.MANDATORY)
protected OpenstackTelemetryService openstackTelemetryService;
@@ -128,15 +154,12 @@
PrometheusTelemetryConfig prometheusConfig = (PrometheusTelemetryConfig) config;
try {
- // TODO Offer a 'Authentication'
prometheusExporter = new Server(prometheusConfig.port());
ServletContextHandler context = new ServletContextHandler();
context.setContextPath("/");
prometheusExporter.setHandler(context);
context.addServlet(new ServletHolder(new MetricsServlet()), "/metrics");
-
- log.info("Prometeus server start");
-
+ log.info("Prometeus server start (Server port:{})", prometheusConfig.port());
prometheusExporter.start();
} catch (Exception ex) {
log.warn("Exception: {}", ex.toString());
@@ -161,35 +184,69 @@
@Override
public void publish(Set<FlowInfo> flowInfos) {
- if (flowInfos.size() == 0) {
- log.debug("No record to publish");
+ if (flowInfos.isEmpty()) {
+ log.debug("No FlowInfo record to publish");
return;
}
long flowByte;
int flowPkt;
+ String[] labelValues;
+
for (FlowInfo flowInfo: flowInfos) {
flowByte = flowInfo.statsInfo().currAccBytes() - flowInfo.statsInfo().prevAccBytes();
flowPkt = flowInfo.statsInfo().currAccPkts() - flowInfo.statsInfo().prevAccPkts();
-
- byteVM2VM.labels(flowInfo.uniqueFlowInfoKey()).inc(flowByte);
- byteDevice.labels(flowInfo.deviceId().toString()).inc(flowByte);
- byteSrcIp.labels(flowInfo.srcIp().toString()).inc(flowByte);
- byteDstIp.labels(flowInfo.dstIp().toString()).inc(flowByte);
-
- pktVM2VM.labels(flowInfo.uniqueFlowInfoKey()).inc(flowPkt);
- pktDevice.labels(flowInfo.deviceId().toString()).inc(flowPkt);
- pktSrcIp.labels(flowInfo.srcIp().toString()).inc(flowPkt);
- pktDstIp.labels(flowInfo.dstIp().toString()).inc(flowPkt);
-
- pktError.inc(flowInfo.statsInfo().errorPkts());
- pktDrop.inc(flowInfo.statsInfo().dropPkts());
+ labelValues = getLabelValues(flowInfo);
+ byteVM2VM.labels(labelValues).set(flowByte);
+ byteVM2VMPrev.labels(labelValues).set(flowInfo.statsInfo().prevAccBytes());
+ byteVM2VMCurr.labels(labelValues).set(flowInfo.statsInfo().currAccBytes());
+ pktVM2VM.labels(labelValues).set(flowPkt);
+ pktVM2VMPrev.labels(labelValues).set(flowInfo.statsInfo().prevAccPkts());
+ pktVM2VMCurr.labels(labelValues).set(flowInfo.statsInfo().currAccPkts());
+ pktError.labels(labelValues).inc(flowInfo.statsInfo().errorPkts());
+ pktDrop.labels(labelValues).inc(flowInfo.statsInfo().dropPkts());
}
}
+ private String[] getLabelValues(FlowInfo flowInfo) {
+ String[] labelValues = new String[LABEL_TAGS.length];
+
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(FLOW_TYPE)]
+ = String.valueOf(flowInfo.flowType());
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(DEVICE_ID)]
+ = flowInfo.deviceId().toString();
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(INPUT_INTERFACE_ID)]
+ = String.valueOf(flowInfo.inputInterfaceId());
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(OUTPUT_INTERFACE_ID)]
+ = String.valueOf(flowInfo.outputInterfaceId());
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(VXLAN_ID)]
+ = String.valueOf(flowInfo.vxlanId());
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(SRC_IP)]
+ = flowInfo.srcIp().toString();
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(DST_IP)]
+ = flowInfo.dstIp().toString();
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(SRC_PORT)]
+ = getTpPort(flowInfo.srcPort());
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(DST_PORT)]
+ = getTpPort(flowInfo.dstPort());
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(PROTOCOL)]
+ = String.valueOf(flowInfo.protocol());
+ if (flowInfo.vlanId() != null) {
+ labelValues[Arrays.asList(LABEL_TAGS).indexOf(VLAN_ID)]
+ = flowInfo.vlanId().toString();
+ }
+ return labelValues;
+ }
+
+ private String getTpPort(TpPort tpPort) {
+ if (tpPort == null) {
+ return "";
+ }
+ return tpPort.toString();
+ }
+
@Override
public boolean isRunning() {
- log.info("Prometheus Exporter State: {}", prometheusExporter.isRunning());
return prometheusExporter.isRunning();
}
}
diff --git a/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/StatsFlowRuleManager.java b/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/StatsFlowRuleManager.java
index 4ddaf40..3d9f577 100644
--- a/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/StatsFlowRuleManager.java
+++ b/apps/openstacktelemetry/app/src/main/java/org/onosproject/openstacktelemetry/impl/StatsFlowRuleManager.java
@@ -138,7 +138,7 @@
private final Logger log = LoggerFactory.getLogger(getClass());
- private static final byte FLOW_TYPE_SONA = 1; // VLAN
+ private static final byte FLOW_TYPE_SONA = 1;
private static final long MILLISECONDS = 1000L;
private static final long INITIAL_DELAY = 5L;
@@ -148,7 +148,6 @@
private static final String OVS_DRIVER_NAME = "ovs";
private static final String ARBITRARY_IP = "0.0.0.0/32";
- private static final int ARBITRARY_PROTOCOL = 0x0;
private static final int ARBITRARY_LENGTH = 32;
private static final String ARBITRARY_MAC = "00:00:00:00:00:00";
private static final IpAddress NO_HOST_IP = IpAddress.valueOf("255.255.255.255");
@@ -263,16 +262,6 @@
}
@Override
- public void setStatFlowL2Rule(String srcIp, String dstIp, Boolean install) {
- StatsFlowRule statsFlowRule = DefaultStatsFlowRule.builder()
- .srcIpPrefix(IpPrefix.valueOf(IpAddress.valueOf(srcIp), ARBITRARY_LENGTH))
- .dstIpPrefix(IpPrefix.valueOf(IpAddress.valueOf(dstIp), ARBITRARY_LENGTH))
- .ipProtocol((byte) ARBITRARY_PROTOCOL)
- .build();
- setStatFlowRule(statsFlowRule, install);
- }
-
- @Override
public void createStatFlowRule(StatsFlowRule statsFlowRule) {
setStatFlowRule(statsFlowRule, true);
}
@@ -438,8 +427,10 @@
stats.forEach(s -> {
InstancePort instPort = getInstancePort(d, s.portNumber());
- flowInfos.add(buildTxFlowInfoFromInstancePort(instPort, s));
- flowInfos.add(buildRxFlowInfoFromInstancePort(instPort, s));
+ if (instPort != null) {
+ flowInfos.add(buildTxFlowInfoFromInstancePort(instPort, s));
+ flowInfos.add(buildRxFlowInfoFromInstancePort(instPort, s));
+ }
});
});