Skip to content

Commit 67435a9

Browse files
committed
chore(ci): Tweak thresholds
1 parent 92f9783 commit 67435a9

File tree

2 files changed

+23
-8
lines changed

2 files changed

+23
-8
lines changed

.gitlab/benchmarks/bp-runner.fail-on-breach.yml

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -12,28 +12,37 @@ experiments:
1212
# Measurements
1313
# https://benchmarking.us1.prod.dog/trends?projectId=4&branch=master&trendsTab=per_scenario
1414
scenarios:
15+
# Note that thresholds there are choosen based the confidence interval with a 10% adjustment.
1516

1617
# Standard macrobenchmarks
18+
# https://benchmarking.us1.prod.dog/trends?projectId=4&branch=master&trendsTab=per_scenario&scenario=normal_operation%2Fonly-tracing&trendsType=scenario
1719
- name: normal_operation/only-tracing
1820
thresholds:
19-
- agg_http_req_duration_p50 < 2.28 ms
20-
- agg_http_req_duration_p99 < 7.45 ms
21+
- agg_http_req_duration_p50 < 2.36 ms
22+
- agg_http_req_duration_p99 < 7.89 ms
23+
# https://benchmarking.us1.prod.dog/trends?projectId=4&branch=master&trendsTab=per_scenario&scenario=normal_operation%2Fotel-latest&trendsType=scenario
2124
- name: normal_operation/otel-latest
2225
thresholds:
23-
- agg_http_req_duration_p50 < 2.28 ms
24-
- agg_http_req_duration_p99 < 9.00 ms
26+
- agg_http_req_duration_p50 < 2.34 ms
27+
- agg_http_req_duration_p99 < 9.50 ms
2528

29+
# https://benchmarking.us1.prod.dog/trends?projectId=4&branch=master&trendsTab=per_scenario&scenario=high_load%2Fonly-tracing&trendsType=scenario
2630
- name: high_load/only-tracing
2731
thresholds:
28-
- throughput > 1400.0 op/s
32+
- throughput > 1100.0 op/s
33+
# https://benchmarking.us1.prod.dog/trends?projectId=4&branch=master&trendsTab=per_scenario&scenario=high_load%2Fotel-latest&trendsType=scenario
2934
- name: high_load/otel-latest
3035
thresholds:
31-
- throughput > 1400.0 op/s
36+
- throughput > 1100.0 op/s
3237

3338
# Startup macrobenchmarks
39+
# https://benchmarking.us1.prod.dog/trends?projectId=4&branch=master&trendsTab=per_scenario&scenario=startup%3Apetclinic%3Atracing%3AGlobalTracer&trendsType=scenario
40+
# https://benchmarking.us1.prod.dog/trends?projectId=4&branch=master&trendsTab=per_scenario&scenario=startup%3Apetclinic%3Aappsec%3AGlobalTracer&trendsType=scenario
41+
# https://benchmarking.us1.prod.dog/trends?projectId=4&branch=master&trendsTab=per_scenario&scenario=startup%3Apetclinic%3Aiast%3AGlobalTracer&trendsType=scenario
3442
- name: "startup:petclinic:(tracing|appsec|iast):GlobalTracer"
3543
thresholds:
36-
- execution_time < 245 ms
44+
- execution_time < 260 ms
45+
# https://benchmarking.us1.prod.dog/trends?projectId=4&branch=master&trendsTab=per_scenario&scenario=startup%3Apetclinic%3Aprofiling%3AGlobalTracer&trendsType=scenario
3746
- name: "startup:petclinic:profiling:GlobalTracer"
3847
thresholds:
3948
- execution_time < 368 ms

.gitlab/macrobenchmarks.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,12 @@ include:
66
.macrobenchmarks:
77
stage: macrobenchmarks
88
rules:
9+
# - if: $POPULATE_CACHE
10+
# when: never
11+
# - if: ($NIGHTLY_BENCHMARKS || $CI_PIPELINE_SOURCE != "schedule") && $CI_COMMIT_REF_NAME == "master"
12+
# when: always
13+
# - when: manual
14+
# allow_failure: true
915
- when: on_success # TODO: PLEASE revert before merging the PR
1016
tags: ["runner:apm-k8s-same-cpu"]
1117
needs: ["build"]
@@ -96,7 +102,7 @@ check-slo-breaches:
96102
# Need to move the artifacts the benchmarks-* job
97103
- |
98104
export BENCHMARKS_ARTIFACTS_DIR="$(pwd)/reports" && mkdir -p "${BENCHMARKS_ARTIFACTS_DIR}"
99-
for benchmarkType in startup load; do
105+
for benchmarkType in startup load dacapo; do
100106
find "$BENCHMARKS_ARTIFACTS_DIR/$benchmarkType" -name "benchmark-baseline.json" -o -name "benchmark-candidate.json" | while read file; do
101107
relpath="${file#$BENCHMARKS_ARTIFACTS_DIR/$benchmarkType/}"
102108
prefix="${relpath%/benchmark-*}" # Remove the trailing /benchmark-(baseline|candidate).json

0 commit comments

Comments
 (0)