Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #3584 : correction des commandes de migration de la v18 #3586

Merged
merged 1 commit into from
May 11, 2016
Merged

Fix #3584 : correction des commandes de migration de la v18 #3586

merged 1 commit into from
May 11, 2016

Conversation

gustavi
Copy link
Contributor

@gustavi gustavi commented May 5, 2016

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) #3584

Corrections

  • Corrige l'erreur sur la migration ZEP-25
  • Enlève les faux positifs sur les contenus publiés
  • Ajout des derniers contenus publiés
  • Enlève le check sur les FK pour la tables à supprimer

QA

  • Ça a été testé sur la bêta et ça fonctionne parfaitement. À re-tester si vous le souhaitez.

Note : il reste des erreurs sur la commande de suppression des tables. (cf http://paste.awesom.eu/5z8K ), une PR qui fixe entièrement ça est la bienvenue !

@GerardPaligot
Copy link
Member

GerardPaligot commented May 5, 2016

gustavi: 344 content(s) migrated with succes
gustavi: 0 content(s) failed
gustavi: \o/

Source: IRC

En gros, je suis pour le merge.

@DevHugo
Copy link
Contributor

DevHugo commented May 5, 2016

C'est juste du flake8, rien de grave.

@coveralls
Copy link

coveralls commented May 6, 2016

Coverage Status

Coverage increased (+0.03%) to 87.391% when pulling 3a9057b on gustavi:fix-v18-rc5 into 4cd52a4 on zestedesavoir:release-v18.


Il faudra supprimer en SQL:

- `SET FOREIGN_KEY_CHECKS=0;`
- `UPDATE tutorial_tutorial SET last_note_pk=NULL;`
- `UPDATE article_article SET last_reaction=NULL;`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

il se peut que cette commande foire (il manque le _id derrière last_reaction). Je viens de m'en rendre compte.

@GerardPaligot
Copy link
Member

N'oublions pas cette PR. @gustavi tu sais la mettre à jour avec les identifiants et la tester ?

@gustavi
Copy link
Contributor Author

gustavi commented May 9, 2016

Ce soir.

@GerardPaligot
Copy link
Member

Super, merci !

@gustavi
Copy link
Contributor Author

gustavi commented May 9, 2016

Je ne pourrait tester qu'avec une BDD propre, ping @SpaceFox

EDIT : mis à jour avec l'histoire des pk, id

@SpaceFox
Copy link
Contributor

SpaceFox commented May 9, 2016

Ça implique quoi exactement ta demande @gustavi ?

@gustavi
Copy link
Contributor Author

gustavi commented May 9, 2016

Est-ce que tu peux importer une BDD propre sur la beta ? Sinon m'expliquer comment je peux faire ça. Là elle n'est plus propre parce que j'ai lancé des milions de commandes dessus :/

@SpaceFox
Copy link
Contributor

SpaceFox commented May 9, 2016

OK, mais ce soir.

@coveralls
Copy link

coveralls commented May 9, 2016

Coverage Status

Changes Unknown when pulling 05265d1 on gustavi:fix-v18-rc5 into * on zestedesavoir:release-v18*.

@GerardPaligot
Copy link
Member

Sinon m'expliquer comment je peux faire ça.

Je vote pour ça.

@firm1
Copy link
Contributor

firm1 commented May 9, 2016

Sinon m'expliquer comment je peux faire ça.

Je vote pour ça.

Il me semble que tout est là.

@gustavi
Copy link
Contributor Author

gustavi commented May 10, 2016

Oh ! Je découvre des trucs :)

@SpaceFox
Copy link
Contributor

SpaceFox commented May 10, 2016

Ça me fait penser que j'ai complètement zappé de resynchro la bêta hier soir >_<

Ladite synchro nécessite d'avoir accès au serveur de prod (elle se lance depuis le compte zds sur la prod), au serveur bêta, et aux serveurs MySQL (utilisateur root).

Une fois les accès récupérés, il suffit dans la home de zds, de faire ./sync_beta.sh et de suivre scrupuleusement les instructions.

@gustavi
Copy link
Contributor Author

gustavi commented May 10, 2016

Je ferai ça quand j'aurai un accès à SSH, merci @SpaceFox.

@GerardPaligot
Copy link
Member

@gustavi Tu semblais découvrir des trucs, tu as pu avancer efficament ici ?

@gustavi
Copy link
Contributor Author

gustavi commented May 11, 2016

mysql> SET FOREIGN_KEY_CHECKS=0;
Query OK, 0 rows affected (0.00 sec)

mysql> UPDATE tutorial_tutorial SET last_note_id=NULL;
Query OK, 105 rows affected (0.06 sec)
Rows matched: 612  Changed: 105  Warnings: 0

mysql> UPDATE article_article SET last_reaction_id=NULL;
Query OK, 97 rows affected (0.07 sec)
Rows matched: 239  Changed: 97  Warnings: 0

mysql> DROP TABLE tutorial_tutorial_subcategory;
Query OK, 0 rows affected (0.01 sec)

mysql> DROP TABLE tutorial_tutorial_authors;
Query OK, 0 rows affected (0.03 sec)

mysql> DROP TABLE tutorial_note;
Query OK, 0 rows affected (0.03 sec)

mysql> DROP TABLE tutorial_tutorialread;
Query OK, 0 rows affected (0.00 sec)

mysql> DROP TABLE tutorial_tutorial;
Query OK, 0 rows affected (0.01 sec)

mysql> DROP TABLE article_article_subcategory;
Query OK, 0 rows affected (0.03 sec)

mysql> DROP TABLE article_article_authors;
Query OK, 0 rows affected (0.07 sec)

mysql> DROP TABLE article_reaction;
Query OK, 0 rows affected (0.01 sec)

mysql> DROP TABLE article_articleread;
Query OK, 0 rows affected (0.00 sec)

mysql> DROP TABLE article_article;
Query OK, 0 rows affected (0.01 sec)

mysql> SET FOREIGN_KEY_CHECKS=1;
Query OK, 0 rows affected (0.00 sec)

À merger.

@DevHugo
Copy link
Contributor

DevHugo commented May 11, 2016

Bravo o/ .

@GerardPaligot
Copy link
Member

Dès que travis est ok, je merge.

@coveralls
Copy link

coveralls commented May 11, 2016

Coverage Status

Changes Unknown when pulling f9d762c on gustavi:fix-v18-rc5 into * on zestedesavoir:release-v18*.

@GerardPaligot GerardPaligot merged commit c150c5f into zestedesavoir:release-v18 May 11, 2016
gustavi pushed a commit that referenced this pull request Jun 17, 2016
* Enlève le recalcul des votes à la volée lors de la migration

* ZEP-25 : Corrige la commande de migration + instructions MEP

* corrige le MP de la migration

* Rééquilibre le texte de présentation de la home

* Design de la page "Tous les tags"

* Trailing slash des ressources de l'API optionnel

* Fix #3503 : plus possible de creer un tag vide

* Fix #3494 : correction du menu pour les articles

* Fix #3497 : aligne les messages de la page de connexion

* Fix #3489 : corrige l'affiche des article sur la page d'un membre

* Utilise le bon sprite sur écran rétina

* fix 3513: supprime la supperposition des aides markdown

* fix #3492: Corrige le design des content-item de la home

* Affiche 3 articles et 4 tutos sur la home

Signed-off-by: Quentin Gliech <quentingliech@gmail.com>

* Enlève le nombre des commentaires sur les content-items des tutoriels

Signed-off-by: Quentin Gliech <quentingliech@gmail.com>

* Affiche le nombre de publications sur la home

* corrige une migration d'id

* Corrige la marge sur la home "connectée"

* Corrige la souscription à un sujet

* fix(migration): Corrige les souscriptions des membres.

Closes #3498

* Fix #3507 : correction de l'affichage des noms des filtres

* Typo sur la description sur la page des categories

* fix 3490

* enlève les bots

* Fix #3534 : correction de l'encodage des sorties pour la commande zep-25

* fix: Ordonne correctement les sujets des sujets suivis.

Closes #3536

* fix: Recupere la liste des sujets suivis pour la sidebar.

Closes #3498

* test: Retourne toujours bien les sujets suivis dans le bon ordre.

* fix: N'affiche plus "Suivre ce contenu" pour les visiteurs.

Closes #3545

* Fix #3542 : Empêche de créer un tag en double

* Fix #3533 : corrige le nombre de tuto par tag

* Fix #3496 : correction de la page des contenus (#3524)

* [beta v18]Rend la topbar cohérente en matière de toptags (#3564)

* rend la topbar cohérente

* typo

* typo

* typo

* typo

* typo

* typo

* Mise à jour de la configuration nginx

* [v18]Donne les instruction pour corriger #3448 (#3577)

Donne les instruction pour supprimer les anciennes tables du module des tutos #3448

* [ci skip] Correction du message de migration de la ZEP-25 (#3595)

* Fix #3592: Ajout du lien vers "CONTRIBUTING.md" (#3596)

* Fix #3587 : Index les relations generiques des modeles des notifs. (#3594)

* fix(notif): Index les relations generiques des modeles des notifs.

Pour information, Django n'index pas automatiquement les relations
generiques ce qui vient plomber les performances quand on
commence a disposer d'une grosse base de donnees.

Voir https://groups.google.com/forum/#!topic/django-users/GJ-FVh40-F4

Closes #3587

* [ci skip] Ajoute la commande d'opti mysql dans le update.md

* Fix #3584 : correction des commandes de migration de la v18 (#3586)

* Évite les tags répétés plusieurs fois (#3601)

* augmente la taille des tags et évite qu'ils soient recréés quand ils sont trop grands.

* ajoute un test unitaire

* Fix l'unicite des tags

* Corrige l'unicite des tags (again)

* change le processus de création des tags lors du sujet de beta

* fix indexerror

* pep8

* pep8

* fix indexerror

* remove useless test

* fix test

* pep8

* Correction affichage du tag (#3597)

* ZEP-25 : correction mineure de la commande de migration (#3612)

* Fix #3614 - ZEP-25 : corrige les URL dans le mp de migration (#3616)

* Fix #3626 : corrige les espaces dans l'ajout de tags sur les contenus (#3627)

* Hotfix v18 : votes negatifs (#3635)

* fix(notif): Marque comme lu une notif d'un contenu non suivi. (#3664)

Closes #3659

* fix(notif): Evite les notifs persistantes a la suppression. (#3663)

Closes #3639

* fix(3630): Migration des souscriptions par e-mail. (#3661)

Closes #3630

* fix(notif): Mise a jour de la date des notifications. (#3662)

Closes #3628
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants