Skip to content

ci: run RPC end-to-end check on local cluster - #74

Merged
sshaplygin merged 1 commit into
mainfrom
codex/issue-64-rpc-e2e
Aug 16, 2026
Merged

ci: run RPC end-to-end check on local cluster#74
sshaplygin merged 1 commit into
mainfrom
codex/issue-64-rpc-e2e

Conversation

@sshaplygin

Copy link
Copy Markdown
Owner

Closes #64.

Summary

  • start the local YTsaurus cluster with an RPC proxy on port 8011
  • run rpc_e2e in the post-merge Cluster E2E workflow and emit RPC-proxy logs on failure
  • document the CI-backed live RPC check and the local setup

Validation

  • bash -n tests/cluster-e2e/run_local_cluster.sh
  • cargo fmt --check
  • cargo test --workspace
  • cargo test --doc
  • cargo clippy --workspace --all-targets -- -D warnings

The live Docker check was not run locally because a user-owned yt.frontend container is already using the local stack name.

@github-actions

Copy link
Copy Markdown

📊 Criterion benchmarks: main vs PR

🔍 Main ce41f7a8af9a → PR cf477d5c7e7c

📊 25 benchmark(s) compared · ⚠️ 0 regression(s) at ≥20% · ✅ 0 improvement(s) at ≥20%

ℹ️ Time is lower-is-better. Both revisions ran on the same GitHub-hosted VM; ±20% is an advisory marker, not a merge gate.

Benchmark main PR change signal
read/read_table_rows/1000 554.92 µs 552.95 µs -0.4% ℹ️ within runner noise
read/read_table_rows/10000 4.8641 ms 4.8630 ms +0.0% ℹ️ within runner noise
read/read_table_rows/100000 50.784 ms 50.335 ms -0.9% ℹ️ within runner noise
Skiff codec throughput/decode_dynamic 45.254 ms 43.482 ms -3.9% ℹ️ within runner noise
Skiff codec throughput/encode_dynamic 5.1059 ms 5.2617 ms +3.1% ℹ️ within runner noise
Skiff codec throughput/validate_and_skip 6.5407 ms 6.4875 ms -0.8% ℹ️ within runner noise
Skiff job throughput/skiff_dynamic 48.618 ms 47.560 ms -2.2% ℹ️ within runner noise
write/encode_then_write_table/1000 235.87 µs 240.09 µs +1.8% ℹ️ within runner noise
write/encode_then_write_table/10000 1.7260 ms 1.7615 ms +2.1% ℹ️ within runner noise
write/encode_then_write_table/100000 16.446 ms 17.220 ms +4.7% ℹ️ within runner noise
write/write_table_rows/1000 238.22 µs 239.78 µs +0.7% ℹ️ within runner noise
write/write_table_rows/10000 1.6268 ms 1.6325 ms +0.4% ℹ️ within runner noise
write/write_table_rows/100000 15.390 ms 15.213 ms -1.2% ℹ️ within runner noise
YSON job throughput/parse_borrowed 74.168 ms 72.104 ms -2.8% ℹ️ within runner noise
YSON job throughput/parse_dynamic 111.88 ms 110.10 ms -1.6% ℹ️ within runner noise
YSON job throughput/parse_owned 79.282 ms 79.296 ms +0.0% ℹ️ within runner noise
YSON job throughput/pass_through 25.062 ms 25.108 ms +0.2% ℹ️ within runner noise
YSON Throughput/Deserialize Binary 7.6270 ms 7.5736 ms -0.7% ℹ️ within runner noise
YSON Throughput/Deserialize Text 9.4873 ms 9.4897 ms +0.0% ℹ️ within runner noise
YSON Throughput/Serialize Binary 1.0264 ms 1.0371 ms +1.0% ℹ️ within runner noise
YSON Throughput/Serialize Text 3.6437 ms 3.6631 ms +0.5% ℹ️ within runner noise
YSON vs Skiff dynamic encoding/skiff_dynamic 23.262 ms 23.017 ms -1.1% ℹ️ within runner noise
YSON vs Skiff dynamic encoding/yson_dynamic 85.459 ms 87.751 ms +2.7% ℹ️ within runner noise
YSON vs Skiff dynamic job API/skiff_dynamic 48.647 ms 48.951 ms +0.6% ℹ️ within runner noise
YSON vs Skiff dynamic job API/yson_dynamic 116.65 ms 114.30 ms -2.0% ℹ️ within runner noise

@sshaplygin
sshaplygin merged commit 96ee419 into main Aug 16, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Put rpc_e2e into CI: the RPC client's only live check runs by hand

1 participant