Skip to content

AIP-67 - Multi Team: Pass args/kwargs to super in Celery executors - #56006

Merged
eladkal merged 2 commits into
apache:mainfrom
aws-mwaa:onikolas/celery_exec_args_kwargs
Sep 23, 2025
Merged

AIP-67 - Multi Team: Pass args/kwargs to super in Celery executors#56006
eladkal merged 2 commits into
apache:mainfrom
aws-mwaa:onikolas/celery_exec_args_kwargs

Conversation

@o-nikolas

Copy link
Copy Markdown
Contributor

This allows the team_name to be passed to the super class. This is low hanging fruit to allow the Celery executor to be used for multi team testing. More changes will be needed to allow the Celery executor to use team-based config, but that will be done at a future time.


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

This allows the team_name to be passed to the super class. This is low
hanging fruit to allow the Celery executor to be used for multi team testing.
More changes will be needed to allow the Celery executor to use team-based
config, but that will be done at a future time.
@o-nikolas

Copy link
Copy Markdown
Contributor Author

@vincbeck @ramitkataria

@o-nikolas o-nikolas added the multi-team - aip-67 Issues related to multi-team (AIP-67) label Sep 23, 2025
@eladkal

eladkal commented Sep 23, 2025

Copy link
Copy Markdown
Contributor

tests fail

@ramitkataria ramitkataria left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A suggestion for the failing unit test but looks good otherwise!

@o-nikolas

Copy link
Copy Markdown
Contributor Author

tests fail

Passing now @eladkal

@eladkal
eladkal merged commit 17290c6 into apache:main Sep 23, 2025
74 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers multi-team - aip-67 Issues related to multi-team (AIP-67) provider:celery

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants