Repository navigation
demos/parking: size each worker for one actor and fail load.sh on req… - #3391
Open
Bo-Cheng Chu (bochengchu) wants to merge 1 commit into
Open
Bo-Cheng Chu (bochengchu) wants to merge 1 commit into
Bo-Cheng Chu (bochengchu) wants to merge 1 commit into
Conversation
Bo-Cheng Chu (bochengchu)
force-pushed
the
parking-demo-fix
branch
from
October 28, 2026 17:55
61a0141 to
cfa0421
Compare
Bo-Cheng Chu (bochengchu)
marked this pull request as ready for review
October 28, 2026 20:15
Bo-Cheng Chu (bochengchu)
requested review from
Benjamin Elder (BenTheElder) and
Lior Lieberman (LiorLieberman)
October 28, 2026 20:15
Bo-Cheng Chu (bochengchu)
force-pushed
the
parking-demo-fix
branch
from
October 28, 2026 23:15
cfa0421 to
696f9cd
Compare
WorkerPool/parking omitted container resource limits, so the Kubernetes Downward API reported the node's allocatable capacity in /run/worker-capacity. Two workers held all four 1000m actors at once without triggering request parking. Set per-worker container limits to 1 CPU and 1Gi memory so each worker hosts at most one actor. Add a wakeupProbe on /readyz port 80 to parking-template.yaml.tmpl so golden memory snapshots are captured after the counter server binds port 80. In load.sh, wait for each proxy to start listening before generating load, aggregate only actor request logs (excluding .proxy.log files), and exit non-zero when requests fail with a non-200/503 status or when zero requests return 200 OK. Signed-off-by: Bo-Cheng Chu <bochengchu@google.com>
Bo-Cheng Chu (bochengchu)
force-pushed
the
parking-demo-fix
branch
from
October 28, 2026 23:20
696f9cd to
c91c92c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #3287
WorkerPool/parkingindemos/parking/parking.yaml.tmpldid not set container resource limits, so the Kubernetes Downward API exposed the node's allocatable capacity in/run/worker-capacity. As a result, 2 workers had room for all 41000mactors at once and request parking was never triggered. Set per-worker container limits to1CPU and1Gimemory so each worker hosts at most one actor.Also add a
wakeupProbeon/readyzport80todemos/parking/parking-template.yaml.tmpl(matchingdemos/counter/counter-template.yaml.tmpl) so golden memory snapshots are captured after the counter HTTP server binds port 80.In
demos/parking/load.sh:kubectl ate proxy actoroutput to.proxyfiles instead of.proxy.logsocat "${TMP}"/*.logdoes not count proxy startup lines as failed requests.1when any request fails with a non-200/503 status (other > 0) or when no requests succeed (ok == 0).