-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Smoke test the FTL upgrade path #2966
Comments
You can probably do something like this using a similar approach to the existing kube test: https://github.com/TBD54566975/ftl/blob/a41614931789322eab98e36ac65b0a46ee7dc635/backend/controller/scaling/kube_scaling_integration_test.go#L21 There is nothing in there about upgrades but I don't think it would be that much work. It's not quite the same as running on a 'real' cluster but is a lot lighter weight in terms of providing a smoke test. |
Smoke testing the upgrade path from the last FTL release is a bit of a chicken and egg, especially for the first upgrade test, but also because the smoke tests themselves can be updated. We also can't just grab the latest smoke test, since it may refer to newly added FTL features / test cases in the exemplar. My current thinking is to:
For (2), we skip since we can't test upgrade and the integration test will cover the non-upgrade path. |
The above tests that a new release is still compatible with old modules, but it doesn't test the upgrade path the way FTL client projects would upgrade. We'll need to use the long-living FTL cluster for that. |
On merge into main (or part of the manual release process?), we should have a GitHub action that will:
If this is run on merges to main, we'll have to figure out a way to be notified of failures. If it's run as part of the manual release process, failures should block the release.
The text was updated successfully, but these errors were encountered: