Parametrize gitlab org#8426
Conversation
BenchmarksBenchmark execution time: 2026-04-13 10:14:20 Comparing candidate commit 7b62df1 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 27 metrics, 0 unstable metrics, 87 known flaky benchmarks.
|
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (8426) and master. ✅ No regressions detected - check the details below Full Metrics ComparisonFakeDbCommand
HttpMessageHandler
Comparison explanationExecution-time benchmarks measure the whole time it takes to execute a program, and are intended to measure the one-off costs. Cases where the execution time results for the PR are worse than latest master results are highlighted in **red**. The following thresholds were used for comparing the execution times:
Note that these results are based on a single point-in-time result for each branch. For full results, see the dashboard. Graphs show the p99 interval based on the mean and StdDev of the test run, as well as the mean value of the run (shown as a diamond below the graph). Duration chartsFakeDbCommand (.NET Framework 4.8)gantt
title Execution time (ms) FakeDbCommand (.NET Framework 4.8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8426) - mean (72ms) : 69, 75
master - mean (72ms) : 69, 74
section Bailout
This PR (8426) - mean (76ms) : 73, 78
master - mean (76ms) : 75, 78
section CallTarget+Inlining+NGEN
This PR (8426) - mean (1,065ms) : 1023, 1108
master - mean (1,066ms) : 1023, 1110
FakeDbCommand (.NET Core 3.1)gantt
title Execution time (ms) FakeDbCommand (.NET Core 3.1)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8426) - mean (112ms) : 108, 116
master - mean (111ms) : 108, 115
section Bailout
This PR (8426) - mean (112ms) : 110, 114
master - mean (113ms) : 110, 116
section CallTarget+Inlining+NGEN
This PR (8426) - mean (781ms) : 761, 801
master - mean (783ms) : 763, 803
FakeDbCommand (.NET 6)gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8426) - mean (99ms) : 95, 102
master - mean (99ms) : 95, 103
section Bailout
This PR (8426) - mean (101ms) : 98, 103
master - mean (99ms) : 97, 102
section CallTarget+Inlining+NGEN
This PR (8426) - mean (935ms) : 892, 979
master - mean (931ms) : 903, 960
FakeDbCommand (.NET 8)gantt
title Execution time (ms) FakeDbCommand (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8426) - mean (98ms) : 95, 101
master - mean (98ms) : 95, 101
section Bailout
This PR (8426) - mean (100ms) : 98, 102
master - mean (98ms) : 96, 100
section CallTarget+Inlining+NGEN
This PR (8426) - mean (817ms) : 777, 857
master - mean (811ms) : 778, 844
HttpMessageHandler (.NET Framework 4.8)gantt
title Execution time (ms) HttpMessageHandler (.NET Framework 4.8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8426) - mean (191ms) : 188, 195
master - mean (191ms) : 188, 195
section Bailout
This PR (8426) - mean (195ms) : 193, 197
master - mean (195ms) : 193, 196
section CallTarget+Inlining+NGEN
This PR (8426) - mean (1,151ms) : 1090, 1213
master - mean (1,141ms) : 1097, 1185
HttpMessageHandler (.NET Core 3.1)gantt
title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8426) - mean (275ms) : 269, 281
master - mean (273ms) : 269, 277
section Bailout
This PR (8426) - mean (275ms) : 272, 279
master - mean (274ms) : 271, 277
section CallTarget+Inlining+NGEN
This PR (8426) - mean (933ms) : 913, 953
master - mean (925ms) : 905, 944
HttpMessageHandler (.NET 6)gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8426) - mean (269ms) : 265, 273
master - mean (268ms) : 265, 272
section Bailout
This PR (8426) - mean (268ms) : 266, 271
master - mean (268ms) : 265, 270
section CallTarget+Inlining+NGEN
This PR (8426) - mean (1,136ms) : 1098, 1174
master - mean (1,136ms) : 1093, 1179
HttpMessageHandler (.NET 8)gantt
title Execution time (ms) HttpMessageHandler (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8426) - mean (267ms) : 262, 271
master - mean (267ms) : 262, 271
section Bailout
This PR (8426) - mean (267ms) : 263, 270
master - mean (266ms) : 263, 270
section CallTarget+Inlining+NGEN
This PR (8426) - mean (1,024ms) : 986, 1063
master - mean (1,018ms) : 979, 1058
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
## Summary of changes Follow-up to #8426. - Parameterize the remaining internal `DDOCTOSTS_SCOPE` in `macrobenchmarks.yml`. - Tag the three `include: project: 'DataDog/...'` lines as `MIGRATION TOUCHPOINT`s. These can't be parameterized — GitLab evaluates `include:` before top-level `variables:` is in scope ([docs](https://docs.gitlab.com/ee/ci/yaml/includes.html)) — so the comments make them greppable at cutover. ## Reason for change GitHub org / GitLab group split mitigation. ## Implementation details - `.gitlab-ci.yml:14` and `.gitlab/benchmarks/macrobenchmarks.yml:1` — touchpoint comments. - `.gitlab/benchmarks/macrobenchmarks.yml:577` — `DDOCTOSTS_SCOPE` → `"${GITHUB_ORG_INTERNAL}/benchmarking-platform"`. ## Test coverage No behavior change: `GITHUB_ORG_INTERNAL` defaults to `DataDog`. ## Other details Non-URL mitigations (runners, registry, `id_tokens`, schedules/secrets recreation) need infra coordination and are out of scope.
Summary of changes
Parameterize the GitLab group path for cross-repo references in CI, preparing for the GitHub org split and GitLab instance reorganization.
Reason for change
GitHub org split and GitLab instance reorganization.
Implementation details
.gitlab-ci.yml:GITLAB_GROUP: DataDog— for internal repo referencesGITLAB_GROUP_OSS: DataDog— GitLab group for repos mirrored from the OSS GitHub org - may differ from GITLAB_GROUP after org splitDataDogwith${GITLAB_GROUP}for internal repos:DataDogwith${GITLAB_GROUP_OSS}for OSS repos:Test coverage
Other details