From abf0babac755535c7d0e027e6a7031823dc14dfc Mon Sep 17 00:00:00 2001
From: Sapan Bhatia <sapanb@cs.princeton.edu>
Date: Tue, 21 Oct 2014 20:53:41 -0400
Subject: [PATCH] Handle two error messages, when a user is orphaned

---
 .../openstack_observer/steps/sync_user_deployments.py     | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/planetstack/openstack_observer/steps/sync_user_deployments.py b/planetstack/openstack_observer/steps/sync_user_deployments.py
index 3656461..f7e41a0 100644
--- a/planetstack/openstack_observer/steps/sync_user_deployments.py
+++ b/planetstack/openstack_observer/steps/sync_user_deployments.py
@@ -26,8 +26,6 @@ class SyncUserDeployments(OpenStackSyncStep):
             return UserDeployments.objects.filter(Q(enacted__lt=F('updated')) | Q(enacted=None)) 
 
     def sync_record(self, user_deployment):
-	import pdb
-	pdb.set_trace()
         logger.info("sync'ing user %s at deployment %s" % (user_deployment.user, user_deployment.deployment.name))
 
         if not user_deployment.deployment.admin_user:
@@ -46,11 +44,15 @@ class SyncUserDeployments(OpenStackSyncStep):
             if site_deployments:
                 # need the correct tenant id for site at the deployment
                 tenant_id = site_deployments[0].tenant_id  
-		tenant_name =site_deployments[0].site.login_base
+		tenant_name = site_deployments[0].site.login_base
 
 		roles.append('user')
                 if user_deployment.user.is_admin:
                     roles.append('admin')
+	    else:
+		raise Exception('Internal error. Missing SiteDeployment for user %s'%user_deployment.user.email)
+	else:
+	    raise Exception('Siteless user %s'%user_deployment.user.email)
 
 
         user_fields = {'endpoint':user_deployment.deployment.auth_url,
-- 
2.47.0