Skip to content

Copy task SDK integration test dags to compose location before job start#56655

Merged
potiuk merged 2 commits into
apache:mainfrom
amoghrajesh:fix-task-sdk-tests
Oct 15, 2025
Merged

Copy task SDK integration test dags to compose location before job start#56655
potiuk merged 2 commits into
apache:mainfrom
amoghrajesh:fix-task-sdk-tests

Conversation

@amoghrajesh

Copy link
Copy Markdown
Contributor

Due to recent merge of #56139, some changes came in that lead to failures in dag processor finding dags at the right location.

This solution ensures that:

  • The test_dag.py file is available in the correct location for docker-compose to mount
  • The docker-compose volume mount ${PWD}/dags:/opt/airflow/dags will find the DAGs in the temporary directory

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@amoghrajesh amoghrajesh added the full tests needed We need to run full set of tests for this PR to merge label Oct 15, 2025
@amoghrajesh amoghrajesh reopened this Oct 15, 2025
@potiuk potiuk merged commit 7203012 into apache:main Oct 15, 2025
111 checks passed
snreddygopu pushed a commit to Teradata/airflow that referenced this pull request Oct 16, 2025
…art (apache#56655)

* Copy task SDK integration test dags to compose location before job start

* add AIRFLOW_UID
TyrellHaywood pushed a commit to TyrellHaywood/airflow that referenced this pull request Oct 22, 2025
…art (apache#56655)

* Copy task SDK integration test dags to compose location before job start

* add AIRFLOW_UID
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants