test: add e2e test for OpenCensus to OpenTelemetry metrics migration#10355
test: add e2e test for OpenCensus to OpenTelemetry metrics migration#10355khrm wants to merge 1 commit into
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
72e2291 to
f0a1fed
Compare
There was a problem hiding this comment.
Pull request overview
Adds a new Go e2e integration test in ./test to validate the OpenCensus → OpenTelemetry metrics migration from PR #9043 by creating a representative set of TaskRuns/PipelineRuns and scraping the controller’s /metrics endpoint to assert expected metric presence/absence and basic value characteristics.
Changes:
- Introduces a consolidated e2e test that provisions successful/failed/cancelled/timed TaskRuns and several PipelineRun variants.
- Scrapes controller Prometheus metrics via the apiserver pod proxy and parses metric families for assertions.
- Adds checks for renamed infrastructure metrics (e.g.,
kn_workqueue_*,go_*) and confirms removal of legacy reconcile metrics.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
e83d83a to
eaf005e
Compare
eaf005e to
4ef25aa
Compare
4ef25aa to
a7ff17d
Compare
a7ff17d to
36a9fb7
Compare
ce5e2e4 to
0edfb95
Compare
khrm
left a comment
There was a problem hiding this comment.
Somehow build is failing in CI. But locally everything works.
SYSTEM_NAMESPACE=tekton-pipelines go test -v -tags=e2e -count=1 -timeout=15m -run ^TestOTelMetrics$ ./test/
|
Somehow tests are failing with latest |
ad5f2fd to
48e2d3e
Compare
Verifies PR tektoncd#9043's OTel migration by creating TaskRuns (success, fail, cancel, timed) and PipelineRuns (single-task, multi-task, cancel), then scraping the controller /metrics endpoint to assert counter values, histogram observation counts and bucket placement, gauge presence, metric renames (kn_workqueue_*, go_*), and removed metrics. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
48e2d3e to
f5b3a2f
Compare
Verifies PR #9043's OTel migration by creating TaskRuns (success, fail, cancel, timed) and PipelineRuns (single-task, multi-task, cancel), then scraping the controller /metrics endpoint to assert counter values, histogram observation counts and bucket placement, gauge presence, metric renames (kn_workqueue_, go_), and removed metrics.
Changes
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
/kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes