Skip to content

Conversation

@o-nikolas
Copy link
Contributor

@o-nikolas o-nikolas commented Jan 16, 2026

The documentation and even most of the code properly refers to the config as max_invoke_attempts but the config name itself (which was read from Airflow config/env) was using the wrong config name (max_run_task_attempts which comes from the ECS executor)

Note: The Lambda Executor is still in alpha/experimental mode. So we are able to make this breaking change. However, this config is not often used, so it should be fairly low impact as well.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

The documentation and even most of the code properly refers to the
config as `max_invoke_attempts` but the config name itself (which was
read from Airflow config/env) was using the wrong config name
(`max_run_task_attempts` which comes from the ECS executor)
@boring-cyborg boring-cyborg bot added area:providers provider:amazon AWS/Amazon - related issues labels Jan 16, 2026
@o-nikolas o-nikolas merged commit 06fbabb into apache:main Jan 17, 2026
165 of 166 checks passed
@o-nikolas o-nikolas deleted the onikolas/lambda_max_invoke_config branch January 17, 2026 00:52
jason810496 pushed a commit to jason810496/airflow that referenced this pull request Jan 22, 2026
* Fix lambda executor max attempts config

The documentation and even most of the code properly refers to the
config as `max_invoke_attempts` but the config name itself (which was
read from Airflow config/env) was using the wrong config name
(`max_run_task_attempts` which comes from the ECS executor)

* Update changelog for the breaking change
shreyas-dev pushed a commit to shreyas-dev/airflow that referenced this pull request Jan 29, 2026
* Fix lambda executor max attempts config

The documentation and even most of the code properly refers to the
config as `max_invoke_attempts` but the config name itself (which was
read from Airflow config/env) was using the wrong config name
(`max_run_task_attempts` which comes from the ECS executor)

* Update changelog for the breaking change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:amazon AWS/Amazon - related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants