[GitHub] [airflow] potiuk commented on pull request #20069: Add migration-wait-timeout to the helm chart

2021-12-06 Thread GitBox
potiuk commented on pull request #20069: URL: https://github.com/apache/airflow/pull/20069#issuecomment-986756000 Ah I see. But I hopw #18439 could be also merged, that would help A LOT in a meaningful message to the user if the timeout actually happens. Or is timeout equivalent with "erro

[GitHub] [airflow] potiuk commented on pull request #20069: Add migration-wait-timeout to the helm chart

2021-12-06 Thread GitBox
potiuk commented on pull request #20069: URL: https://github.com/apache/airflow/pull/20069#issuecomment-986744395 Question - what woudl be the cases where timing out on migration is "wrong" ? Woudl not that leave the DB in a state that is not expected ? However I think that one is fine to