feat: promote FleetAutoscaleRequestMetaData to beta#4365
Conversation
markmandel
left a comment
There was a problem hiding this comment.
Thanks for the submission 🥳 , but I also noticed that we don't have any documentation on this feature in https://agones.dev/site/docs/reference/fleetautoscaler/#webhook-endpoint-specification
Any chance you want to also add them as part of this PR?
| | [Player Tracking]({{< ref "/docs/Guides/player-tracking.md" >}}) | `PlayerTracking` | Disabled | `Alpha` | 1.6.0 | | ||
| | [Scheduled Fleet Autoscaling](https://github.com/googleforgames/agones/issues/3008) | `ScheduledAutoscaler` | Enabled | `Beta` | 1.51.0 | | ||
| | [Extend Webhook autoscaler to send fleet metadata with the request](https://github.com/googleforgames/agones/issues/3951) | `FleetAutoscaleRequestMetaData` | Disabled | `Alpha` | 1.48.0 | | ||
| | [Extend Webhook autoscaler to send fleet metadata with the request](https://github.com/googleforgames/agones/issues/3951) | `FleetAutoscaleRequestMetaData` | Enabled | `Beta` | 1.48.0 | |
There was a problem hiding this comment.
Same things as #4364 - this shouldn't be changed, since this is the current docs that are displayed, and the feature will only move in the next version.
|
/gcbrun (not that I think there will be an issue) |
|
Build Failed 😭 Build Id: 16a61980-7824-430a-bf59-834c4fe8eef8 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
I added a brief description of FleetAutoscaleRequestMetaData. |
| The `FleetAutoscaleResponse`'s `Replica` field is used to set the target `Fleet` count with each sync interval, thereby | ||
| providing the autoscaling functionality. | ||
|
|
||
| {{< beta title="Fleet Metadata in Webhook Requests" gate="FleetAutoscaleRequestMetaData" >}} |
There was a problem hiding this comment.
Forgot to mention, will need to be feature short coded like so:
https://agones.dev/site/docs/contribute/documentation-editing-contribution/#within-a-page
1.54.0 just released, so the shortcode and version numbers in this PR will also need to be updated.
Thanks!
There was a problem hiding this comment.
I’ve added the feature shortcode.
e9f4ba8
markmandel
left a comment
There was a problem hiding this comment.
Not sure if the ticket captured everything in:
So you also need to do:
https://github.com/googleforgames/agones/blob/e2d352233317c7f7a3acf091d6b25aed1757da6c/cloudbuild.yaml#L309-L311
But otherwise, this is looking good! Thanks!
| {{% feature publishVersion="1.54.0" %}} | ||
| {{< beta title="Fleet Metadata in Webhook Requests" gate="FleetAutoscaleRequestMetaData" >}} | ||
|
|
||
| When the `FleetAutoscaleRequestMetaData` feature gate is enabled by default, the Fleet's `labels` and | ||
| `annotations` are included in the `FleetAutoscaleRequest`, allowing webhooks to make scaling decisions based on | ||
| Fleet-specific metadata. | ||
| {{% /feature %}} |
There was a problem hiding this comment.
This would bring things up to date for what is current, and also move it to beta with the next release 👍🏻
| {{% feature publishVersion="1.54.0" %}} | |
| {{< beta title="Fleet Metadata in Webhook Requests" gate="FleetAutoscaleRequestMetaData" >}} | |
| When the `FleetAutoscaleRequestMetaData` feature gate is enabled by default, the Fleet's `labels` and | |
| `annotations` are included in the `FleetAutoscaleRequest`, allowing webhooks to make scaling decisions based on | |
| Fleet-specific metadata. | |
| {{% /feature %}} | |
| {{% feature expireyVersion="1.54.0" %}} | |
| {{< alpha title="Fleet Metadata in Webhook Requests" gate="FleetAutoscaleRequestMetaData" >}} | |
| {{% /feature %}} | |
| {{% feature publishVersion="1.54.0" %}} | |
| {{< beta title="Fleet Metadata in Webhook Requests" gate="FleetAutoscaleRequestMetaData" >}} | |
| {{% /feature %}} | |
| When the `FleetAutoscaleRequestMetaData` feature gate is enabled, the Fleet's `labels` and | |
| `annotations` are included in the `FleetAutoscaleRequest`, allowing webhooks to make scaling decisions based on | |
| Fleet-specific metadata. |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: 91dce7b4-711a-487f-9e5b-9c18ef0c586a 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: |
|
/gcbrun |
|
Build Failed 😭 Build Id: d20d8330-3bbf-4e80-9885-8b31ed1be223 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Co-authored-by: Mark Mandel <mark@compoundtheory.com>
Head branch was pushed to by a user without write access
|
/gcbrun |
|
Build Failed 😭 Build Id: 41733f92-0bc7-4f6c-94fc-962647011879 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
/gcbrun Haven't had a flake in a while. Will look properly later. |
|
Build Succeeded 🥳 Build Id: e12c1ff6-3a02-495a-9966-b2e67f1768b5 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: |
* Promote FleetAutoscaleRequestMetaData to Beta * reflect comments * add documentation * update feature short coded * fix cloudbuild * fix docs Co-authored-by: Mark Mandel <mark@compoundtheory.com> * typo expiryVersion Co-authored-by: Mark Mandel <mark@compoundtheory.com> --------- Co-authored-by: Mark Mandel <mark@compoundtheory.com>
What type of PR is this?
/kind feature
What this PR does / Why we need it:
promote FleetAutoscaleRequestMetaData to beta
Which issue(s) this PR fixes:
Closes #4360
Special notes for your reviewer: