-
Notifications
You must be signed in to change notification settings - Fork 340
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
Operator restart Integrations on upgrade #5491
Comments
* We should use RuntimeVersion and RuntimeProvider instead of Operator version to verify the upgrade process is correct * We must make sure that no further Pod is started to take over an application automatically during upgrade process Ref apache#5491
... in favour of RuntimeProvider/Version Additionally we strengthen the calculation of a digest to take in consideration all possible values which should trigger a drift. Closes apache#5491
... in favour of RuntimeProvider/Version Additionally we strengthen the calculation of a digest to take in consideration all possible values which should trigger a drift. Closes apache#5491
I had a quick look at the history of the project and talking to historical contributors and it seems that I was making confusion between the concept of a build (which trigger a new Build, IntegrationKit, etc...) and the deploy. The original design was done to make sure the operator don't rebuild until the user would trigger the build manually ( I will take the opportunity and clarify this aspect adding the proper documentation. In any case, the general review of the upgrade process is something that could be discussed and redesigned in the wider scope of the multi tenancy work effort that is ongoing as it seems to be a key feature that the new model should be able to provide in a seamless way. |
Verified when upgrading from Camel version 2.3.0 to 2.3.1. Any existing integration is restarted and even a new Pod is scheduled (see log below). It is very likely because we use the operator version to calculate digest (see #5192) when we really should account for runtime version/provider instead.
The rebuild should only be performed manually.
The text was updated successfully, but these errors were encountered: