tektoncd/pipeline

feat(tracing): add spans for When Expression evaluation

Aberta

#9.782 aberto em 16 de abr. de 2026

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (1 responsável)Go (1.943 forks)auto 404
good first issuekind/feature

Métricas do repositório

Stars
 (9.013 estrelas)
Métricas de merge de PR
 (Mesclagem média 9d 3h) (137 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Identifique as funções principais para avaliação de When Expression (EvaluateCEL, AllowsExecution, skipBecauseWhenExpressionsEvaluatedToFalse) e adicione spans do OpenTelemetry com atributos para texto da expressão, resultado e motivo de ignorar. Siga os spans pai existentes (reconcile, runNextSchedulableTask) para garantir aninhamento adequado.
Pilha de tecnologia
go
Domain
backendobservability
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GoOpenTelemetry
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80