-
Notifications
You must be signed in to change notification settings - Fork 813
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates allocation load testing documentation #2883
Conversation
Build Succeeded 👏 Build Id: a8f05eb6-0d6b-467c-9f4e-68bc4aaef4da 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:
|
d27e5e9
to
069c11e
Compare
Build Succeeded 👏 Build Id: df00af23-8ada-4ef7-83e9-0f5cfddde213 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:
|
Build Failed 😱 Build Id: f5a41dab-7aab-4c86-b6df-f40273858d2a To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
91c9723
to
8dc9bdd
Compare
Build Succeeded 👏 Build Id: c873f8b5-9aad-4d93-942e-20f7f63ac037 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:
|
Build Failed 😱 Build Id: b643532a-2bc6-4761-9dd7-c6acb746ad8c To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Looks like a flaky test:
|
Build Succeeded 👏 Build Id: c871c0ab-0079-48d9-a2e9-5428ffd9f1bf 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:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of the grammar comments are on text that was pre-existing, but since you moved around it seems like a good time to fix it.
Build Succeeded 👏 Build Id: 29d51a96-53ca-4dc0-abd5-f50ba6f08f73 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:
|
test/load/allocation/README.md
Outdated
minReplicas: 10000 | ||
maxReplicas: 20000 | ||
``` | ||
We used the sample [fleet configuration](./scenario-fleet.yaml and [fleet autoscaler configuration](./autoscaler.yaml). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing a closing ) for the first yaml file.
Build Succeeded 👏 Build Id: 674c3aee-da7c-4cbc-966b-a721ed8de5ed 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:
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: igooch, roberthbailey The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Build Failed 😱 Build Id: 23235b83-3550-4a4f-a575-76540b4a4287 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Test flake in e2e-feature-gates:
|
Build Succeeded 👏 Build Id: f51e0f7c-6fcf-49be-bcf6-a8455942ee90 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:
|
What type of PR is this?
/kind documentation
What this PR does / Why we need it:
Clean up of load test documentation makes instructions on how to run the tests clearer.
Which issue(s) this PR fixes:
NA
Special notes for your reviewer: