Change database-level helm_major_version to 3 in 13.7
In !46267 (merged), we added support for Helm 3 in the rails backend and made it default for all new clusters. The default was set at the model-level to avoid a race condition (see below), but the default should also be set at the database level. Assuming !46267 (merged) is rolled out during %13.6, the database-level default can be updated safely in %13.7.
-
@ahegyi started a discussion: (+2 comments) There might be a race condition here with zero downtime deployment, not sure if this will be problem on the app level:
- Migrations are executed.
- Old app picks up the new column and sets the default version to 3. The old version of the app doesn't know about Helm 3.
- Records created during this period should be Helm 2 but the DB sets version 3.
Suggestion:
In this MR change the default value for
helm_major_version
on the model level to 3. In the next release we can safely change the default value on the DB level.
Edited by Hordur Freyr Yngvason