tektoncd/pipeline

perf: strip ManagedFields from PipelineRun and TaskRun informer caches

Aberta

#9.573 aberto em 15 de mar. de 2026

 (0 comentário) (1 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 SetTransform on PipelineRun and TaskRun shared informers to strip metadata.managedFields on cache insert.

Kubernetes objects carry managedFields which can be 30-70% of the object's serialized size. The Tekton controller never reads managedFields, but every DeepCopy (3-4 per reconcile cycle via the Knative reconciler framework) copies them in full.

Where to add the transform

Implementation

stripManagedFields := func(obj interface{}) (interface{}, error) {
    if accessor, ok := obj.(metav1.ObjectMetaAccessor); ok {
        accessor.GetObjectMeta().SetManagedFields(nil)
    }
    return obj, nil
}
if err := pipelineRunInformer.Informer().SetTransform(stripManagedFields); err != nil {
    logging.FromContext(ctx).Warnf("Failed to set informer transform: %v", err)
}

Use case

  • Reduces memory footprint of the controller's informer caches
  • Reduces CPU time spent on DeepCopy operations during reconciliation
  • This pattern is established in the Kubernetes ecosystem (e.g., kube-controller-manager added ManagedFields stripping in k8s 1.27)
  • SetTransform is available in client-go's shared_informer.go

Guia do colaborador