Skip to content

Ajout d'une option de tracing

Lucas Seguinot requested to merge feat/go-trace into develop

Corrige https://git.canopsis.net/canopsis/go-revolution/issues/115.

À merger après https://git.canopsis.net/canopsis/go-revolution/merge_requests/217 (il va y avoir des conflits).

Pour tester :

  • lancer axe avec la variable d'environnement CPS_DEBUG_TRACE=./trace.out (par exemple)
  • interrompre axe
  • lancer BROWSER=chromium go tool trace ./trace.out (ce n'est malheureusement pas compatible avec firefox)

Il y a beaucoup de fichiers modifiés dans cette MR, mais les changements sont assez minimes :

  • lib/canopsis/debug/pprof/trace.go a été déplacé dans lib/canopsis/debug/trace.go (ça a plus de sens puisque ce package ne gère plus uniquement le profiling, mais aussi les traces d'exécution), et modifié pour ajouter l'option de tracing, et éviter d'utiliser des variables globales.
  • cmd/engine-*/main.go ont été modifiés pour prendre en compte le changement de l'interface de lib/canopsis/debug
  • les autres fichiers ont été modifiés pour ajouter des annotations, et pour ajouter un argument ctx context.Context au début des méthodes qui en ont besoin (c'est-à-dire toutes celles dans lesquelles on utilise trace).
Edited by Lucas Seguinot

Merge request reports