Skip to content

Conversation

peterzhongyi
Copy link
Collaborator

@peterzhongyi peterzhongyi commented Apr 10, 2025

The load test started clients concurrently, but didn't make request concurrently. Without this change, clients make synchronous allocation calls, which is super slow.

Screenshot 2025-04-09 at 6 17 06 PM

With this change, we can achieve allocation speed of ~300 GameServer per second. (Note that with 15 clients making 100 requests, the allocation speed was ~400 gameservers per second)

Screenshot 2025-04-09 at 6 14 03 PM

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: b5fea322-6bd6-42dd-9edf-b7b55ce0ff03

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4149/head:pr_4149 && git checkout pr_4149
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.49.0-dev-4770868

Copy link
Collaborator

@igooch igooch left a comment

Choose a reason for hiding this comment

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

LGTM good catch.

@peterzhongyi peterzhongyi enabled auto-merge (squash) April 10, 2025 16:50
@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 74e2a610-9fa2-4be9-9cfc-e3d80c8191de

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4149/head:pr_4149 && git checkout pr_4149
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.49.0-dev-70805d5

@peterzhongyi peterzhongyi merged commit 4bd071d into googleforgames:main Apr 10, 2025
4 checks passed
@peterzhongyi peterzhongyi deleted the b2 branch April 10, 2025 18:09
@kamaljeeti kamaljeeti added area/performance Anything to do with Agones being slow, or making it go faster. and removed kind/other labels May 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/performance Anything to do with Agones being slow, or making it go faster.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants