commit 533d543f53213d701db6d0ee7198ae00dbacfd37
author Scott Baker <smbaker@gmail.com> Wed Jan 22 00:08:50 2014 -0800
committer Scott Baker <smbaker@gmail.com> Wed Jan 22 00:08:50 2014 -0800
tree 95449363f7685608ab6590e4b3384de4b47ebc4b
parent d6313f71c939c00c899eadd971043e600b856914

service_provider_id is a create-only attribute, don't try to update it

planetstack/hpc_observer/steps/sync_contentprovider.py

diff --git a/planetstack/hpc_observer/steps/sync_contentprovider.py b/planetstack/hpc_observer/steps/sync_contentprovider.py
index 00117b8..fc10dae 100644
--- a/planetstack/hpc_observer/steps/sync_contentprovider.py
+++ b/planetstack/hpc_observer/steps/sync_contentprovider.py
@@ -28,7 +28,7 @@
         return ContentProvider.objects.filter(Q(enacted__lt=F('updated')) | Q(enacted=None))
 
     def sync_record(self, cp):
-        logger.info("sync'ing service provider %s" % str(cp))
+        logger.info("sync'ing content provider %s" % str(cp))
         account_name = self.make_account_name(cp.name)
         print "XXX", cp.name, account_name
 
@@ -37,11 +37,12 @@
 
         spid = cp.serviceProvider.service_provider_id
 
-        cp_dict = {"account": account_name, "name": cp.name, "enabled": cp.enabled, "service_provider_id": spid}
+        cp_dict = {"account": account_name, "name": cp.name, "enabled": cp.enabled}
 
         #print cp_dict
 
         if not cp.content_provider_id:
+            cp_dict["service_provider_id"] = spid
             id = self.client.onev.Create("ContentProvider", cp_dict)
             cp.content_provider_id = id
         else: