commit 5025e1ce710f626012e5e51f492e637483c4a245
Author: Lucas Cavalcante <email address hidden>
Date: Wed Jun 29 10:48:31 2022 -0300
Skip Recover From Armada to FluxCD apps
If an system application-update is triggered updating an armada app
to a fluxcd app (preceded by a helm release migration) and
update fails, the application framework will try to perform a recover.
Recover will fail as fluxcd uses helm3 and armada helm2. This will
create resources both in helm2 and helm3 leaving the app
in a inconsistent state.
To prevent that from happening recover is skipped if to_app and from_app
use different chart managers.
TEST PLAN:
PASS: recover skipped after update from armada to fluxcd without
migrating helmrelease
Closes-bug: 1980242
Signed-off-by: Lucas Cavalcante <email address hidden>
Change-Id: I9061b75f443730e973b79cc93e955069951113ff
Reviewed: https:/ /review. opendev. org/c/starlingx /config/ +/848133 /opendev. org/starlingx/ config/ commit/ 5025e1ce710f626 012e5e51f492e63 7483c4a245
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit 5025e1ce710f626 012e5e51f492e63 7483c4a245
Author: Lucas Cavalcante <email address hidden>
Date: Wed Jun 29 10:48:31 2022 -0300
Skip Recover From Armada to FluxCD apps
If an system application-update is triggered updating an armada app
to a fluxcd app (preceded by a helm release migration) and
update fails, the application framework will try to perform a recover.
Recover will fail as fluxcd uses helm3 and armada helm2. This will
create resources both in helm2 and helm3 leaving the app
in a inconsistent state.
To prevent that from happening recover is skipped if to_app and from_app
use different chart managers.
TEST PLAN:
PASS: recover skipped after update from armada to fluxcd without
migrating helmrelease
Closes-bug: 1980242 e973b79cc93e955 069951113ff
Signed-off-by: Lucas Cavalcante <email address hidden>
Change-Id: I9061b75f443730