Salvatore bégaie

marcimat · Janvier 7, 2024, 2:23

J’ai envoyé la branche issue_1818_bis mais la PR associée n’a pas l’air contente…

#5822 - Fix #5818 (on retente) - spip - SPIP on GIT donc. Il ne montre même pas les commits

marcimat · Janvier 7, 2024, 3:07

Bien ! A priori tout est revenu dans le game ! Finger crossing !
Merci pour les investigations @azerttyu

azerttyu · Janvier 7, 2024, 5:03

Hello

Pour le suivi ; avec @marcimat on a fait plusieurs vérifications.
Pour l’historique on n’a malheureusement pas assez d’éléments pour savoir ce qui s’est passé.
L’hypothèse d’un push --force reste la plus probable mais on n’a pas trouvé d’élément permettant de l’affirmer ou de mieux cibler le point d’origine.

Le hook de protection est toujours actif et a été adapté suite aux tests réalisés. Le mode verbeux est toujours actif pour avoir des informations si le problème revient.

On a rejoué à la main la PR cassé et vérifié que coté gitea tout se comportait comme prévu, cela semble bien le cas.

En l’état on surveille et on avisera selon.

b_b · Janvier 7, 2024, 6:38

Merci à vous deux pour le temps passé là dessus, il est temps qu’on lance le débat à propos de la restriction de commit direct sur master, je m’y colle dès que j’ai un moment et de la motivation.

Jack31 · Février 2, 2024, 7:32

Salvatore recommence…
Les trois messages de Salvatore sur SPIP au nom de Serge sont répétés depuis une semaine. Par exemple i18n: [Salvatore] [source:ecrire/lang/ ecrire] Export depuis https://trad.spip.net de la langue uk · b6d2959516 - spip - SPIP on GIT
Les commits ne sont pas intégrés dans master, la ligne 9 par exemple

- 'activer_plugin' => 'Активувати плагін',
+ 'activer_plugin' => 'Увімкнути плагін',

reste inchangée spip/ecrire_uk.php at master - spip - SPIP on GIT

marcimat · Février 2, 2024, 8:54

Bon donc il semble que

@azerttyu a enlevé son hook a priori (?) il y a quelques jours qui devait peut être bloquer salvatore ; ce faisant salvatore recommit dans spip/spip (et c’est bien) mais ça plante sur spip/spip comme avant
1.bis) il a aussi enlevé un cache redis, ce qui semble du coup ne pas avoir non plus corrigé le problème
on voit que les commits ne sont pas dans master de spip/spip (mais présents sur le repos git), comme on avait l’autre jour et eu sur d’autres PR
j’avais ajouté plus de logs dans salvatore, du coup je vois bien que de notre point de vue, le retour du push indique que les commits sont bien arrivés dans master (ce qui confirme aussi ce que disent les logs de notif de shiraz — qui s’appuie sur une notif de gitea)

2024-02-02 01:30:01: =======================================
2024-02-02 01:30:01: POUSSEUR [Prend les fichiers langue dans sa copie locale et les commite+push SVN|GIT]
2024-02-02 01:30:01: =======================================
2024-02-02 01:30:01: 658 modules dans le fichier traductions salvatore/traductions/traductions.txt
2024-02-02 01:30:01: 
2024-02-02 01:30:01: --- Module spip | spip--spip-4b393 | https://git.spip.net/spip/spip.git
2024-02-02 01:30:01: Commit de Serge Markitanenko <ss1945@gmail.com> :ecrire/lang/spip_uk.php, ecrire/lang/spip.xml
2024-02-02 01:30:01:    i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk
2024-02-02 01:30:01:    i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net
2024-02-02 01:30:01: > git add 'ecrire/lang/spip_uk.php' 'ecrire/lang/spip.xml' 2>&1
2024-02-02 01:30:01: > git commit -m 'i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk
2024-02-02 01:30:01: i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net' --author='Serge Markitanenko <ss1945@gmail.com>' 2>&1
2024-02-02 01:30:01: [master 48af8183a] i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net
2024-02-02 01:30:01:  Author: Serge Markitanenko <ss1945@gmail.com>
2024-02-02 01:30:01:  2 files changed, 26 insertions(+), 7 deletions(-)
2024-02-02 01:30:06: > git log --graph --pretty=tformat:'%h %cd -%d %s (%an)' --abbrev-commit --date=short -3
2024-02-02 01:30:06: * 48af8183a 2024-02-02 - (HEAD -> master) i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:06: * e115480d9 2024-02-01 - (origin/master, origin/HEAD) docs(changelog): pour #5847 (Matthieu Marcillaud)
2024-02-02 01:30:06: * f9540c756 2024-02-01 - fix: Tolérer un zéro dans l’analyse de certains critères `{critere 0,5}` (Matthieu Marcillaud)
2024-02-02 01:30:06: > git fetch --all
2024-02-02 01:30:06: Fetching origin
2024-02-02 01:30:06: > git stash -q 2>&1
2024-02-02 01:30:06: > git pull --rebase 2>&1
2024-02-02 01:30:06: Current branch master is up to date.
2024-02-02 01:30:06: > bash -c 'if [[ $(git stash list) ]]; then git stash pop -q 2>&1; fi;'
2024-02-02 01:30:06: > git log --graph --pretty=tformat:'%h %cd -%d %s (%an)' --abbrev-commit --date=short -3
2024-02-02 01:30:06: * 48af8183a 2024-02-02 - (HEAD -> master) i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:06: * e115480d9 2024-02-01 - (origin/master, origin/HEAD) docs(changelog): pour #5847 (Matthieu Marcillaud)
2024-02-02 01:30:06: * f9540c756 2024-02-01 - fix: Tolérer un zéro dans l’analyse de certains critères `{critere 0,5}` (Matthieu Marcillaud)
2024-02-02 01:30:06: > git push 2>&1
2024-02-02 01:30:06: remote: . Processing 1 references
2024-02-02 01:30:06: remote: Processed 1 references in total
2024-02-02 01:30:06: To https://git.spip.net/spip/spip.git
2024-02-02 01:30:06:    e115480d9..48af8183a  master -> master
2024-02-02 01:30:06: > git log --graph --pretty=tformat:'%h %cd -%d %s (%an)' --abbrev-commit --date=short -3
2024-02-02 01:30:06: * 48af8183a 2024-02-02 - (HEAD -> master, origin/master, origin/HEAD) i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:06: * e115480d9 2024-02-01 - docs(changelog): pour #5847 (Matthieu Marcillaud)
2024-02-02 01:30:06: * f9540c756 2024-02-01 - fix: Tolérer un zéro dans l’analyse de certains critères `{critere 0,5}` (Matthieu Marcillaud)
2024-02-02 01:30:06: Module spip OK
2024-02-02 01:30:06:
2024-02-02 01:30:06: 
2024-02-02 01:30:06: --- Module ecrire | ecrire--spip-4b393 | https://git.spip.net/spip/spip.git
2024-02-02 01:30:06: Commit de Serge Markitanenko <ss1945@gmail.com> :ecrire/lang/ecrire_uk.php, ecrire/lang/ecrire.xml
2024-02-02 01:30:06:    i18n: [Salvatore] [source:ecrire/lang/ ecrire] Export depuis https://trad.spip.net de la langue uk
2024-02-02 01:30:06:    i18n: [Salvatore] [source:ecrire/lang/ ecrire] Mise a jour du bilan depuis https://trad.spip.net
2024-02-02 01:30:06: > git add 'ecrire/lang/ecrire_uk.php' 'ecrire/lang/ecrire.xml' 2>&1
2024-02-02 01:30:06: > git commit -m 'i18n: [Salvatore] [source:ecrire/lang/ ecrire] Export depuis https://trad.spip.net de la langue uk
2024-02-02 01:30:06: i18n: [Salvatore] [source:ecrire/lang/ ecrire] Mise a jour du bilan depuis https://trad.spip.net' --author='Serge Markitanenko <ss1945@gmail.com>' 2>&1
2024-02-02 01:30:06: [master b6d295951] i18n: [Salvatore] [source:ecrire/lang/ ecrire] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ ecrire] Mise a jour du bilan depuis https://trad.spip.net
2024-02-02 01:30:06:  Author: Serge Markitanenko <ss1945@gmail.com>
2024-02-02 01:30:06:  2 files changed, 20 insertions(+), 7 deletions(-)
2024-02-02 01:30:11: > git log --graph --pretty=tformat:'%h %cd -%d %s (%an)' --abbrev-commit --date=short -3
2024-02-02 01:30:11: * b6d295951 2024-02-02 - (HEAD -> master) i18n: [Salvatore] [source:ecrire/lang/ ecrire] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ ecrire] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:11: * 48af8183a 2024-02-02 - (origin/master, origin/HEAD) i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:11: * e115480d9 2024-02-01 - docs(changelog): pour #5847 (Matthieu Marcillaud)
2024-02-02 01:30:11: > git fetch --all
2024-02-02 01:30:11: Fetching origin
2024-02-02 01:30:11: > git stash -q 2>&1
2024-02-02 01:30:11: > git pull --rebase 2>&1
2024-02-02 01:30:11: Current branch master is up to date.
2024-02-02 01:30:11: > bash -c 'if [[ $(git stash list) ]]; then git stash pop -q 2>&1; fi;'
2024-02-02 01:30:11: > git log --graph --pretty=tformat:'%h %cd -%d %s (%an)' --abbrev-commit --date=short -3
2024-02-02 01:30:11: * b6d295951 2024-02-02 - (HEAD -> master) i18n: [Salvatore] [source:ecrire/lang/ ecrire] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ ecrire] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:11: * 48af8183a 2024-02-02 - (origin/master, origin/HEAD) i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:11: * e115480d9 2024-02-01 - docs(changelog): pour #5847 (Matthieu Marcillaud)
2024-02-02 01:30:11: > git push 2>&1
2024-02-02 01:30:11: remote: . Processing 1 references
2024-02-02 01:30:11: remote: Processed 1 references in total
2024-02-02 01:30:11: To https://git.spip.net/spip/spip.git
2024-02-02 01:30:11:    48af8183a..b6d295951  master -> master
2024-02-02 01:30:11: > git log --graph --pretty=tformat:'%h %cd -%d %s (%an)' --abbrev-commit --date=short -3
2024-02-02 01:30:11: * b6d295951 2024-02-02 - (HEAD -> master, origin/master, origin/HEAD) i18n: [Salvatore] [source:ecrire/lang/ ecrire] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ ecrire] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:11: * 48af8183a 2024-02-02 - i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:11: * e115480d9 2024-02-01 - docs(changelog): pour #5847 (Matthieu Marcillaud)
2024-02-02 01:30:11: Module ecrire OK
2024-02-02 01:30:11:
2024-02-02 01:30:11: 
2024-02-02 01:30:11: --- Module public | public--spip-4b393 | https://git.spip.net/spip/spip.git
2024-02-02 01:30:11: Commit de Serge Markitanenko <ss1945@gmail.com> :ecrire/lang/public_uk.php, ecrire/lang/public.xml
2024-02-02 01:30:11:    i18n: [Salvatore] [source:ecrire/lang/ public] Export depuis https://trad.spip.net de la langue uk
2024-02-02 01:30:11:    i18n: [Salvatore] [source:ecrire/lang/ public] Mise a jour du bilan depuis https://trad.spip.net
2024-02-02 01:30:11: > git add 'ecrire/lang/public_uk.php' 'ecrire/lang/public.xml' 2>&1
2024-02-02 01:30:11: > git commit -m 'i18n: [Salvatore] [source:ecrire/lang/ public] Export depuis https://trad.spip.net de la langue uk
2024-02-02 01:30:11: i18n: [Salvatore] [source:ecrire/lang/ public] Mise a jour du bilan depuis https://trad.spip.net' --author='Serge Markitanenko <ss1945@gmail.com>' 2>&1
2024-02-02 01:30:11: [master 385178b03] i18n: [Salvatore] [source:ecrire/lang/ public] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ public] Mise a jour du bilan depuis https://trad.spip.net
2024-02-02 01:30:11:  Author: Serge Markitanenko <ss1945@gmail.com>
2024-02-02 01:30:11:  2 files changed, 6 insertions(+), 1 deletion(-)
2024-02-02 01:30:16: > git log --graph --pretty=tformat:'%h %cd -%d %s (%an)' --abbrev-commit --date=short -3
2024-02-02 01:30:16: * 385178b03 2024-02-02 - (HEAD -> master) i18n: [Salvatore] [source:ecrire/lang/ public] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ public] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:16: * b6d295951 2024-02-02 - (origin/master, origin/HEAD) i18n: [Salvatore] [source:ecrire/lang/ ecrire] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ ecrire] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:16: * 48af8183a 2024-02-02 - i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:16: > git fetch --all
2024-02-02 01:30:16: Fetching origin
2024-02-02 01:30:16: > git stash -q 2>&1
2024-02-02 01:30:16: > git pull --rebase 2>&1
2024-02-02 01:30:16: Current branch master is up to date.
2024-02-02 01:30:16: > bash -c 'if [[ $(git stash list) ]]; then git stash pop -q 2>&1; fi;'
2024-02-02 01:30:16: > git log --graph --pretty=tformat:'%h %cd -%d %s (%an)' --abbrev-commit --date=short -3
2024-02-02 01:30:16: * 385178b03 2024-02-02 - (HEAD -> master) i18n: [Salvatore] [source:ecrire/lang/ public] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ public] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:16: * b6d295951 2024-02-02 - (origin/master, origin/HEAD) i18n: [Salvatore] [source:ecrire/lang/ ecrire] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ ecrire] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:16: * 48af8183a 2024-02-02 - i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:16: > git push 2>&1
2024-02-02 01:30:16: remote: . Processing 1 references
2024-02-02 01:30:16: remote: Processed 1 references in total
2024-02-02 01:30:16: To https://git.spip.net/spip/spip.git
2024-02-02 01:30:16:    b6d295951..385178b03  master -> master

Et juste après là 2024-02-02 01:30:16 on voit origin/master sur le 3è commit de salvatore :

2024-02-02 01:30:16: > git log --graph --pretty=tformat:'%h %cd -%d %s (%an)' --abbrev-commit --date=short -3
2024-02-02 01:30:16: * 385178b03 2024-02-02 - (HEAD -> master, origin/master, origin/HEAD) i18n: [Salvatore] [source:ecrire/lang/ public] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ public] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:16: * b6d295951 2024-02-02 - i18n: [Salvatore] [source:ecrire/lang/ ecrire] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ ecrire] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:16: * 48af8183a 2024-02-02 - i18n: [Salvatore] [source:ecrire/lang/ spip] Export depuis https://trad.spip.net de la langue uk i18n: [Salvatore] [source:ecrire/lang/ spip] Mise a jour du bilan depuis https://trad.spip.net (Serge Markitanenko)
2024-02-02 01:30:16: Module public OK

Ce qui est aussi marrant c’est qu’à 3 temps différents 1:30:06, 1:30:11, 1:30:16, on fait un un git-fetch qui a bien les commits dans master qu’il avait poussé juste avant

Et ça perd donc les commits dans master après cela à un moment…

azerttyu · Février 2, 2024, 11:25

Hello

Je viens de réaligner le commit 385178b03 de salvatore sur master, soit
depuis e115480d9b1aa72cf41fc2aff868036e1108a13a
en passant par 48af8183a9eceed7564115cb3a304ec6742249ba, b6d29595160f418c1a7d71b8d7090bf556802794
pour finir 385178b03569f33c6177ebd52f536826d121dcae

De ce que je vois le ref/head/master n’est pas maintenu dans le temps. Si le fetch a bien récupéré l’entête c’est qu’il a bien été pris en compte à un moment.
~~Je n’arrive pas à avoir de trace sur la cause du retour arrière ou non sauvegarde du push.~~

J’ai une trace étrange

c772b7c5c951774640d96f32a484168a38a05cf3 e115480d9b1aa72cf41fc2aff868036e1108a13a marcimat <marcimat@noreply.git.spip.net> 1706784210 +0100     push
e115480d9b1aa72cf41fc2aff868036e1108a13a 48af8183a9eceed7564115cb3a304ec6742249ba Gitea <gitea@fake.local> 1706833805 +0100     push
48af8183a9eceed7564115cb3a304ec6742249ba b6d29595160f418c1a7d71b8d7090bf556802794 Gitea <gitea@fake.local> 1706833810 +0100     push
b6d29595160f418c1a7d71b8d7090bf556802794 385178b03569f33c6177ebd52f536826d121dcae Gitea <gitea@fake.local> 1706833815 +0100     push
385178b03569f33c6177ebd52f536826d121dcae e115480d9b1aa72cf41fc2aff868036e1108a13a Gitea <gitea@fake.local> 1706833834 +0100     update by push

On voit bien dans cette historique la prise en compte des commit avec les 3 actions push qui correspondent bien au timing de Salvatore.
Ensuite on voit git push update qui ramène le commit en arrière à 01:30:34 soit 20 secondes après le dernier push de salvatore

azerttyu · Février 2, 2024, 11:39

En complément je vois que le push sur github se fait entre les différents temps.
Comme il est en mode mirroir , est ce qu’il s’amuserait à faire un fetch qui casserait le truc.?
Je viens de modifier le comportement du cron. En retirant le mode mirroir de la configuration du remote et en explicitant le push

Je ne pense pas que ce soit là le problème vu que le cron tourne ainsi depuis assez longtemps et avant les incidents remontés

azerttyu · Février 3, 2024, 1:18

Hello

Le comportement différents que je vois c’est que salvatore passe par https et non ssh
Ce qui serai aussi cohérent avec certaines actions foireuse sur les merge/rebase via l’interface.

@marcimat @cerdic Est ce qu’il serait possible de modifier le comportement salvatore en passant par ssh ?

azerttyu · Février 3, 2024, 9:13

Hello

Je continue à regarder. Les logs du dépôt git montre le même comportement. Un git update by push suit les commits poussé et restore à l’état précédent.
J’ai complété le ticket coté gitea en conséquence refs/ are not updated · Issue #28986 · go-gitea/gitea · GitHub

J’ai également augmenté le niveau de log cote gitea. Je ne trouve pas de trace d’action correspondante aux horaires indiqués.