Make invirt-lvm lvcreate-all use sqlalchemy correctly. dev iannucci prod 0.4.14
authorPeter Iannucci <iannucci@mit.edu>
Wed, 29 May 2013 00:21:11 +0000 (20:21 -0400)
committerPeter Iannucci <iannucci@mit.edu>
Wed, 29 May 2013 00:21:11 +0000 (20:21 -0400)
debian/changelog
host/usr/sbin/invirt-lvm

index 9a903b8..9b74add 100644 (file)
@@ -1,3 +1,9 @@
+invirt-remote (0.4.14) unstable; urgency=low
+
+  * Make invirt-lvm lvcreate-all use sqlalchemy correctly
+
+ -- Peter A. Iannucci <iannucci@mit.edu>  Tue, 28 May 2013 20:19:00 -0400
+
 invirt-remote (0.4.13) unstable; urgency=low
 
   * Make the listvms remctl include domains that get stuck in pygrub
 invirt-remote (0.4.13) unstable; urgency=low
 
   * Make the listvms remctl include domains that get stuck in pygrub
index cae6b2b..6860d9c 100755 (executable)
@@ -31,12 +31,12 @@ if machine_specific:
     lvpath = "/dev/" + vg + "/" + lvname
 
 if subcommand == "lvcreate-all":
     lvpath = "/dev/" + vg + "/" + lvname
 
 if subcommand == "lvcreate-all":
-    from invirt import database
+    from invirt.database import models, connect
     import re
     import re
-    database.connect()
-    for d in Disk.select():
+    connect()
+    for d in models.Disk.query().all():
         check(re.match('^[A-Za-z0-9]+$', d.guest_device_name))
         check(re.match('^[A-Za-z0-9]+$', d.guest_device_name))
-        machine = Machine.get(d.machine_id)
+        machine = models.Machine.query().filter_by(machine_id=d.machine_id).one()
         check(re.match('^[A-Za-z0-9][A-Za-z0-9._-]*$', machine.name))
         lvname = prefix + machine.name + "_" + d.guest_device_name
         if not os.path.exists("/dev/%s/%s" % (vg, lvname)):
         check(re.match('^[A-Za-z0-9][A-Za-z0-9._-]*$', machine.name))
         lvname = prefix + machine.name + "_" + d.guest_device_name
         if not os.path.exists("/dev/%s/%s" % (vg, lvname)):