commit edf9c4de64497d7b5674886bf17f056f1151d5ec
author Thomas Vachuska <tom@opennetworking.org> Mon Oct 15 19:14:19 2018 -0700
committer Thomas Vachuska <tom@opennetworking.org> Mon Oct 15 19:14:19 2018 -0700
tree 63d377da2ab9bf689ddb1cff5ce3fd7b79e8c3ba
parent d0d7ea101e0451c453f497f880afe519729ec920

Fixing component properties in incubator.

Change-Id: I6479271c4465300b561994ba9d3aab430fab1db6

incubator/store/src/main/java/org/onosproject/incubator/store/virtual/impl/SimpleVirtualFlowRuleStore.java

diff --git a/incubator/store/src/main/java/org/onosproject/incubator/store/virtual/impl/SimpleVirtualFlowRuleStore.java b/incubator/store/src/main/java/org/onosproject/incubator/store/virtual/impl/SimpleVirtualFlowRuleStore.java
index c12044d..c2edd81 100644
--- a/incubator/store/src/main/java/org/onosproject/incubator/store/virtual/impl/SimpleVirtualFlowRuleStore.java
+++ b/incubator/store/src/main/java/org/onosproject/incubator/store/virtual/impl/SimpleVirtualFlowRuleStore.java
@@ -62,6 +62,7 @@
 import java.util.concurrent.TimeoutException;
 import java.util.concurrent.atomic.AtomicInteger;
 
+import static org.onosproject.incubator.store.virtual.impl.OsgiPropertyDefaults.PENDING_FUTURE_TIMEOUT_MINUTES_DEFAULT;
 import static org.onosproject.net.flow.FlowRuleEvent.Type.RULE_REMOVED;
 import static org.slf4j.LoggerFactory.getLogger;
 
@@ -71,7 +72,10 @@
  */
 //TODO: support distributed flowrule store for virtual networks
 
-@Component(immediate = true, service = VirtualNetworkFlowRuleStore.class)
+@Component(immediate = true, service = VirtualNetworkFlowRuleStore.class,
+        property = {
+                 "pendingFutureTimeoutMinutes:Integer=" + PENDING_FUTURE_TIMEOUT_MINUTES_DEFAULT,
+        })
 public class SimpleVirtualFlowRuleStore
         extends AbstractVirtualStore<FlowRuleBatchEvent, FlowRuleStoreDelegate>
         implements VirtualNetworkFlowRuleStore {
@@ -88,10 +92,9 @@
 
     private final AtomicInteger localBatchIdGen = new AtomicInteger();
 
-    private static final int DEFAULT_PENDING_FUTURE_TIMEOUT_MINUTES = 5;
     //@Property(name = "pendingFutureTimeoutMinutes", intValue = DEFAULT_PENDING_FUTURE_TIMEOUT_MINUTES,
     //        label = "Expiration time after an entry is created that it should be automatically removed")
-    private int pendingFutureTimeoutMinutes = DEFAULT_PENDING_FUTURE_TIMEOUT_MINUTES;
+    private int pendingFutureTimeoutMinutes = PENDING_FUTURE_TIMEOUT_MINUTES_DEFAULT;
 
     private Cache<Integer, SettableFuture<CompletedBatchOperation>> pendingFutures =
             CacheBuilder.newBuilder()
@@ -136,7 +139,7 @@
         Integer newPendingFutureTimeoutMinutes =
                 Tools.getIntegerProperty(properties, "pendingFutureTimeoutMinutes");
         if (newPendingFutureTimeoutMinutes == null) {
-            pendingFutureTimeoutMinutes = DEFAULT_PENDING_FUTURE_TIMEOUT_MINUTES;
+            pendingFutureTimeoutMinutes = PENDING_FUTURE_TIMEOUT_MINUTES_DEFAULT;
             log.info("Pending future timeout is not configured, " +
                              "using current value of {}", pendingFutureTimeoutMinutes);
         } else {