[VOL-3590] Synchronizing read ans subsequent write to the pending Meters map to avoid duplicate meters
Change-Id: I7c08a642dd7183f5bed2da86f59f53a30865790a
diff --git a/app/src/main/java/org/opencord/olt/impl/Olt.java b/app/src/main/java/org/opencord/olt/impl/Olt.java
index d572657..3323480 100644
--- a/app/src/main/java/org/opencord/olt/impl/Olt.java
+++ b/app/src/main/java/org/opencord/olt/impl/Olt.java
@@ -797,11 +797,10 @@
}
}
private void checkAndCreateDevMeter(DeviceId deviceId, BandwidthProfileInformation bwpInfo) {
- if (oltMeterService.isMeterPending(deviceId, bwpInfo)) {
- log.debug("Meter is already pending {} on device {}", bwpInfo, deviceId);
+ //If false the meter is already being installed, skipping installation
+ if (!oltMeterService.checkAndAddPendingMeter(deviceId, bwpInfo)) {
return;
}
- oltMeterService.addToPendingMeters(deviceId, bwpInfo);
createMeter(deviceId, bwpInfo);
}
diff --git a/app/src/main/java/org/opencord/olt/impl/OltFlowService.java b/app/src/main/java/org/opencord/olt/impl/OltFlowService.java
index f934c54..6ae705a 100644
--- a/app/src/main/java/org/opencord/olt/impl/OltFlowService.java
+++ b/app/src/main/java/org/opencord/olt/impl/OltFlowService.java
@@ -473,12 +473,10 @@
null, bpInfo.id());
pendingEapolForMeters.add(fi);
- if (oltMeterService.isMeterPending(devId, bpInfo)) {
- log.debug("Meter is already pending for EAPOL on {} with bp {}",
- devId, bpInfo);
+ //If false the meter is already being installed, skipping installation
+ if (!oltMeterService.checkAndAddPendingMeter(devId, bpInfo)) {
return;
}
- oltMeterService.addToPendingMeters(devId, bpInfo);
MeterId innerMeterId = oltMeterService.createMeter(devId, bpInfo,
meterFuture);
fi.setUpMeterId(innerMeterId);
diff --git a/app/src/main/java/org/opencord/olt/impl/OltMeterService.java b/app/src/main/java/org/opencord/olt/impl/OltMeterService.java
index 7616141..478e280 100644
--- a/app/src/main/java/org/opencord/olt/impl/OltMeterService.java
+++ b/app/src/main/java/org/opencord/olt/impl/OltMeterService.java
@@ -256,20 +256,6 @@
}
@Override
- public void addToPendingMeters(DeviceId deviceId, BandwidthProfileInformation bwpInfo) {
- if (deviceId == null) {
- return;
- }
- pendingMeters.compute(deviceId, (id, bwps) -> {
- if (bwps == null) {
- bwps = new HashSet<>();
- }
- bwps.add(bwpInfo);
- return bwps;
- });
- }
-
- @Override
public void removeFromPendingMeters(DeviceId deviceId, BandwidthProfileInformation bwpInfo) {
if (deviceId == null) {
return;
@@ -281,11 +267,24 @@
}
@Override
- public boolean isMeterPending(DeviceId deviceId, BandwidthProfileInformation bwpInfo) {
- if (!pendingMeters.containsKey(deviceId)) {
+ public synchronized boolean checkAndAddPendingMeter(DeviceId deviceId, BandwidthProfileInformation bwpInfo) {
+ if (pendingMeters.containsKey(deviceId)
+ && pendingMeters.get(deviceId).contains(bwpInfo)) {
+ log.debug("Meter is already pending for EAPOL on {} with bp {}",
+ deviceId, bwpInfo);
return false;
}
- return pendingMeters.get(deviceId).contains(bwpInfo);
+ log.debug("Adding bandwidth profile {} to pending on {}",
+ bwpInfo, deviceId);
+ pendingMeters.compute(deviceId, (id, bwps) -> {
+ if (bwps == null) {
+ bwps = new HashSet<>();
+ }
+ bwps.add(bwpInfo);
+ return bwps;
+ });
+
+ return true;
}
@Override