AIP-67 - Multi Team: Update AwsLambdaExecutor to support multi team#61321
AIP-67 - Multi Team: Update AwsLambdaExecutor to support multi team#61321o-nikolas merged 1 commit intoapache:mainfrom
Conversation
providers/amazon/tests/unit/amazon/aws/executors/aws_lambda/test_lambda_executor.py
Show resolved
Hide resolved
aa136f7 to
5ce81c4
Compare
Enable AwsLambdaExecutor to read team-based config correctly
5ce81c4 to
2571e25
Compare
Yeah, I validated the multi-team configuration precedence in an Airflow 2.11 environment using a small smoke-style check (I set team and global env vars inside a 2.11 container and inspected the resolved values). This verifies that team-specific keys override global keys while unspecified keys correctly fall back to global. |
Thanks for doing that @henry3260! Did you actually run any dags with the executor to ensure task execution is working end to end? |
Not yet. Since I'm still getting familiar with the AWS setup, I'll need a bit more time to run the end-to-end tests. I'll update you as soon as I have the results. |
Thanks, much appreciated @henry3260 |
Hi @o-nikolas , I think it works fine in 2.11! |
|
Lovely! Thanks for doing the back compat testing 😄 |
Enable AwsLambdaExecutor to read team-based config correctly



Changes
Add
supports_multi_team: bool = True flag toAwsLambdaExecutorUpdate all
conf.get()calls to use self.conf.get() for team-specific configurationAdd
test_team_configtest to verify team isolation worksWas generative AI tooling used to co-author this PR?
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.