Commit b821c57
authored
fix(microbenchmarks): restore CI Visibility reporting to Datadog (#8483)
## Summary of changes
- Set `DD_CIVISIBILITY_AGENTLESS_ENABLED=1` on the benchmark process in
`.gitlab/benchmarks/microbenchmarks/scripts/run-benchmarks.ps1` when
`DD_API_KEY` is present, and default `DD_SITE=datadoghq.com`.
- Change the SSM parameter name in
`.gitlab/benchmarks/microbenchmarks.yml` from
`ci.dd-trace-dotnet.dd_api_key` to `ci.dd-trace-dotnet.dd_api_key-prod`
to match the key the previous pipeline was using.
## Reason for change
Microbenchmark results stopped being reported to Datadog on April 8,
2026, after #8300 moved the Windows microbenchmark runner from
`benchmarking-platform@dd-trace-dotnet/micro` into this repo. The
`Datadog.Trace.BenchmarkDotNet` exporter ships results to Datadog via CI
Visibility, and the old flow enabled agentless mode
(`DD_CIVISIBILITY_AGENTLESS_ENABLED=1`) with a production SSM key — two
details that were not carried over in the rewrite, so the exporter
silently failed to reach intake.
Cross-reference with the old script:
`benchmarking-platform/run-benchmarks.ps1:128`
(`$env:DD_CIVISIBILITY_AGENTLESS_ENABLED=1`) and
`benchmarking-platform/steps/run-windows-benchmarks.sh:9`
(`ci.${CI_PROJECT_NAME}.dd_api_key-prod`).
## Implementation details
- `run-benchmarks.ps1` now guards the agentless toggle on `DD_API_KEY`
being set, so local/dev invocations without a key continue to behave as
before (warn and skip agentless) rather than attempting — and failing —
to hit intake.
- No changes to `Build.cs`, tracer code, or the benchmark projects. This
is purely a CI config fix.
## Test coverage
- Verified the execution order in `bp-runner.windows.yml`: the env vars
are set in `run-benchmarks.ps1` after
`BuildTracerHome`/`BuildBenchmarks` and before the BDN CLI is invoked —
i.e., inherited by every BDN child process.
## Other details1 parent d5e3bbf commit b821c57
2 files changed
Lines changed: 14 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
106 | 106 | | |
107 | 107 | | |
108 | 108 | | |
109 | | - | |
| 109 | + | |
110 | 110 | | |
111 | 111 | | |
112 | 112 | | |
| |||
Lines changed: 13 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
114 | 114 | | |
115 | 115 | | |
116 | 116 | | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
117 | 130 | | |
118 | 131 | | |
119 | 132 | | |
| |||
0 commit comments