Miretpl commented on PR #65006: URL: https://github.com/apache/airflow/pull/65006#issuecomment-4229205103
> I found in the values.yaml that there was a Note that fernet-key,redis-password and broker-url secrets deliberately did not use the airflow-fullname helper as it might break existing installations (see https://github.com/apache/airflow/blob/main/chart/values.yaml#L38-L40) I removed that note, as it is outdated since https://github.com/apache/airflow/pull/52953 Good catch! > Dang, it's not everything. Since there is also the change in the service name of the statefulset as I said previously and also the ingress can't be replaced if it is used. Tried to use fullnameOverride=Release.Name and manually removing the ingress before helm upgrade but that then creates new log pvcs and so on. I went through all of the doc, and I haven't found any information regarding the old setup. So I would say that we are good with the doc, and newsfragment should be sufficient. -- 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]
