starlingx/config
Code Issues Proposed changes

Only migrate helm v2 releases for supported apps

Browse Source 
Currently all helm v2 releases are migrated except for those that are
explicitly excluded. This commit will switch that logic to only migrate
data for those releases that have been explicitly tested and confirmed
to need data migration.

There are 3 categories of apps that need to have their helm v2 release
information managed:
 - SUPPORTED     : Migrate helm v2 data, execute application-upgrade
                   for the FluxCD version.
 - NOT SUPPORTED : Additional testing required and/or inclusion in
                   65-k8s-app-upgrade.sh and/or a dedicated upgrade
                   script is needed
 - SPECIAL HANDLE: A special upgrade script is provided to upgrade and
                   helm v2 data migration is not required by this
                   script

Test Plan:
PASS upgrade-activation with successful upgrades of applied
     applications nginx-ingress-controller and platform-integ-apps

Change-Id: If842db25fd0a00971a6b5587a0cfcb0c7cefa7a6
Story: 2009138
Task: 45611
Signed-off-by: Robert Church <robert.church@windriver.com>
This commit is contained in:
Robert Church
2022-06-12 17:39:54 -04:00
parent e8309c2d34
commit a45474a985
1 changed files with 45 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
controllerconfig/controllerconfig/upgrade-scripts/63-helm-v2-to-v3-rel_migration.sh
+45 -6
View File
@@ -87,15 +87,54 @@ function get_helmv2_config {
function migrate_apps {
log "$NAME: Migrating helm releases"
APPS=$(KUBECONFIG=/etc/kubernetes/admin.conf helmv2-cli -- helm list -a | tail -n+2 | awk '{print $1}')
for app in ${APPS}; do
case $app in
HELM_REL=$(KUBECONFIG=/etc/kubernetes/admin.conf helmv2-cli -- helm list -a | tail -n+2 | awk '{print $1}')
for rel in ${HELM_REL}; do
case $rel in
# NOT SUPPORTED: auditd-1.0-20.tgz
ns-auditd)
log "$NAME: migration of helm release $rel is not currently supported."
;;
# SPECIAL HANDLE: cert-manager-1.0-26.tgz -> 64-upgrade-cert-manager.sh
cm-cert-manager | cm-cert-manager-psp-rolebinding)
log "$NAME: unsuported migration for $app"
log "$NAME: helm release $rel is being migrated with a dedicated upgrade script."
;;
# NOT SUPPORTED: metrics-server-1.0-8.tgz
ms-metrics-server-psp-rolebinding | ms-metrics-server)
log "$NAME: migration of helm release $rel is not currently supported."
;;
# SUPPORTED: nginx-ingress-controller-1.1-18.tgz -> 65-k8s-app-upgrade.sh
ic-nginx-ingress )
log "$NAME: migrating helm release $rel."
/usr/bin/migrate_helm_release.py $rel
;;
# SPECIAL HANDLE: oidc-auth-apps-1.0-61.tgz -> 82-upgrade-oidc.py
oidc-dex | oidc-oidc-client | oidc-auth-secret-observer)
log "$NAME: helm release $rel is being migrated with a dedicated upgrade script."
;;
# SUPPORTED: platform-integ-apps-1.0-44.tgz -> 65-k8s-app-upgrade.sh
stx-ceph-pools-audit | stx-cephfs-provisioner | stx-rbd-provisioner)
log "$NAME: migrating helm release $rel."
/usr/bin/migrate_helm_release.py $rel
;;
# NOT SUPPORTED: portieris-1.0-33.tgz
portieris-portieris-psp-rolebinding | portieris-portieris-certs | portieris-portieris)
log "$NAME: migration of helm release $rel is not currently supported."
;;
# NOT SUPPORTED: ptp-notification-1.0-52.tgz
ptp-ptp-notification-psp-rolebinding | ptp-ptp-notification)
log "$NAME: migration of helm release $rel is not currently supported."
;;
# SUPPORTED: snmp-1.0-25.tgz -> 65-k8s-app-upgrade.sh
ns-snmp)
log "$NAME: migrating helm release $rel."
/usr/bin/migrate_helm_release.py $rel
;;
# NOT SUPPORTED: vault-1.0-23.tgz
sva-vault-psp-rolebinding | sva-vault)
log "$NAME: migration of helm release $rel is not currently supported."
;;
*)
log "$NAME: migrating $app helmrelease"
/usr/bin/migrate_helm_release.py $app
log "$NAME: migration of UNKNOWN helm release $rel is not currently supported."
;;
esac
done