Projet

Général

Profil

Bug #20937

migration unicité slug vs index

Ajouté par Frédéric Péters il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
01 janvier 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Bug django/django-tenant-schemas (https://code.djangoproject.com/ticket/28646) qui fait que la migration "slug unique" ajoute un index déjà existant.

E               CommandError: tenant creation failed (relation "airquality_airquality_slug_bf868c3d_like" already exists

(airquality état le premier connecteur).


Fichiers

Révisions associées

Révision 5ef64b09 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

misc: disable db_index in slug fields to work around migration bug (#20937)

Révision 4bbfc767 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

misc: disable db_index in slug fields to work around migration bug (#20937)

Révision 0daa3334 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

migrations: mark slug as unique from the start (#20937)

Révision 425398fb (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

migrations: mark slug as unique from the start (#20937)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 6 ans

Contournement en marquant la version initiale de SlugField d'un db_index=False. (la migration "slug unique" basculera sur db_index=True). Patch tapé pour Passerelle mais la même chose est à faire dans les connecteurs externes.

#2

Mis à jour par Serghei Mihai il y a plus de 6 ans

Ack

#3

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 5ef64b09621a85708735164a85a15e7cfb0db4c8
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Jan 1 17:17:26 2018 +0100

    misc: disable db_index in slug fields to work around migration bug (#20937)

(dans le dépôt depuis un certain temps et recettes mis à jour avec mais ça a planté côté imio, et je me demande si la première migration pourrait être changée pour déjà noter unique=True plutôt, mais je vais creuser essayer de comprendre ce qui s'est passé chez imio)

#4

Mis à jour par Frédéric Péters il y a plus de 6 ans

Testé là-bas, pas capté la différence avec nous, mais je ne vois pas de mal à spécifier dès la première migration que le slug a unique=True.

#5

Mis à jour par Serghei Mihai il y a plus de 6 ans

Oui, ack

#6

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 0daa33344387df4e5bf0c434462f07c1adf914b0
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Jan 8 12:13:30 2018 +0100

    migrations: mark slug as unique from the start (#20937)
#7

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF