-
Notifications
You must be signed in to change notification settings - Fork 896
Graduate SafeToEvict to Beta #2931
Description
I propose graduating SafeToEvict to Beta: It was Alpha in 1.29, with documentation in #2924.
The reasoning for the fast promotion is discussed in #2794:
Graduation to Beta should be relatively rapid as the feature has broad backward compatibility. The only situation where backward compatibility is violated is if the user is negatively impacted by a PDB on the game server pods, e.g. the user is relying on the default
safe-to-evict=falsepod annotation, but also is assuming that node drain (e.g. via cluster upgrade) will evict. Given that our existing documentation doesn't allow for node drains, we think this is a safe case for backwards incompatibility. To be conservative about it, though, when we graduateSafeToEvicttoBeta, we will flag it a potentially breaking change in Release Notes to call attention to it. To allow this situation, the user simply needs to seteviction: safe: OnUpgrade.
I recognize that this feature was relatively undocumented as of yet (hence #2924) and the contract might be not quite right, but I think the risk to graduation is minimal.
Towards #2777