Development #30560
clicrdv: restaurer les migrations supprimer lors du squash
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0001-clicrdv-restore-deleted-migrations-30560.patch 0001-clicrdv-restore-deleted-migrations-30560.patch ajouté
- Patch proposed changé de Non à Oui
Enlevé les commits qui n'ont rien à faire là.
Mis à jour par Frédéric Péters il y a environ 5 ans
Il y a quand même eu plein de déploiements depuis. (???)
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Frédéric Péters a écrit :
Il y a quand même eu plein de déploiements depuis. (???)
À ma lecture du code du chargeur de migrations ça se passe comme cela :
1. il lit tous les fichiers de migrations ceux qui ont un attribut replaces = ['0001', '0002'] etc.. il les mets dans une liste replacing et les autre dans normal
2. ensuite il regarde les migrations déjà appliquées, si ça matche un ensemble replaces il bouge la migration de replacing dans la liste des migrations normales/appliquées,
3. quand il parcours les migrations "normales" il cherche les migrations dont il dépend dans normal
Or là c'est une migration squashé et initiale, si pas de migration elle aurait été appliqué automatiquement (parce qu'initiale, je suppose qu'est initiale toute migration sans dépendance), mais ce n'est pas le cas il y a déjà des migrations, ensuite il ne trouve pas la totalité des migrations qu'elle remplace donc il ne fait pas le point 2. plus haut, à la fin il ne la trouve pas. J'avoue que ça devrait être traité mieux que ça histoire de donner un message plus intelligible mais ce n'est pas le cas.
Donc ça a marché parce qu'effectivement on était au même niveau partout quand tu as fait le squash.
Mis à jour par Frédéric Péters il y a environ 5 ans
J'étais sur le point de pousser ça avant de me souvenir qu'il y avait un objectif derrière ça, c'était dégager les références à passerelle.datasources; pour tester l'affaire je viens de pousser dans la même branche la suppression effective de passerelle.datasources.
Et si ça ne marche pas, soit il y aura à traffiquer les migrations clicrdv, soit autre chose, mais perso j'aimerais vraiment arriver à nettoyer (le passerelle.datasources, retirer les migrations passées peu importe).
Mis à jour par Frédéric Péters il y a environ 5 ans
- Lié à Development #31059: supprimer passerelle.datasources ajouté
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Nouveau à En cours
Je regarde pour ne pas référencer passerelle.datasources.
Mis à jour par Frédéric Péters il y a environ 5 ans
Je regarde pour ne pas référencer passerelle.datasources.
Mais c'est ok l'autre ticket gère ça.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de En cours à Résolu (à déployer)
commit 2cd08c373b5da02d860a71e385d8ed8db49691c4 (wip/30560-clicrdv-restaurer-les-migrations)
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Feb 12 12:29:33 2019 +0100
clicrdv: restore deleted migrations (#30560)
There were deleted after a squash in issue #22205.
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution déployée à Fermé
Mis à jour par Emmanuel Cazenave il y a plus de 4 ans
- Lié à Bug #22205: Migration clicrdv.0001_initial dependencies reference nonexistent parent node (u'datasources', u'0001_initial') ajouté
clicrdv: restore deleted migrations (#30560)
There were deleted after a squash in issue #22205.