Fix migration to prepare for Django 5 #2152
Conversation
| field=models.ManyToManyField( | ||
| through="container.ManifestListManifest", to="container.manifest" | ||
| ), | ||
| field=models.ManyToManyField(through='container.ManifestListManifest', through_fields=('image_manifest', 'manifest_list'), to='container.manifest'), |
There was a problem hiding this comment.
I think we can get away with this, because it isn't even a change to the schema in postgres. Just the internal Django state model was missing the 'trough_fields'.
And the migration coming later to fix this is just the same but also idempotent.
pedro-psb
left a comment
pedro-psb
left a comment
There was a problem hiding this comment.
The migration does not change the database schema, so we can safely backport.
# pulpcore-manager sqlmigrate container 0035
BEGIN;
--
-- Alter field content_ptr on blob
--
-- (no-op)
--
-- Alter field distribution_ptr on containerdistribution
--
-- (no-op)
--
-- Alter field pulp_id on containernamespace
--
-- (no-op)
--
-- Alter field manifest_signing_service on containerpushrepository
--
-- (no-op)
--
Backport to 2.20: 💚 backport PR created✅ Backport PR branch: Backported as #2153 🤖 @patchback |
Backport to 2.19: 💚 backport PR created✅ Backport PR branch: Backported as #2154 🤖 @patchback |
Backport to 2.22: 💚 backport PR created✅ Backport PR branch: Backported as #2155 🤖 @patchback |
Backport to 2.24: 💚 backport PR created✅ Backport PR branch: Backported as #2156 🤖 @patchback |
2 participants
No description provided.