Perso je ne l’avais jamais vu sur pull/5812/head quelque soit le moment où j’avais regardé
Bon dans tous les cas j’ai déployé un hook sur update qui interdit tous les non fast forward (dont les --force)
Si ça essaye d’envoyer du force ça devrait être bloqué
Je me base sur How to use the update hook
Sur toutes les branches ??
Hello
Pour le suivi, vu qu’une partie a été discutée sur IRC.
Le script a été adapté pour essayer de prendre en compte le --force[-with-lease]
Du point de vu du serveur, on ne sait pas qu’on force les commits. On précise juste au serveur qu’on veut pousser un commit dont le parent diverge de celui de référence. (dit autrement no fast forward).
Dans le cas d’un force-with-lease, on peut faire une grossière approximation en considérant que l’auteur est le même entre les 2 commits.
Mais je crains que ça revalide la cas initial où on essaye de protéger de certains push force non identifié.
Il faudrait probablement affiner en ciblant spécifiquement les branches de type PR (non main/master), à affiner selon les retours.
On a aussi constaté qu’un script salvatore a été modifié courant décembre, à voir si cela peut avoir un impact ou non.
- tu ajoutes des hook en affirmant que Salvatore ferait des
--force, ce qui n’est pas le cas. Alors d’où viennent ils s’il y a effectivement des--forcequi passent ? Est-ce une action de Gitea ? - aussi, des PR de spip/spip sont en erreur «Cette demande de fusion est impossible par manque d’informations de bifurcation. » suite à leur rebase. Est-ce qu’on peut savoir ce qu’il se passe ? (je t’assure que ça ne vient pas de la façon dont on fait les commits… push, force, peut importe, y a pas de raison…)
- on a tenté des merges de PR (depuis Gitea) qui ont été déclaré fusionnées dans master (en voyant passer les notifs), qui n’y apparaissent finalement pas… Ça n’a pas à voir avec Salvatore là du coup… Est-ce qu’on a une idée de ce qu’il se passe là aussi ?
Ça fait beaucoup de soucis je trouve 
Les fichiers de crons qui ont été déplacés ; les scripts derrière n’ont pas bougé, et ils sont toujours sur trad-lang/spip-cli at master - trad-lang - SPIP on GIT
Hello
Non je n’affirme rien concernant Salvatore, je n’ai aucune idée de comment il fonctionne.
Ma seule affirmation est un git push --force-[with-lease] a lieu, ni plus ni moins. C’est ce qu’indique le reflog précédent.
Coté serveur on ne peut connaitre la nature du force, c’est uniquenet un no fast forward qui est envoyé.
La dernière fois qu’on a eu ce problème, c’était l’IDE d’un des contributeurs qui avait la fonctionnalité --force d’active par défaut.
Gitea de son coté ne force rien il réceptionne les commits et les affiche. Sur ce cas de figure en l’état je vois qui incrimine directement un comportement de la forge.
Je rappelle que la configuration de la forge voulue par la communauté, c’est un mode open bar de type administrateur.
Par conséquent tous les profils team peuvent faire ce qu’il veulent sur les dépots. Gitea acceptera sans broncher. N’importe quel compte de l’organisation peut être à l’origine du problème.
Concernant les 2 points concernant les PR, c’est à mon avis l’effet de bord du premier point. Vu qu’un force a lieu les commits des PR et autres deviennent orphelins.
De ce que je vois en regardant les différents point abordé, je retombe souvent sur le commit orphelin build: ecrire/boostrap/config doit être présent dans le zip généré par Composer · c2652d58d9 - spip - SPIP on GIT qui devrait être sur master de ce que j’ai suivi.
Les PR/commits suivants s’appuyant dessus, sont de même orphelins.
En l’état ce que j’ai fait/proposé c’est de mettre une rustine de type hook, pour réduire les effet d’un push force. Ce hook restreint fortement les push peu importe la provenance des contributions.
La seule exception à cette interdiction est le fait que le commit source serveur et local soit du même auteur. Ce qui permet d’accepter le cas classique du force-with-lease. (Mais avec plein d’effet de bord comme expliqué plus tôt)
Grâce au reflog je peux « restaurer » les commits orphelins et via la base de donnée indiquer les informations de commits actualisées.
Il vient de la PR suivante, mergée avec Gitea : #5817 - build: ecrire/boostrap/config doit être présent dans le zip généré par Composer - spip - SPIP on GIT
En quoi il serait orphelin ?
Oui, et pour moi c’est un problème effectivement que ce soit open-bar… D’ailleurs on en avait discuté en juillet pour revoir ça (réduire la liste des personnes pouvant commiter dessus)… mais bon…
En tout cas, est-ce que là on peut merger des PR de nouveau ? on a le feu vert ?
En plus il faut qu’on rattrape celles qui se sont mal passé…
J’essayais de cherry-pick pour refaire une branche de la PR #5819 - fix: HTML valide pour le formulaire `configurer_redacteurs` - spip - SPIP on GIT
Et en fait, impossible de récupérer les commits en local !
Que ce soit git fetch ou checkout…
git pull
git show 5ace3dd0c96d896924240c0275da7725244e8aca
> fatal : bad object 5ace3dd0c96d896924240c0275da7725244e8aca
# OK je n’ai pas cette référence en local… je la récupère
git fetch origin 5ace3dd0c96d896924240c0275da7725244e8aca
> erreur : Le serveur n'autorise pas de requête pour l'objet 5ace3dd0c96d896924240c0275da7725244e8aca non annoncé
Le commit est là fix: HTML valide pour le formulaire `configurer_redacteurs` · 5ace3dd0c9 - spip - SPIP on GIT et on le voit aussi dans le graph https://git.spip.net/spip/spip/graph
Je ne vois pas ce que je fais mal !
Hello
Là je ne matrise plus assez les arcanes de git, mais comme ils sont orphelins coté serveur, je ne pense pas qu’ils soient disponibles à distance.
Je dois pouvoir le faire en direct sur le dépot soit en faisant un clone serveur pour avoir les objets.
Peux tu me confirmer que :
- pour salvatore on peut repousser tous les commits depuis le bot ?
- sinon je dois pouvoir les rejouer depuis les reflog , aurais tu les commits/horaire associés ?
- pour les PR nous avons à rattraper :
- Les PR sont bien en rebase sur master ? donc sans de commit de merge ?
Est ce qu’il y en a d’autres PR ?
Mais Salvatore, il n’y a rien à rejouer, les commits sont arrivés dans master hier midi. C’est bon.
Pour les PR.
- Non #5817 est aussi dans master.
- Là il y a #5819 à reprendre oui, mais je peux le faire en exportant les .patch depuis Gitea aussi. Je vais le faire comme cela.
Et pour info sur d’autres lieux j’ai déjà récupéré des commits orphelins via la procédure du dessus, je ne sais pas pourquoi ça bloque ici.
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
Bien ! A priori tout est revenu dans le game ! Finger crossing !
Merci pour les investigations @azerttyu
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.
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.
1 « J'aime »
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
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…
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
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
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 ?
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.