Skip to content

Commit 9abeefa

Browse files
committed
Enhance observability: wire up metrics for reload count, access log drops, and tracing spans
1 parent 0eec113 commit 9abeefa

37 files changed

+1787
-570
lines changed

.github/workflows/pipeline.yaml

Lines changed: 439 additions & 0 deletions
Large diffs are not rendered by default.

Cargo.lock

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

bench/cases/system/config_reload_convergence.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,7 @@ main() {
293293
pod_label=$(get_proxy_pod_label)
294294
proxy_port=$(get_proxy_port)
295295

296-
local output_dir="${BENCH_OUTPUT_DIR}/${BENCH_MODE}/${BENCH_PROXY}/${CASE_NAME}"
296+
local output_dir="${BENCH_OUTPUT_DIR}/${BENCH_MODE}/${BENCH_PROXY}/${CASE_NAME}${BENCH_CASE_SUFFIX:+__${BENCH_CASE_SUFFIX}}"
297297
ensure_dir "$output_dir"
298298

299299
# Setup port-forward to access test backend (for load generation only)

bench/cases/system/multi_hour_soak.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ main() {
3636
container_name=$(get_proxy_container_name)
3737
proxy_port=$(get_proxy_port)
3838

39-
local output_dir="${BENCH_OUTPUT_DIR}/${BENCH_MODE}/${BENCH_PROXY}/${CASE_NAME}"
39+
local output_dir="${BENCH_OUTPUT_DIR}/${BENCH_MODE}/${BENCH_PROXY}/${CASE_NAME}${BENCH_CASE_SUFFIX:+__${BENCH_CASE_SUFFIX}}"
4040
ensure_dir "$output_dir"
4141

4242
# Setup port-forward to access test backend

bench/cases/system/rollback_performance.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ main() {
151151
pod_label=$(get_proxy_pod_label)
152152
proxy_port=$(get_proxy_port)
153153

154-
local output_dir="${BENCH_OUTPUT_DIR}/${BENCH_MODE}/${BENCH_PROXY}/${CASE_NAME}"
154+
local output_dir="${BENCH_OUTPUT_DIR}/${BENCH_MODE}/${BENCH_PROXY}/${CASE_NAME}${BENCH_CASE_SUFFIX:+__${BENCH_CASE_SUFFIX}}"
155155
ensure_dir "$output_dir"
156156

157157
# Setup port-forward to access test backend

bench/cases/system/stress_recovery.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ main() {
131131
container_name=$(get_proxy_container_name)
132132
proxy_port=$(get_proxy_port)
133133

134-
local output_dir="${BENCH_OUTPUT_DIR}/${BENCH_MODE}/${BENCH_PROXY}/${CASE_NAME}"
134+
local output_dir="${BENCH_OUTPUT_DIR}/${BENCH_MODE}/${BENCH_PROXY}/${CASE_NAME}${BENCH_CASE_SUFFIX:+__${BENCH_CASE_SUFFIX}}"
135135
ensure_dir "$output_dir"
136136

137137
# Setup port-forward to access test backend

bench/config/standalone/pavis.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ telemetry:
88
pingora: "error"
99
service_name: "pavis-benchmark"
1010
access_log: false
11+
metrics: "0.0.0.0:9090"
1112

1213
upstreams:
1314
- name: "backend"

bench/config/system/pavis/pavis.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ telemetry:
88
pingora: "error"
99
service_name: "pavis-benchmark"
1010
access_log: false
11+
metrics: "0.0.0.0:9090"
1112

1213
upstreams:
1314
- name: "backend"

bench/config/system/pavis/relay-configmap.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ data:
6565
pingora: "error"
6666
service_name: "pavis-benchmark"
6767
access_log: false
68+
metrics: "0.0.0.0:9090"
6869
6970
upstreams:
7071
- name: "backend"

bench/config/system/pavis/test-workload.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,9 @@ spec:
7373
- name: proxy
7474
containerPort: 8080
7575
protocol: TCP
76+
- name: metrics
77+
containerPort: 9090
78+
protocol: TCP
7679
env:
7780
- name: PAVIS_RELAY_URL
7881
value: "http://pavis-relay:8090"

0 commit comments

Comments
 (0)