Re: [PR] Helm chart should set AIRFLOW_HOME from airflowHome [airflow]

2023-11-07 Thread via GitHub
dstandish merged PR #34839: URL: https://github.com/apache/airflow/pull/34839 -- 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:

Re: [PR] Helm chart should set AIRFLOW_HOME from airflowHome [airflow]

2023-10-15 Thread via GitHub
amoghrajesh commented on code in PR #34839: URL: https://github.com/apache/airflow/pull/34839#discussion_r1359909746 ## helm_tests/airflow_core/test_env.py: ## @@ -0,0 +1,32 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

Re: [PR] Helm chart should set AIRFLOW_HOME from airflowHome [airflow]

2023-10-09 Thread via GitHub
dstandish commented on code in PR #34839: URL: https://github.com/apache/airflow/pull/34839#discussion_r1351131520 ## helm_tests/airflow_core/test_env.py: ## @@ -0,0 +1,32 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

Re: [PR] Helm chart should set AIRFLOW_HOME from airflowHome [airflow]

2023-10-09 Thread via GitHub
dstandish commented on code in PR #34839: URL: https://github.com/apache/airflow/pull/34839#discussion_r1350998852 ## chart/templates/_helpers.yaml: ## @@ -53,13 +53,8 @@ If release name contains chart name it will be used as a full name. {{/* Standard Airflow environment

Re: [PR] Helm chart should set AIRFLOW_HOME from airflowHome [airflow]

2023-10-09 Thread via GitHub
jedcunningham commented on code in PR #34839: URL: https://github.com/apache/airflow/pull/34839#discussion_r1350997608 ## helm_tests/airflow_core/test_env.py: ## @@ -0,0 +1,32 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

Re: [PR] Helm chart should set AIRFLOW_HOME from airflowHome [airflow]

2023-10-09 Thread via GitHub
jedcunningham commented on code in PR #34839: URL: https://github.com/apache/airflow/pull/34839#discussion_r1350996572 ## chart/templates/_helpers.yaml: ## @@ -53,13 +53,8 @@ If release name contains chart name it will be used as a full name. {{/* Standard Airflow environment

[PR] Helm chart should set AIRFLOW_HOME from airflowHome [airflow]

2023-10-09 Thread via GitHub
dstandish opened a new pull request, #34839: URL: https://github.com/apache/airflow/pull/34839 Currently if you change airflowHome from /opt/airflow to something else it doesn't totally work because the official image hardcodes the env var AIRFLOW_HOME. This should fix that. -- This is