Add Wasm autoscaler policy support to FleetAutoscaler CRDs YAML#4298
Conversation
|
Build Succeeded 🥳 Build Id: c0e35baa-b2ce-4218-8c3d-a88273258a59 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: |
lacroixthomas
left a comment
There was a problem hiding this comment.
Other than the spaces, LGTM !
| - type: integer | ||
| - type: string | ||
| webhook: | ||
| webhook: |
There was a problem hiding this comment.
Not sure why it added these spaces, I don't see them on the template
There was a problem hiding this comment.
Honestly - no idea. Code gen add them 🤷🏻 I can't see any good reason why.
- Add 'Wasm' as a new policy type option for FleetAutoscaler - Create reusable 'url.configuration' template for webhook and wasm URL configs - Add wasm policy schema with function, config, from.url, and hash fields - Refactor webhook policy to use shared url.configuration template - Update generated install.yaml with corresponding CRD changes Work on agones-dev#4080
5a71b18 to
d8f0f5b
Compare
|
Build Succeeded 🥳 Build Id: dffcdfa9-fe21-4a47-8344-fa381a45fe29 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: |
…v#4298) - Add 'Wasm' as a new policy type option for FleetAutoscaler - Create reusable 'url.configuration' template for webhook and wasm URL configs - Add wasm policy schema with function, config, from.url, and hash fields - Refactor webhook policy to use shared url.configuration template - Update generated install.yaml with corresponding CRD changes Work on agones-dev#4080
What type of PR is this?
/kind feature
What this PR does / Why we need it:
Which issue(s) this PR fixes:
Work on #4080
Special notes for your reviewer:
N/A