Skip to content

Conversation

@o-nikolas
Copy link
Contributor

The two main components are:

  1. Start an api server within breeze for system-tests test group. That
    gives any tasks started with the sdk a server to communicate with.
  2. Update dag.test to support task sdk interaction with the executor.

Testing

Tested end to end with the ECS executor converted to support task sdk


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

The two main components are:

1) Start an api server within breeze for system-tests test group. That
   gives any tasks started with the sdk a server to communicate with.
2) Update dag.test to support task sdk interaction with the executor.
@o-nikolas o-nikolas marked this pull request as draft April 2, 2025 17:44
@o-nikolas o-nikolas marked this pull request as ready for review April 3, 2025 18:37
@o-nikolas o-nikolas merged commit 6687ddd into apache:main Apr 7, 2025
95 checks passed
@o-nikolas o-nikolas deleted the onikolas/dag_test_task_sdk branch April 7, 2025 15:59
simonprydden pushed a commit to simonprydden/airflow that referenced this pull request Apr 8, 2025
* Enable task sdk for system testing

The two main components are:

1) Start an api server within breeze for system-tests test group. That
   gives any tasks started with the sdk a server to communicate with.
2) Update dag.test to support task sdk interaction with the executor.

* Reduce scope of without-db-init usage

* Updated pytest plugin cli docs

* fix comment
@kaxil kaxil mentioned this pull request May 7, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools area:Executors-core LocalExecutor & SequentialExecutor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants