You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "V0lantis (via GitHub)" <gi...@apache.org> on 2023/02/20 10:04:11 UTC

[GitHub] [airflow] V0lantis opened a new issue, #29632: Postgres secret Bug when upgrading helm chart version (1.7.0 -> 1.8.0)

V0lantis opened a new issue, #29632:
URL: https://github.com/apache/airflow/issues/29632

   ### Official Helm Chart version
   
   1.8.0 (latest released)
   
   ### Apache Airflow version
   
   2.4.3
   
   ### Kubernetes Version
   
   v1.21.14-eks
   
   ### Helm Chart configuration
   
   _No response_
   
   ### Docker Image customizations
   
   _No response_
   
   ### What happened
   
   When trying to applying our deployment configuration with the new helm-chart release, we faced the following issue:
   ```
   COMBINED OUTPUT:
     Error: UPGRADE FAILED: execution error at (airflow/charts/postgresql/templates/secrets.yaml:17:24): 
     PASSWORDS ERROR: The secret "airflow-postgresql" does not contain the key "postgres-password"
   ```
   
   Although, when I go look for the secret, I find it but with the key `postgresQL-password` (Trailing **ql** at the end)
   
   ### What you think should happen instead
   
   It should work fine. Although I think the issue come from the postgresql bitnani version
   
   ### How to reproduce
   
   Create a Airflow deployment with helm-chart 1.7.0 and with a `postgres.enabled: true`. Then upgrade your helm chart to 1.8.0 and try to deploy again. 
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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: commits-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] potiuk closed issue #29632: Postgres secret Bug when upgrading helm chart version (1.7.0 -> 1.8.0)

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk closed issue #29632: Postgres secret Bug when upgrading helm chart version (1.7.0 -> 1.8.0)
URL: https://github.com/apache/airflow/issues/29632


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] potiuk commented on issue #29632: Postgres secret Bug when upgrading helm chart version (1.7.0 -> 1.8.0)

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on issue #29632:
URL: https://github.com/apache/airflow/issues/29632#issuecomment-1436805177

   Yeah. This is expected - see Release Notes: https://airflow.apache.org/docs/helm-chart/stable/release_notes.html#airflow-helm-chart-1-8-0-2023-02-06  where we explain the migration steps. The bult in postgres is really a "development" feature and if you go production, you should switch to external one, so we do not consider that as a breaking change - that's why we left 1.* version but we also provided migration steps in release notes in case somoene would like to upgrade their development setup.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] V0lantis commented on issue #29632: Postgres secret Bug when upgrading helm chart version (1.7.0 -> 1.8.0)

Posted by "V0lantis (via GitHub)" <gi...@apache.org>.
V0lantis commented on issue #29632:
URL: https://github.com/apache/airflow/issues/29632#issuecomment-1436712605

   Find the previous version I probably used for my deployment [here](https://github.com/bitnami/charts/blob/e2404c63cc5e72d5dee91e50b4597fd9e0b1ab0c/bitnami/postgresql/templates/secrets.yaml#L20) and the PR is bitnami/charts#8827. I guess a similar issue is bitnami/charts#4416.
   A potential workaround is to update the secret in place and add a second line with the same password. (Tested and worked in my environment). Feel free to close my issue if you think it is fixed.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org