Skip to content

Fix task sdk integration tests, copy dags to working dir#56643

Closed
gopidesupavan wants to merge 2 commits into
apache:mainfrom
gopidesupavan:fix-task-sdk-tests
Closed

Fix task sdk integration tests, copy dags to working dir#56643
gopidesupavan wants to merge 2 commits into
apache:mainfrom
gopidesupavan:fix-task-sdk-tests

Conversation

@gopidesupavan

Copy link
Copy Markdown
Member

https://github.com/apache/airflow/actions/runs/18515458033/job/52767293548


^ 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.

@gopidesupavan gopidesupavan added the full tests needed We need to run full set of tests for this PR to merge label Oct 15, 2025
@gopidesupavan gopidesupavan reopened this Oct 15, 2025
Comment thread task-sdk-tests/tests/task_sdk_tests/conftest.py
@amoghrajesh

Copy link
Copy Markdown
Contributor

@gopidesupavan doesnt look like the fix works?

@gopidesupavan

Copy link
Copy Markdown
Member Author

@gopidesupavan doesnt look like the fix works?

hm looks like not fixed, seems some permission issues on the dir, unfortunately am not at my laptop. please feel free to take a look, or i will check later in the day :)

@potiuk

potiuk commented Oct 15, 2025

Copy link
Copy Markdown
Member

I added a fixup that might solve it @gopidesupavan @amoghrajesh

@potiuk

potiuk commented Oct 15, 2025

Copy link
Copy Markdown
Member

AIRFLOW_UID was not set - which causes a) warning that it's not set b) in linux it will change the ownership of shared volumes to be host user id

@gopidesupavan

Copy link
Copy Markdown
Member Author

closing in favour of #56655

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.

3 participants