From 95470622600c66d3d76aa0ecdacd56559604c175 Mon Sep 17 00:00:00 2001 From: Sapan Bhatia Date: Mon, 4 Aug 2014 10:48:28 -0400 Subject: [PATCH] If an object's peer is not set, it implies no dependency. So rather than failing the dependency check, we pass it. --- planetstack/ec2_observer/syncstep.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/planetstack/ec2_observer/syncstep.py b/planetstack/ec2_observer/syncstep.py index dcfea7d..d5f7523 100644 --- a/planetstack/ec2_observer/syncstep.py +++ b/planetstack/ec2_observer/syncstep.py @@ -60,7 +60,10 @@ class SyncStep: for dep in self.dependencies: peer_name = dep[0].lower() + dep[1:] # django names are camelCased with the first letter lower peer_object = getattr(obj, peer_name) - if (peer_object.pk==failed.pk): + + # peer_object can be None, and if so there + # is no object-level dependency + if (peer_object and peer_object.pk==failed.pk): raise FailedDependency def call(self, failed=[], deletion=False): -- 2.47.0