-
Notifications
You must be signed in to change notification settings - Fork 858
fleetautoscaler.md references metadata incorrectly #4217
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
fleetautoscaler.md references metadata incorrectly #4217
Conversation
/gcbrun |
Build Failed 😭 Build Id: 23294625-e612-430d-bb7c-1f6d4a4e656a Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Flake:
Weird. |
/gcbrun |
Build Succeeded 🥳 Build Id: 21a4dcf5-d417-45a3-a134-35657b2729e4 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.
Good catch! Thanks!
/gcbrun |
Build Succeeded 🥳 Build Id: ba0558ab-a4bf-4860-a697-6a70d627c215 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:
I started to write my own Webhook Autoscaler using the documentation as a reference and never saw a
metadata
key in my scaling requests from Agones. I started researching the codebase and read through #3957 and the associated issues. I think this error in documentation slipped through during the code review and it was not updated to what actually shipped in that PR.This change updates the documentation to match the struct again.
Which issue(s) this PR fixes:
Closes #
Special notes for your reviewer: