[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Stable update proposed for ganeti (fixing #528618)



Hi,

I'd like to update ganeti to stable-proposed-updates to fix the above
bug. It's a simple one-line change; I see only one potential breakage
out of it - if people already did the symlink fix pointed out in the
bug, but with the symlink pointing to a different version of Xen, then
this version will change what xen version they use. More clearly:
 - if currently /usr/lib/xen points to anything else than
   /usr/lib/xen-3.2-1, then
 - this upload will break people's HVM-based clusters

Here is the proposed diff:

diff -u ganeti-1.2.6/debian/changelog ganeti-1.2.6/debian/changelog
--- ganeti-1.2.6/debian/changelog
+++ ganeti-1.2.6/debian/changelog
@@ -1,3 +1,10 @@
+ganeti (1.2.6-3+lenny1) stable; urgency=low
+
+  * Fix hvmloader path to match Lenny's xen-utils-3.2-1
+    (Closes: #528618)
+
+ -- Iustin Pop <iusty@k1024.org>  Tue, 28 Jul 2009 19:29:14 +0200
+
 ganeti (1.2.6-3) unstable; urgency=low
 
   * Cherry-pick commit 2461 from upstream, fixing (yet again, hopefully 
only in patch2:
unchanged:
--- ganeti-1.2.6.orig/debian/patches/01-fix-lenny-hvmloader.patch
+++ ganeti-1.2.6/debian/patches/01-fix-lenny-hvmloader.patch
@@ -0,0 +1,16 @@
+This is a fix for #528618
+
+Iustin Pop, <iusty@k1024.org> Tue, 28 Jul 2009 18:18:40 +0200
+diff --git a/lib/hypervisor.py b/lib/hypervisor.py
+index 4a8d892..d76c773 100644
+--- a/lib/hypervisor.py
++++ b/lib/hypervisor.py
+@@ -635,7 +635,7 @@ class XenHvmHypervisor(XenHypervisor):
+     """
+     config = StringIO()
+     config.write("# this is autogenerated by Ganeti, please do not edit\n#\n")
+-    config.write("kernel = '/usr/lib/xen/boot/hvmloader'\n")
++    config.write("kernel = '/usr/lib/xen-3.2-1/boot/hvmloader'\n")
+     config.write("builder = 'hvm'\n")
+     config.write("memory = %d\n" % instance.memory)
+     config.write("vcpus = %d\n" % instance.vcpus)

Let me know if this is OK or not, or if you need more information.

thanks,
iustin


Reply to: