Save memory values changed via Xen API

PATCH New Jim Fehlig 1 participant 1 message Jul 17, 2007

Share & tools

Jim Fehlig

author

2007-07-17 22:41 UTC #0 lore raw

PATCH 1/1 Save memory values changed via Xen API

Memory values changed via Xen API methods VM.set_memory_[dynamic|static]_[min|max] are not saved to managed domain's config. Attached is a trivial fix.

Regards,
Jim

# HG changeset patch
# User Jim Fehlig <[email protected]>
# Date 1184711414 21600
# Node ID ff3f73de683f9c720ba3c2922d4f51897b9b5ef0
# Parent 1f348e70a5affdea9b44c1e39cd5ef094ad4a0bf
Save new memory values in domain config when calling VM.set_memory_[dynamic|static]_[min|max] via Xen API.

Signed-off-by Jim Fehlig <[email protected]>

tools/python/xen/xend/XendAPI.py +4 -4

diff -r 1f348e70a5af -r ff3f73de683f tools/python/xen/xend/XendAPI.py
@@ -1410,22 +1410,22 @@ class XendAPI(object):
     def VM_set_memory_dynamic_max(self, session, vm_ref, mem):
         dom = XendDomain.instance().get_vm_by_uuid(vm_ref)
         dom.set_memory_dynamic_max(int(mem))
-        return xen_api_success_void()
+        return self._VM_save(dom)
 
     def VM_set_memory_dynamic_min(self, session, vm_ref, mem):
         dom = XendDomain.instance().get_vm_by_uuid(vm_ref)
         dom.set_memory_dynamic_min(int(mem))
-        return xen_api_success_void()
+        return self._VM_save(dom)
 
     def VM_set_memory_static_max(self, session, vm_ref, mem):
         dom = XendDomain.instance().get_vm_by_uuid(vm_ref)
         dom.set_memory_static_max(int(mem))
-        return xen_api_success_void()
+        return self._VM_save(dom)
     
     def VM_set_memory_static_min(self, session, vm_ref, mem):
         dom = XendDomain.instance().get_vm_by_uuid(vm_ref)
         dom.set_memory_static_min(int(mem))
-        return xen_api_success_void()
+        return self._VM_save(dom)
 
     def VM_set_memory_dynamic_max_live(self, session, vm_ref, mem):
         dom = XendDomain.instance().get_vm_by_uuid(vm_ref)

_______________________________________________
Xen-devel mailing list
[email protected]
http://lists.xensource.com/xen-devel