X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=planetstack%2Fhpc_observer%2Fsteps%2Fsync_serviceprovider.py;h=78fd1961a70e54ad19ad1caf4c78fc5d5660566c;hb=7c30a429254eaf1cd2fe6721f783c2d3cb7e8d67;hp=5eb991a35b3ba9bc9cd1aaff20a4457ec654ece4;hpb=88be16c0264985f0b6fa16a7f4c047bd598115a7;p=plstackapi.git

diff --git a/planetstack/hpc_observer/steps/sync_serviceprovider.py b/planetstack/hpc_observer/steps/sync_serviceprovider.py
index 5eb991a..78fd196 100644
--- a/planetstack/hpc_observer/steps/sync_serviceprovider.py
+++ b/planetstack/hpc_observer/steps/sync_serviceprovider.py
@@ -24,8 +24,19 @@ class SyncServiceProvider(SyncStep, HpcLibrary):
         SyncStep.__init__(self, **args)
         HpcLibrary.__init__(self)
 
-    def fetch_pending(self):
-        return ServiceProvider.objects.filter(Q(enacted__lt=F('updated')) | Q(enacted=None))
+    def fetch_pending(self, deleted):
+        self.sanity_check()
+
+        return SyncStep.fetch_pending(self, deleted)
+
+    def sanity_check(self):
+        # sanity check to make sure our PS objects have CMI objects behind them
+        all_sp_ids = [x["service_provider_id"] for x in self.client.onev.ListAll("ServiceProvider")]
+        for sp in ServiceProvider.objects.all():
+            if (sp.service_provider_id is not None) and (sp.service_provider_id not in all_sp_ids):
+                logger.info("Service provider %s was not found on CMI" % sp.service_provider_id)
+                sp.service_provider_id=None
+                sp.save()
 
     def sync_record(self, sp):
         logger.info("sync'ing service provider %s" % str(sp))
@@ -39,3 +50,7 @@ class SyncServiceProvider(SyncStep, HpcLibrary):
             self.client.onev.Update("ServiceProvider", sp.service_provider_id, sp_dict)
 
         sp.save()
+
+    def delete_record(self, m):
+        if m.service_provider_id is not None:
+            self.client.onev.Delete("ServiceProvider", m.service_provider_id)