Skip to content

fix: address issue #4589 — Promote RollingUpdateFix Feature Gate from Beta to Stable#4597

Open
santoshguru-11 wants to merge 10 commits into
agones-dev:mainfrom
santoshguru-11:fix/issue-4589-promote-rollingupdatefix-feature-gate-fr
Open

fix: address issue #4589 — Promote RollingUpdateFix Feature Gate from Beta to Stable#4597
santoshguru-11 wants to merge 10 commits into
agones-dev:mainfrom
santoshguru-11:fix/issue-4589-promote-rollingupdatefix-feature-gate-fr

Conversation

@santoshguru-11
Copy link
Copy Markdown

Fixes #4589: Promote RollingUpdateFix Feature Gate from Beta to Stable


Contributed via oss-contrib · @santoshguru-11

RollingUpdateFix has been enabled by default since 1.50.0 and is
stable enough to graduate.

Changes:
- pkg/util/runtime/features.go: move FeatureRollingUpdateFix to Stable section
- install/helm/agones/defaultfeaturegates.yaml: move to Stable section
- build/Makefile: remove from BETA_FEATURE_GATES
- cloudbuild.yaml: remove RollingUpdateFix=false from inverted e2e config
- site/content/en/docs/Guides/feature-stages.md: move row to Stable table
- test/upgrade/versionMap.yaml: add stableGates entry for Dev version

Closes agones-dev#4589
Copy link
Copy Markdown
Collaborator

@lacroixthomas lacroixthomas left a comment

Choose a reason for hiding this comment

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

There is also some code to update as this will be stable: https://github.com/agones-dev/agones/blob/main/pkg/fleets/controller.go#L598
This part of the code was only used when the feature gate was enabled runtime.FeatureEnabled(runtime.FeatureRollingUpdateFix) but as it will be stable, we can just use it directly and cleanup the unused code from it

"alphaGates": ["PlayerAllocationFilter", "PlayerTracking", "WasmAutoscaler"],
"betaGates": ["CountsAndLists", "GKEAutopilotExtendedDurationPods", "PortPolicyNone", "PortRanges", "RollingUpdateFix", "ScheduledAutoscaler", "FleetAutoscaleRequestMetaData", "SidecarContainers"]
"betaGates": ["CountsAndLists", "GKEAutopilotExtendedDurationPods", "PortPolicyNone", "PortRanges", "ScheduledAutoscaler", "FleetAutoscaleRequestMetaData", "SidecarContainers"],
"stableGates": ["RollingUpdateFix"]
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There is no "stableGates", you can remove this line

Comment on lines 39 to +56

| Feature Name | Gate | Default | Stage | Since |
|------------------------------------------------------------------------------------------------------------------------|------------------------------------|---------|--------|--------|
| [CountsAndLists](https://github.com/agones-dev/agones/issues/2716) | `CountsAndLists` | Enabled | `Beta` | 1.41.0 |
| [Support for Extended Duration Pods on GKE Autopilot (*1.28+ only*)](https://github.com/agones-dev/agones/issues/3386) | `GKEAutopilotExtendedDurationPods` | Enabled | `Beta` | 1.44.0 |
| [Port Policy None](https://github.com/agones-dev/agones/issues/3804) | `PortPolicyNone` | Enabled | `Beta` | 1.49.0 |
| [Multiple dynamic port ranges](https://github.com/agones-dev/agones/issues/1911) | `PortRanges` | Enabled | `Beta` | 1.49.0 |
| [Rolling Update Fixes](https://github.com/agones-dev/agones/issues/3688) | `RollingUpdateFix` | Enabled | `Beta` | 1.50.0 |
| [Scheduled Fleet Autoscaling](https://github.com/agones-dev/agones/issues/3008) | `ScheduledAutoscaler` | Enabled | `Beta` | 1.51.0 |
| [Extend Webhook autoscaler to send fleet metadata with the request](https://github.com/agones-dev/agones/issues/3951) | `FleetAutoscaleRequestMetaData` | Enabled | `Beta` | 1.54.0 |
| [Sidecar Containers](https://github.com/agones-dev/agones/issues/3642) | `SidecarContainers` | Enabled | `Beta` | 1.56.0 |

### Stable Features

| Feature Name | Gate | Default | Stage | Since |
|------------------------------------------------------------------------------------------|----------------------------|---------|----------|--------|
| [DisableResyncOnSDKServer](https://github.com/agones-dev/agones/issues/3377) | `DisableResyncOnSDKServer` | Enabled | `Stable` | 1.49.0 |
| [Support Passthrough on GKE Autopilot](https://github.com/agones-dev/agones/issues/3721) | `AutopilotPassthroughPort` | Enabled | `Stable` | 1.54.0 |
| [Rolling Update Fixes](https://github.com/agones-dev/agones/issues/3688) | `RollingUpdateFix` | Enabled | `Stable` | 1.50.0 |
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

You will need to add shortcodes around it, as it will only be the case from a specific version
https://agones.dev/site/docs/contribute/documentation-editing-contribution/#within-a-page
We usually copy the whole table and wrap them with shortcodes (< 1.58.0 and > 1.58.0 for example)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Promote RollingUpdateFix Feature Gate from Beta to Stable

2 participants