only save numberCores to OS metadata if it changes

diff --git a/plstackapi/core/models/plcorebase.py b/plstackapi/core/models/plcorebase.py
index 52aa0f7..eac3114 100644 (file)
--- a/plstackapi/core/models/plcorebase.py
+++ b/plstackapi/core/models/plcorebase.py
@@ -1,5 +1,6 @@
 import os
 from django.db import models
+from django.forms.models import model_to_dict
 
 class PlCoreBase(models.Model):
 
@@ -10,5 +11,36 @@ class PlCoreBase(models.Model):
         abstract = True
         app_label = "core"
 
+    def __init__(self, *args, **kwargs):
+        super(PlCoreBase, self).__init__(*args, **kwargs)
+        self.__initial = self._dict
+
+    @property
+    def diff(self):\r
+        d1 = self.__initial\r
+        d2 = self._dict\r
+        diffs = [(k, (v, d2[k])) for k, v in d1.items() if v != d2[k]]\r
+        return dict(diffs)\r
+\r
+    @property\r
+    def has_changed(self):\r
+        return bool(self.diff)\r
+\r
+    @property\r
+    def changed_fields(self):\r
+        return self.diff.keys()\r
+\r
+    def get_field_diff(self, field_name):\r
+        return self.diff.get(field_name, None)\r
+\r
+    def save(self, *args, **kwargs):\r
+        super(PlCoreBase, self).save(*args, **kwargs)\r
+        self.__initial = self._dict
+
+    @property
+    def _dict(self):\r
+        return model_to_dict(self, fields=[field.name for field in\r
+                             self._meta.fields])
+
 
 

diff --git a/plstackapi/openstack/manager.py b/plstackapi/openstack/manager.py
index 171946e..f991676 100644 (file)
--- a/plstackapi/openstack/manager.py
+++ b/plstackapi/openstack/manager.py
@@ -215,7 +215,7 @@ class OpenStackManager:
             sliver.instance_id = instance.id
             sliver.instance_name = getattr(instance, 'OS-EXT-SRV-ATTR:instance_name')
 
-        if sliver.instance_id:
+        if sliver.instance_id and ("numberCores" in sliver.changed_fields):
             self.driver.update_instance_metadata(sliver.instance_id, {"cpu_cores": str(sliver.numberCores)})
 
     @require_enabled