Skip to content

Commit

Permalink
Fix for start-up script not working in local docker image (#226)
Browse files Browse the repository at this point in the history
*Issue #, if available:*
The startup script didn't work for local docker image testing. So there
was no way to test startup scripts.

*Description of changes:*
- Added a startup script path environment variable in docker-compose
file.
- Tested by running local docker image with a startup script.
By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.

Co-authored-by: Vraj Patel <[email protected]>
  • Loading branch information
Vraj2725 and Vraj Patel authored Mar 5, 2025
1 parent 728029d commit 4995a88
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 0 deletions.
2 changes: 2 additions & 0 deletions images/airflow/2.10.1/docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@ x-airflow-common: &airflow-common
# container is up.
MWAA__CORE__REQUIREMENTS_PATH: ${MWAA__CORE__REQUIREMENTS_PATH}
MWAA__CORE__AUTH_TYPE: "testing"
# Use this environment variable if you have a custom start-up script.
MWAA__CORE__STARTUP_SCRIPT_PATH: ${MWAA__CORE__STARTUP_SCRIPT_PATH}
# Additional Airflow configuration can be passed here in JSON form.
MWAA__CORE__CUSTOM_AIRFLOW_CONFIGS: "{}"
MWAA__CORE__FERNET_KEY: ${FERNET_KEY}
Expand Down
2 changes: 2 additions & 0 deletions images/airflow/2.10.1/startup/startup.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
#!/bin/bash
#Add your startup script for testing here
2 changes: 2 additions & 0 deletions images/airflow/2.10.3/docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@ x-airflow-common: &airflow-common
# container is up.
MWAA__CORE__REQUIREMENTS_PATH: ${MWAA__CORE__REQUIREMENTS_PATH}
MWAA__CORE__AUTH_TYPE: "testing"
# Use this environment variable if you have a custom start-up script.
MWAA__CORE__STARTUP_SCRIPT_PATH: ${MWAA__CORE__STARTUP_SCRIPT_PATH}
# Additional Airflow configuration can be passed here in JSON form.
MWAA__CORE__CUSTOM_AIRFLOW_CONFIGS: "{}"
MWAA__CORE__FERNET_KEY: ${FERNET_KEY}
Expand Down
2 changes: 2 additions & 0 deletions images/airflow/2.10.3/startup/startup.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
#!/bin/bash
#Add your startup script for testing here
2 changes: 2 additions & 0 deletions images/airflow/2.9.2/docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@ x-airflow-common: &airflow-common
# container is up.
MWAA__CORE__REQUIREMENTS_PATH: ${MWAA__CORE__REQUIREMENTS_PATH}
MWAA__CORE__AUTH_TYPE: "testing"
# Use this environment variable if you have a custom start-up script.
MWAA__CORE__STARTUP_SCRIPT_PATH: ${MWAA__CORE__STARTUP_SCRIPT_PATH}
# Additional Airflow configuration can be passed here in JSON form.
MWAA__CORE__CREATED_AT: "Tue Sep 18 23:05:58 UTC 2024"
MWAA__CORE__CUSTOM_AIRFLOW_CONFIGS: "{}"
Expand Down
2 changes: 2 additions & 0 deletions images/airflow/2.9.2/startup/startup.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
#!/bin/bash
#Add your startup script for testing here

0 comments on commit 4995a88

Please sign in to comment.