]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
SQL scripts should not manage transactions
authorJohn Griffith <john.griffith8@gmail.com>
Mon, 15 Jun 2015 20:02:28 +0000 (14:02 -0600)
committerJohn Griffith <john.griffith8@gmail.com>
Mon, 15 Jun 2015 21:09:30 +0000 (15:09 -0600)
This bug was fixed a while back, but the migration added for
multi-attach reintroduced it for its sqlite_downgrade file.

This patch removes it.

blueprint enable-mysql-connector

Change-Id: Iaec8713bf5cf36bcc97afbca8dd025cda9c2238d
Closes-Bug: 1368391

cinder/db/sqlalchemy/migrate_repo/versions/040_sqlite_downgrade.sql

index dbe89d7c19cca0204e83d29d48aa0edbb19cb76f..e0d0afffb53d38b0208a02310b29e8ffb284a31e 100644 (file)
@@ -1,5 +1,3 @@
-BEGIN TRANSACTION;
-
 CREATE TABLE volumes_v39 (
     created_at DATETIME,
     updated_at DATETIME,
@@ -84,4 +82,3 @@ INSERT INTO volumes_v39
 DROP TABLE volumes;
 ALTER TABLE volumes_v39 RENAME TO volumes;
 DROP TABLE volume_attachment;
-COMMIT;