Skip to content

run_locally.sh: enable simulations with tc netem; add corresponding CI jobs for crdb/ybdb#1431

Open
mickmis wants to merge 1 commit intointeruss:mainfrom
Orbitalize:loadtest-run_locally
Open

run_locally.sh: enable simulations with tc netem; add corresponding CI jobs for crdb/ybdb#1431
mickmis wants to merge 1 commit intointeruss:mainfrom
Orbitalize:loadtest-run_locally

Conversation

@mickmis
Copy link
Copy Markdown
Contributor

@mickmis mickmis commented Apr 21, 2026

  • in containers of crdb and ybdb: if an env variable for configuring tc netem is present, install tc through an entrypoint script; keep existing default behavior when not set
  • make start-locally now waits for the deployment to be ready before returning
  • updated some documentation for the ports mapping (ideally should have been done in run_locally.sh: refactor to run any number of nodes, defaults to two #1420)
  • adds in CI jobs running the configuration 'minimal_probing' with the simulated deployment (two USSs, two nodes per USS, so 4 in total)

Evidence for CRDB:
Screenshot From 2026-04-21 17-54-33
Screenshot From 2026-04-21 17-54-50

@mickmis mickmis force-pushed the loadtest-run_locally branch 2 times, most recently from ce9e5fa to 2ded9f6 Compare April 22, 2026 13:47
@mickmis mickmis changed the title run_locally.sh: enable loadtest simulation with CRDB run_locally.sh: enable simulations with tc netem; add corresponding CI jobs for crdb/ybdb Apr 22, 2026
@mickmis mickmis force-pushed the loadtest-run_locally branch 12 times, most recently from 9e3aae7 to 6cf8430 Compare April 27, 2026 13:43
Comment thread Makefile

.PHONY: start-locally
start-locally:
build/dev/run_locally.sh up -d
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • --wait implies -d
  • We need that for the ybdb deployment, as it is slower to be ready, and by the time the mocks are started and the uss_qualifier starts testing, the DSS is actually still not ready. Additionally it does IMO make sense for make start-locally to return once the stack is actually ready.

@mickmis mickmis force-pushed the loadtest-run_locally branch from 6cf8430 to 55b405c Compare April 28, 2026 09:48
@mickmis mickmis force-pushed the loadtest-run_locally branch from 55b405c to cb78ffa Compare April 28, 2026 09:58
@mickmis mickmis marked this pull request as ready for review April 28, 2026 11:52
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.

1 participant