Desdroid commented on PR #65006: URL: https://github.com/apache/airflow/pull/65006#issuecomment-4224645772
On a side note #52953 (PR for https://github.com/apache/airflow/commit/eabe6b8dd77204f7c0d117c9d9ad1f4166869671 ) also made the upgrade from 1.18 not straight forward. This PR was only in the Bug Fixes section and no migration hint provided but effectively leads to renaming a bunch of k8s resources which helm does not automatically migrate or doesn't support changing. E.g. it seems like the serviceName of the Triggerer StatefulSet was also changed, which I think is not supported by helm. We then get `Error: UPGRADE FAILED: cannot patch "pipelines-airflow-triggerer" with kind StatefulSet: StatefulSet.apps "pipelines-airflow-triggerer" is invalid: spec: Forbidden: updates to statefulset spec for fields other than 'replicas', 'ordinals', 'template', 'updateStrategy', 'persistentVolumeClaimRetentionPolicy' and 'minReadySeconds' are forbidden` trying to do a simple helm upgrade. So we ended up with uninstalling the entire helm chart release and doing a fresh install, while retaining the persistent volumes of db etc. While I think this is a bit of an edge case (Having the helm chart of airflow as a dependency) - should I add some Info in the docs ? Or the GH Releases notes be adapted? What do you think? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
