Ajout d'une option de tracing
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é danslib/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 delib/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 utilisetrace
).
Edited by Lucas Seguinot