Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#23331 closed Bug (invalid)

Circular dependencies are not resolved by new Migrate framework

Reported by: Aidan Lister Owned by: Andrew Godwin
Component: Migrations Version: dev
Severity: Normal Keywords: migrate
Cc: aidan@…, Markus Holtermann Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Aidan Lister)

It's very easy and not explicitly not-supported to create circular dependencies in your apps. I have attached a test project which has two circular dependency resolution failures.

This fails to create migrations:

./ makemigrations contractors inspections organisations properties reports tasks timeline

This creates the migrations:

./ makemigrations properties reports tasks timeline
./ makemigrations contractors inspections

But then ./ migrate fails:

django.db.migrations.graph.CircularDependencyError: [('contractors', u'0002_auto_20140821_0156'), ('reports', u'0001_initial'), ('inspections', u'0001_initial'), ('contractors', u'0002_auto_20140821_0156')]

I have tried manually breaking up the migrations into lots of little steps, but you'll have the same problem with the reports and inspections app. Even with lots of tiny migrations it eventually loops around.

Attachments (1) (46.3 KB ) - added by Aidan Lister 10 years ago.

Download all attachments as: .zip

Change History (4)

by Aidan Lister, 10 years ago

Attachment: added

comment:1 by Aidan Lister, 10 years ago

Description: modified (diff)

comment:2 by Andrew Godwin, 10 years ago

Resolution: invalid
Status: newclosed

Tried to reproduce from example project, and it worked perfectly with the line:

./ makemigrations tasks timeline organisations reports contractors inspections properties

The generated migrations then also ran successfully.

Closing as INVALID as I suspect this was encountered just before the fixes to #23322 and #23315 landed in master earlier, and in particular #23315 would have manifested as the first command not being able to run.

If you can reproduce this on the most recent commit in master, please reopen and this time provide full console output from when the command runs.

comment:3 by Markus Holtermann, 10 years ago

Cc: Markus Holtermann added
Note: See TracTickets for help on using tickets.
Back to Top