Migrating a legacy table results in data loss

[Bibhas C Debnath]

I created several models for our legacy database using inspectdb and hence all those models had their managed flag set to False. Now I need to make some changes to that model, hence I removed the managed flag and the generated migration wants to drop my table first and then create it again. Which leads to loss of all my data.

So I edited the migration file and removed the DeleteModel operation. And faked that migration to avoid the error raised by CreateModel as the table already exists.

But now I cannot go backward beyond that migration as the backward operation of CreateModel will drop my table anyway and there is no way to make a migration irreversible or ask it Not to drop my table.

What would help in such cases is a way to handle the backward migration. So that I can either choose not to drop my table or raise an exception to make the migration irreversible. Or at least make the CreateModel operation irreversible. I found that the Operation class has a flag called reversible. But I could find no way to set it to False.

For now I've added a fake RunPython operation that returns None to make that migration irreversible, as suggested by Markus ​/ here. But I'd love some way to handle it properly.

[Simon Charette]

Shouldn't the autodetector simply issue a AlterModelOptions to change the model state from managed=False to True in this case? This would be no-op at the database level.

[Markus Holtermann]

Yes, precisely charettes. The related discussion on django-users ishttps://groups.google.com/forum/m/#!topic/django-users/w9aVT3NOpkM

[Tim Graham]

​PR

[Tim Graham <timograham@…>]

In 061caa5b386681dc7bdef16918873043224a299c:

Fixed #24037 -- Prevented data loss possibility when changing Meta.managed.

The migrations autodetector now issues AlterModelOptions operations for
Meta.managed changes instead of DeleteModel + CreateModel.

Thanks iambibhas for the report and Simon and Markus for review.

[Tim Graham <timograham@…>]

In 51ea30a43bafad40b4a24ad0be346796a9c7ab6a:

[1.7.x] Fixed #24037 -- Prevented data loss possibility when changing Meta.managed.

The migrations autodetector now issues AlterModelOptions operations for
Meta.managed changes instead of DeleteModel + CreateModel.

Thanks iambibhas for the report and Simon and Markus for review.

Backport of 061caa5b386681dc7bdef16918873043224a299c from master