tektoncd/pipeline

feat(tracing): add spans for When Expression evaluation

Ouverte

#9 782 ouverte le 16 avr. 2026

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (1 personne assignée)Go (1 943 forks)auto 404
good first issuekind/feature

Métriques du dépôt

Stars
 (9 013 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Feature request

Add OpenTelemetry spans to the When Expression evaluation path. CEL compilation/evaluation and Input/Operator/Values matching are currently invisible in traces, making it impossible to debug why tasks were skipped.

Use case

When a PipelineRun skips tasks due to When Expressions, the trace shows the parent reconcile span but gives no visibility into which expression evaluated to false or how long CEL compilation took.

Implementation hints

Key functions to instrument:

  1. pkg/reconciler/pipelinerun/pipelinerun.go:746 — EvaluateCEL() call for DAG tasks (inside loop)
  2. pkg/reconciler/pipelinerun/pipelinerun.go:965 — EvaluateCEL() call for finally tasks
  3. pkg/reconciler/pipelinerun/resources/pipelinerunresolution.go:104 — EvaluateCEL() function itself — CEL env creation, compilation, evaluation
  4. pkg/reconciler/pipelinerun/resources/pipelinerunresolution.go:488 — skipBecauseWhenExpressionsEvaluatedToFalse()
  5. pkg/apis/pipeline/v1/when_types.go:129 — AllowsExecution()

Span attributes to add: expression text, result (true/false), skip reason.

Parent spans exist: reconcile and runNextSchedulableTask are already spanned.

Part of umbrella: #9701

good first issue — focused scope, 2-3 functions, clear parent spans.

Guide contributeur

Direction de recherche
Identifiez les fonctions clés pour l'évaluation des When Expression (EvaluateCEL, AllowsExecution, skipBecauseWhenExpressionsEvaluatedToFalse) et ajoutez des spans OpenTelemetry avec des attributs pour le texte de l'expression, le résultat et la raison du saut. Suivez les spans parents existants (reconcile, runNextSchedulableTask) pour garantir un imbrication correcte.
Stack technique
go
Domaine
backendobservability
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GoOpenTelemetry
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80