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

Résumé de l'expérience de migration en pré-prod' #238

Closed
pierre-24 opened this issue Aug 16, 2015 · 23 comments
Closed

Résumé de l'expérience de migration en pré-prod' #238

pierre-24 opened this issue Aug 16, 2015 · 23 comments
Labels

Comments

@pierre-24
Copy link
Collaborator

En gros, pour ce faciliter la vie:

  • Il faudrait pouvoir migrer sélectivement les articles, les mini ou les bigs-tutos
  • Il faudrait pouvoir ne pas migrer ce qui l'as déjà été (mais comment ?)
  • Il faudrait pouvoir exlure de la migration certaines choses (mais comment ?), ou migrer sélectivement des choses données (mais comment ?)

ping @sandhose pour d'autres retours :)

NOTE: le log de la migration : http://sprunge.us/SfFh

@artragis
Copy link
Owner

Il faudrait pouvoir migrer sélectivement les articles, les mini ou les bigs-tutos

déjà possible

Il faudrait pouvoir ne pas migrer ce qui l'as déjà été (mais comment ?)

là c'est plus complexe

@sandhose
Copy link

J'ai trouvé une liste de tutos HS via: find tutoriels-private/ -mindepth 1 -maxdepth 1 -type d '!' -exec test -e '{}/.git/refs/heads/master' ';' -print

faudrait faire ça en prod pour vérifier si c'est propre à la preprod ou non

@pierre-24
Copy link
Collaborator Author

Tu sais la copier ici, ou c'est "privé" ?

@pierre-24
Copy link
Collaborator Author

Il faudrait pouvoir ne pas migrer ce qui l'as déjà été (mais comment ?)
là c'est plus complexe

Ben on peut comparer tuto.old_pk + '_' + tuto.slug avec le slug du vieux tuto (marche aussi avec les articles)

@sandhose
Copy link

tutoriels-private/238_decouvrir-et-utiliser-wordpress      
tutoriels-private/207_deployez-votre-site-avec-wordpress   
tutoriels-private/208_deployez-votre-site-avec-wordpress   
tutoriels-private/516_la-programmation-cobol               
tutoriels-private/467_apprenez-a-programmer-en-c           
tutoriels-private/515_la-programmation-cobol               
tutoriels-private/206_deployez-votre-site-avec-wordpress      

@artragis
Copy link
Owner

Problème avec le tutoriel/article
le C++14 est arrivé
en effet le slug de ce dernier a changé (de le-c14-est-arreve on passe à le-c-14-est-arrive) et du coup ça foire.

@pierre-24
Copy link
Collaborator Author

Bien vu. C'est idiot, d'ailleurs --"

@artragis
Copy link
Owner

problème : je ne vois pas d'où ça vient

@pierre-24
Copy link
Collaborator Author

Ce dernier problème est réglé. Reste les autres ^^

@pierre-24
Copy link
Collaborator Author

Tient, quid des tutos que @sandhose a mentionné plus tôt ? On envoit MP aux auteurs, ou bien ?

@DevHugo
Copy link

DevHugo commented Aug 20, 2015

La programmation cobol semble marcher.

Les deux autres, je serais pour contacter les auteurs.

@pierre-24
Copy link
Collaborator Author

"programmer en C", c'est le tuto de Taurre, qu'il a bousillé je sais pas trop comment, donc il faudra le ré-importer (il a une archive quelque part).

@pierre-24
Copy link
Collaborator Author

Note: le tuto qui pose problème porte le pk 516. Celui de prod porte le pk 533. J'en déduis que c'est pas le même et que quelqu'un a pêté un tuto sans le dire ;)

@DevHugo
Copy link

DevHugo commented Aug 20, 2015

Tu veux dire que c'est bon, y'a plus aucun souci ?

@pierre-24
Copy link
Collaborator Author

Une màj pour l'indexation: http://sprunge.us/IFXM

@DevHugo
Copy link

DevHugo commented Aug 20, 2015

Le mieux c'est qu'on peut quand même rechercher sur les trois contenus qui ont des erreurs. Les deux premiers semblent liés à l'absence de pubdate.

C'est bizarre car avant il avait pas planter, donc c'est surement liés à ma dernière modification de code. Ou alors, ils y'avait pas de pubdate avant et comme avant il avait un auto_add=now, l'enregistrement ne bloquait pas.

Le dernier faut tester en local pour voir si on peut reproduire.

@pierre-24
Copy link
Collaborator Author

Ou alors, ils y'avait pas de pubdate avant et comme avant il avait un auto_add=now, l'enregistrement ne bloquait pas.

Moi je vote pour cette option là. Même si je m'étonne d'avoir des contenus sans pubdate publiés dans la nature :o

Le dernier faut tester en local pour voir si on peut reproduire.

Ce qui m'étonne, c'est qu'il fini par KeyboardInterupt, comme si sandhose en avait eu marre :o

@artragis
Copy link
Owner

N'oublions pas que nous sommes sur une version du serveur sur laquelle
énormément de tests ont été réalisés. Il est probable que l'état actuel en
prod ne soit pas "pas de pubdate". Néanmoins, il faut remettre le "now" par
défaut quand on fait un set (c'est simple ça ressemble à content.pubdate =
lavaleurquonveut or datetime.now())

Le 21 août 2015 07:54, Pierre Beaujean notifications@github.com a écrit :

Ou alors, ils y'avait pas de pubdate avant et comme avant il avait un
auto_add=now, l'enregistrement ne bloquait pas.

Moi je vote pour cette option là. Même si je m'étonne d'avoir des contenus
sans pubdate publiés dans la nature :o

Le dernier faut tester en local pour voir si on peut reproduire.

Ce qui m'étonne, c'est qu'il fini par KeyboardInterupt, comme si sandhose
en avait eu marre :o


Reply to this email directly or view it on GitHub
#238 (comment).

@DevHugo
Copy link

DevHugo commented Aug 21, 2015

Je pense que je peut rajouter une date par-défaut, c'est une ligne de code. Pour le troisième soucis, je regarde dans la soirée.

J'arrive pas à reproduire, pour le dernier mais pour les autres une PR a été lancé.

@DevHugo
Copy link

DevHugo commented Aug 22, 2015

Du coup, récap, la PR pour mettre par-défaut une pubdate est passé. Il reste la dernière erreur, je sais pas pourquoi elle est la. Peut-être, est t-elle la temporairement, je propose que on laisse tomber pour cette fois et on regarde si elle est encore la, à la prochaine migration.

@sandhose
Copy link

Entre temps, l'option --copy-repository indiquée dans l'update.md pour l'indexation solr n'existe plus, et empêche la synchro. Faut corriger l'update.md à ce niveau la

@DevHugo
Copy link

DevHugo commented Aug 28, 2015

Oups, désolé, on corrige ça.

@pierre-24
Copy link
Collaborator Author

(c'est corrigé)

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

No branches or pull requests

4 participants