[SEBA-505] Do not check for existing ONU on subscriber delete
Change-Id: I4fa8cb00a327aefc37a5ef777d1f5dcff03e1056
diff --git a/.gitreview b/.gitreview
index e8e32bd..ed02026 100644
--- a/.gitreview
+++ b/.gitreview
@@ -3,3 +3,4 @@
port=29418
project=rcord.git
defaultremote=origin
+defaultbranch=1.0
diff --git a/Dockerfile.synchronizer b/Dockerfile.synchronizer
index 5913671..91cde80 100644
--- a/Dockerfile.synchronizer
+++ b/Dockerfile.synchronizer
@@ -16,7 +16,7 @@
# docker build -t xosproject/rcord-synchronizer:candidate -f Dockerfile.synchronizer .
# xosproject/rcord-synchronizer
-FROM xosproject/xos-synchronizer-base:2.1.50
+FROM xosproject/xos-synchronizer-base:2.1.55
COPY xos/synchronizer /opt/xos/synchronizers/rcord
COPY VERSION /opt/xos/synchronizers/rcord/
diff --git a/VERSION b/VERSION
index ce396e1..2ac9634 100644
--- a/VERSION
+++ b/VERSION
@@ -1 +1 @@
-1.0.13-dev
+1.0.13
diff --git a/xos/synchronizer/models/models.py b/xos/synchronizer/models/models.py
index 9713ab4..a9fbace 100644
--- a/xos/synchronizer/models/models.py
+++ b/xos/synchronizer/models/models.py
@@ -115,7 +115,7 @@
self.set_owner()
- if self.status != "pre-provisioned" and hasattr(self.owner.leaf_model, "access") and self.owner.leaf_model.access == "voltha":
+ if self.status != "pre-provisioned" and hasattr(self.owner.leaf_model, "access") and self.owner.leaf_model.access == "voltha" and not self.deleted:
# if the access network is managed by voltha, validate that onu_device actually exist
volt_service = self.owner.provider_services[0].leaf_model # we assume RCORDService is connected only to the vOLTService
diff --git a/xos/synchronizer/models/test_models.py b/xos/synchronizer/models/test_models.py
index 3d3c3f2..406e607 100644
--- a/xos/synchronizer/models/test_models.py
+++ b/xos/synchronizer/models/test_models.py
@@ -67,6 +67,7 @@
self.rcord_subscriber_class = RCORDSubscriber
self.rcord_subscriber = RCORDSubscriber()
+ self.rcord_subscriber.deleted = False
self.rcord_subscriber.id = None # this is a new model
self.rcord_subscriber.is_new = True
self.rcord_subscriber.onu_device = "BRCM1234"
@@ -124,6 +125,12 @@
self.assertEqual(e.exception.message, "The onu_device you specified (BRCM1234) does not exists")
self.models_decl.RCORDSubscriber_decl.save.assert_not_called()
+ def test_missing_onu_device_on_delete(self):
+ self.volt.leaf_model.has_access_device.return_value = False
+ self.rcord_subscriber.deleted = True
+ self.rcord_subscriber.save()
+ self.models_decl.RCORDSubscriber_decl.save.assert_called()
+
def test_validate_c_tag_pass(self):
"""
check that other subscriber attached to the same ONU don't have the same c_tag
@@ -131,7 +138,6 @@
self.models_decl.RCORDSubscriber_decl.objects.filter.return_value = [self.rcord_subscriber]
-
self.rcord_subscriber.save()
self.models_decl.RCORDSubscriber_decl.save.assert_called()