commit 507a2a082935d21b6ffb12aa877f644c09fd8eac
author Matteo Scandolo <teo@opennetworking.org> Mon Sep 17 15:08:25 2018 -0700
committer Matteo Scandolo <teo@opennetworking.org> Tue Sep 18 13:53:22 2018 -0700
tree 1199fc6b6af4bae34eda2a3cecf078b337398755
parent c6ac74aed0b5a6b3677162c4fd081d0a884a84d9

Do not disable ONU as result of an onu.disabled event,
save the subscriber from dhcp only if something has changed.

Change-Id: I4a97bb7fe2e08e408341aa36617709b554e9e5bb

xos/synchronizer/event_steps/dhcp_event.py

diff --git a/xos/synchronizer/event_steps/dhcp_event.py b/xos/synchronizer/event_steps/dhcp_event.py
index d6a789f..eec273d 100644
--- a/xos/synchronizer/event_steps/dhcp_event.py
+++ b/xos/synchronizer/event_steps/dhcp_event.py
@@ -47,6 +47,9 @@
 
         # NOTE it will be better to update the SI and use the model policy to update the subscriber,
         # if this fails for any reason the event is lost
-        subscriber.ip_address = value["ipAddress"]
-        subscriber.mac_address = value["macAddress"]
-        subscriber.save()
+        if subscriber.ip_address != value["ipAddress"] or \
+            subscriber.mac_address != value["macAddress"]:
+
+            subscriber.ip_address = value["ipAddress"]
+            subscriber.mac_address = value["macAddress"]
+            subscriber.save()