fix: update shutdown endpoint URL in perf test templates#2780
Merged
drewrelmas merged 1 commit intoopen-telemetry:mainfrom Apr 29, 2026
Merged
Conversation
The shutdown API endpoint was renamed from /api/v1/pipeline-groups/shutdown to /api/v1/groups/shutdown, causing perf tests to fail with 404. Closes open-telemetry#2774
drewrelmas
approved these changes
Apr 29, 2026
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2780 +/- ##
==========================================
- Coverage 86.08% 86.07% -0.01%
==========================================
Files 694 694
Lines 262919 262919
==========================================
- Hits 226341 226315 -26
- Misses 36054 36080 +26
Partials 524 524
🚀 New features to boost your workflow:
|
Merged
via the queue into
open-telemetry:main
with commit Apr 29, 2026
e07aba3
77 of 78 checks passed
pull Bot
pushed a commit
to thompson-tomo/otel-arrow
that referenced
this pull request
Apr 30, 2026
## Summary Make the pipeline performance test a required CI check so that PRs which break the perf test are caught before merge. > **Dependency**: open-telemetry#2780 must be merged first (it fixes the currently broken perf test). open-telemetry#2774 is an example of the kind of breakage this prevents — a route rename broke the perf test but the PR still merged because the perf test was not a required check. ### Changes - **rust-ci.yml**: Add `pipeline_perf_test` job (runs on `ubuntu-latest`) and include it in `rust-required-status-check` aggregator - **pipeline-perf-on-label.yaml**: Simplify to only run on dedicated Oracle bare-metal hardware when `pipelineperf` label is present — the basic validation path is removed since `rust-ci.yml` now covers it ### Motivation The pipeline perf test has been broken by merged PRs several times because it was not a required check. This change ensures that if a PR breaks the perf test (e.g. build failures, config issues, test infrastructure breakage), it is caught before merge. --------- Co-authored-by: albertlockett <a.lockett@f5.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What issue does this PR close?
Closes #2774
Change Summary
The shutdown API endpoint was renamed from
/api/v1/pipeline-groups/shutdownto/api/v1/groups/shutdown, causing pipeline perf tests to fail with 404 on the shutdown endpoint.This PR updates all 3 perf test template files:
df-loadgen-steps-docker-filtered.yaml(3 URLs)df-loadgen-steps-docker-otel.yaml(1 URL)df-loadgen-steps-docker.yaml(3 URLs)How are these changes tested?
The pipeline perf test CI job will validate the fix.
Are there any user-facing changes?
No.