From c6d374d12aed8a907e2ec696580c26b6d07a2e8b Mon Sep 17 00:00:00 2001
From: Luke Sneeringer <luke@sneeringer.com>
Date: Wed, 10 Sep 2014 15:07:03 -0500
Subject: [PATCH] Add autocommit to migration 36 if it's off.

---
 awx/main/migrations/0036_v148_changes.py | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/awx/main/migrations/0036_v148_changes.py b/awx/main/migrations/0036_v148_changes.py
index 9adf263e60..c70640adfe 100644
--- a/awx/main/migrations/0036_v148_changes.py
+++ b/awx/main/migrations/0036_v148_changes.py
@@ -4,7 +4,7 @@ import sys
 from south.utils import datetime_utils as datetime
 from south.db import db
 from south.v2 import DataMigration
-from django.db import models
+from django.db import models, transaction
 from django.utils.encoding import smart_text
 
 class Migration(DataMigration):
@@ -63,6 +63,11 @@ class Migration(DataMigration):
     def forwards(self, orm):
         "Write your forwards methods here."
 
+        # South seems to perform migrations with autocommit off in some cases.
+        # That breaks this migration, so ensure it's turned on.
+        old_autocommit = transaction.get_autocommit()
+        transaction.set_autocommit(True)
+
         # Copy Project old to new.
         print('Migrating Projects...', end='')
         new_ctype = self._get_content_type_for_model(orm, orm.Project)
@@ -382,11 +387,14 @@ class Migration(DataMigration):
                 a_s.new_job.add(new_job)
         print('')
 
+        # Restore autocommit to what it was.
+        transaction.set_autocommit(old_autocommit)
+
     def backwards(self, orm):
         "Write your backwards methods here."
 
         # FIXME: Would like to have this, but not required.
-        raise NotImplementedError()
+        raise RuntimeError("This migration is not reversable")
 
     models = {
         u'auth.group': {