Skip to content

Conversation

@o-nikolas
Copy link
Contributor

@o-nikolas o-nikolas commented Sep 4, 2025

When running in dag.test we create a special in-process supervisor and comms for fetching conns, variables, etc. But we do not do the same when we run the triggers for deferred operators. Thus, they will fail to fetch connections.

relates to #54692


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

Copy link
Member

@kaxil kaxil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks about right

When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
@o-nikolas o-nikolas force-pushed the onikolas/dag_test_trigger_supervisor_in_process branch from 1d57a80 to 1d36718 Compare September 4, 2025 03:03
@o-nikolas
Copy link
Contributor Author

Okay, cleaned up the code a bit and then did some test runs and the triggers seem to be working correctly now!

@o-nikolas o-nikolas marked this pull request as ready for review September 4, 2025 03:04
Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@potiuk
Copy link
Member

potiuk commented Sep 4, 2025

Just one test to fix :)

@o-nikolas o-nikolas merged commit 1d73c01 into apache:main Sep 4, 2025
79 checks passed
@o-nikolas o-nikolas deleted the onikolas/dag_test_trigger_supervisor_in_process branch September 4, 2025 16:33
RoyLee1224 pushed a commit to RoyLee1224/airflow that referenced this pull request Sep 8, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Sep 30, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 1, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 2, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 3, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 4, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 5, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 5, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 7, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 8, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 9, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 10, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 11, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 12, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 14, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 15, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 17, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 19, 2025
When running in dag.test we create a special in-process supervisor and
comms for fetching conns, variables, etc. But we do not do the same
when we run the triggers for deferred operators. Thus, they will fail to
fetch connections.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants