Issue #8656
closed
Remove external call from 0061_call_handle_artifact_checksums_command
Status:
CLOSED - CURRENTRELEASE
Description
Calling management command from migrations eventually leads to breaking the migration tree in new installations, once the models used in handle-artifact-checksums
go out of sync with schema valid during application of this particular migration.
Migrations must not depend on models neither directly nor indirectly (through management commands).
Possible workaround would be to replace calling management command in the migration with noop
.
- Triaged changed from No to Yes
- Sprint set to Sprint 95
- Sprint changed from Sprint 95 to Sprint 96
- Sprint/Milestone set to 3.13.0
From the pulpcore meeting today, we're going to ensure the code used in the migration will only use code that constructs models from the db.
Was that the only issue you saw here? I had heard that there was an error experienced at upgrade time but I don't see the details here.
Using a noop is not a reasonable option because then the pulp installer, pulp containers, katello installer, individual users, galaxy-ng, etc, etc would all have to handle this manually. This is was the reason it's in the migrations to start with.
- Status changed from NEW to ASSIGNED
- Assignee set to daviddavis
- Status changed from ASSIGNED to POST
- Sprint changed from Sprint 96 to Sprint 97
- Status changed from POST to MODIFIED
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Also available in: Atom
PDF
Change migration to use django's get_model
fixes #8656