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

#300 - Logger et redémarrage service #320

Merged
merged 4 commits into from
Nov 14, 2024
Merged

Conversation

Gaetanbrl
Copy link
Member

@Gaetanbrl Gaetanbrl commented Oct 31, 2024

Contribution

Cette contribution est financée par la Région Bretagne :

image

Description

Cette PR contient des éléments liés à #300 et #246 :

  • Formatter des logs
  • Ajout d'un fichier de logs applicatives séparé (via modification du .service)
  • Documentation
  • Gestion des cas de cas d'erreur (.git avec lock et application XML non conforme)
  • Ajouts d'informations de logs supplémentaires (INFO et DEBUG)

@lecault
Copy link
Collaborator

lecault commented Nov 8, 2024

Premier bilan des tests :

  • Je n'ai plus de soucis pour redémarrer le service
  • J'arrive à voir les logs applicatifs
  • Quand j'ai eu un souci pour récupérer le registrer, pas trouvé un moyen simple d'identifier le problème dans les logs (à cause de l'affichage de multiples erreurs) => je vais essayer de reproduire pour mardi.

@Gaetanbrl
Copy link
Member Author

Quand j'ai eu un souci pour récupérer le registrer, pas trouvé un moyen simple d'identifier le problème dans les logs (à cause de l'affichage de multiples erreurs)

C'est un problème d'analyse des logs ou un problème de logs qui ne remontent pas l'erreur ?

@lecault
Copy link
Collaborator

lecault commented Nov 8, 2024

Quand j'ai eu un souci pour récupérer le registrer, pas trouvé un moyen simple d'identifier le problème dans les logs (à cause de l'affichage de multiples erreurs)

C'est un problème d'analyse des logs ou un problème de logs qui ne remontent pas l'erreur ?

Le log remonte des erreurs sur des projets qui remontent dans le register.json

@Gaetanbrl
Copy link
Member Author

Le log remonte des erreurs sur des projets qui remontent dans le register.json

Ok mais ca ne suffit pas à identifier le problème c'est cela ?

@lecault
Copy link
Collaborator

lecault commented Nov 8, 2024

Le log remonte des erreurs sur des projets qui remontent dans le register.json

Ok mais ca ne suffit pas à identifier le problème c'est cela ?

Oui. Il y a des erreurs sur des applis qui remontent dans le register.json. Difficile de trouver LA vraie erreur

@Gaetanbrl
Copy link
Member Author

C'est tout un art et un peu d'expérience :p

@lecault
Copy link
Collaborator

lecault commented Nov 8, 2024

Ah Gaëtan, je viens de voir quelque chose d'où peut provenir l'erreur.
Quand j'ai mes soucis de redémarrage, il est possible que je stop un redémarrage au milieu de l'alimentation du register.json.
Et du coup, quand je remet ce register en prod, je n'ai pas tout.
Je te propose de mettre de côté cette partie là tant que je ne suis pas certain de reproduire l'erreur et que ce soit une vraie errreur.

@Gaetanbrl
Copy link
Member Author

Quand j'ai mes soucis de redémarrage, il est possible que je stop un redémarrage au milieu de l'alimentation du register.json.

Dans le mesure du possible, il faudrait que tu fasses tes restart en dehors des horaires d'utilisation, mais ce n'est pas super simple à gérer.
Dans quels cas as-tu besoin de restart ton service ?

@lecault
Copy link
Collaborator

lecault commented Nov 12, 2024

Quand j'ai mes soucis de redémarrage, il est possible que je stop un redémarrage au milieu de l'alimentation du register.json.

Dans le mesure du possible, il faudrait que tu fasses tes restart en dehors des horaires d'utilisation, mais ce n'est pas super simple à gérer. Dans quels cas as-tu besoin de restart ton service ?

Le plus courant est mise à jour de la conf ou suite à un plantage server.

@lecault
Copy link
Collaborator

lecault commented Nov 12, 2024

Ok pour un merge, un commentaire je n'ai pas vu à quoi sert ce log :

 StandardError=file:/var/log/mviewerstudio/mvs-dev-appli-error.log

@Gaetanbrl
Copy link
Member Author

Ok pour un merge, un commentaire je n'ai pas vu à quoi sert ce log :

 StandardError=file:/var/log/mviewerstudio/mvs-dev-appli-error.log

Est-ce que tu as un fichier /var/log/mviewerstudio/mvs-dev-appli-error.log ?
si oui, as-tu du contenu dedans ?

@lecault
Copy link
Collaborator

lecault commented Nov 13, 2024

Non je n'ai pas de contenu justement.

@lecault lecault added this to the 4.2 milestone Nov 13, 2024
@Gaetanbrl
Copy link
Member Author

Vu avec @lecault je merge

@Gaetanbrl Gaetanbrl merged commit 4b03fee into mviewer:develop Nov 14, 2024
1 check passed
@lecault lecault mentioned this pull request Jan 9, 2025
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.

2 participants