Design doc for Radius upgrades + Flux #98
Conversation
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
| name: radius | ||
| namespace: flux-system | ||
| annotations: | ||
| radapp.io/upgrade-enabled: "true" # Opt-in to safety checks |
There was a problem hiding this comment.
Is this the only safety mechanism that would block the reconciler from running the preflight checks and the upgrade?
If so, this should be disabled by default and then users should enable it if they want.
There was a problem hiding this comment.
I guess this is false by default.
| # Annotations show: | ||
| # radapp.io/upgrade-hold: "Cannot upgrade from 0.46.0 to 0.48.0: version skipping not supported" | ||
| # radapp.io/upgrade-checked-version: "0.46.0" |
There was a problem hiding this comment.
Is this the only way the users can see the errors in preflight checks?
| radapp.io/upgrade-checked-version: "0.46.0" | ||
|
|
||
| # Controller-managed: Hold reason when upgrades are blocked | ||
| radapp.io/upgrade-hold: "Cannot upgrade from 0.46.0 to 0.48.0: version skipping not supported" |
There was a problem hiding this comment.
What is the rollback scenario here in this case? Will they run helm rollback manually?
| └─────────────────┘ | ||
| ``` | ||
|
|
||
| ### Component Design |
There was a problem hiding this comment.
If a customer is using Flux and Helm to manage their Radius installation, what value does a Radius controller add?
There was a problem hiding this comment.
For the upgrade scenario, Radius controller will run the preflight checks to see if an upgrade is going to be successful.
There was a problem hiding this comment.
Installing Radius via Helm will not be able to bring all the moving parts together, as far as I know. Because rad install kubernetes and rad init will do some other operations other than just installing the Radius Helm release.
| ``` | ||
|
|
||
| **Expected Flow:** | ||
| 1. Radius controller detects incompatible upgrade |
There was a problem hiding this comment.
are there more conditions for incompatible upgrade in addition to version skip?
| ### Architecture Overview | ||
|
|
||
| ``` | ||
| ┌─────────────────┐ ┌───────────────────┐ |
There was a problem hiding this comment.
will Flux be a dependency for radius?
| - [ ] User documentation | ||
| - [ ] Example configurations | ||
|
|
||
| ## Open Questions |
There was a problem hiding this comment.
do we need more information on error handing/cleanup when each of these steps (Scenarios 1, 2)occur
No description provided.