Skip to content

Rename StepFunctionStartExecutionOperator's input field to match its constructor argument - #70544

Merged
vincbeck merged 2 commits into
apache:mainfrom
bramhanandlingala:fix/70296-5
Jul 29, 2026
Merged

Rename StepFunctionStartExecutionOperator's input field to match its constructor argument#70544
vincbeck merged 2 commits into
apache:mainfrom
bramhanandlingala:fix/70296-5

Conversation

@bramhanandlingala

@bramhanandlingala bramhanandlingala commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Fixes the amazon provider's StepFunctionStartExecutionOperator entry from the #70296 exemption-list burn-down.

state_machine_input is a template field, but __init__ assigned it to self.input — a name that doesn't match either the constructor argument (state_machine_input) or the declared template field, so the validate-operators-init prek hook flagged it as a missing/invalid assignment.

Renamed the internal attribute (and the corresponding entry in template_fields) from input to state_machine_input so it matches the constructor argument directly. No transformation or validation was involved — this was a pure naming mismatch, not a rendering-order bug — and the public constructor signature is unchanged, so this isn't a breaking change for existing DAGs.

  • Renamed self.inputself.state_machine_input in __init__ and execute().
  • Updated template_fields accordingly.
  • Updated the one test assertion that referenced op.input.
  • Removed the operator's entry from validate_operators_init_exemptions.txt.

Verified locally that scripts/ci/prek/validate_operators_init.py reports zero findings for this class after the change, and that the existing unit tests in test_step_function.py pass unmodified aside from the one updated attribute assertion.

Related to #70296

Gen-AI disclosure: I used a generative AI tool to help identify the root
cause, write tests, and draft the PR description. I reviewed, tested, and
verified all changes locally before submitting.

Was generative AI tooling used to co-author this PR?

  • Yes [Claude]

Generated-by: Claude following the guidelines

@boring-cyborg boring-cyborg Bot added area:dev-tools area:providers backport-to-v3-3-test Backport to v3-3-test provider:amazon AWS/Amazon - related issues labels Jul 27, 2026
@bramhanandlingala bramhanandlingala changed the title fix(amazon): rename StepFunctionStartExecutionOperator's input field … Rename StepFunctionStartExecutionOperator's input field to match its constructor argument Jul 27, 2026
Comment thread airflow-core/src/airflow/jobs/scheduler_job_runner.py
@potiuk potiuk added the ready for maintainer review Set after triaging when all criteria pass. label Jul 28, 2026
@vincbeck
vincbeck merged commit eed1a39 into apache:main Jul 29, 2026
83 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Backport failed to create: v3-3-test. View the failure log Run details

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
v3-3-test Commit Link

You can attempt to backport this manually by running:

cherry_picker eed1a39 v3-3-test

This should apply the commit to the v3-3-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

If you don't have cherry-picker installed, see the installation guide.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools area:providers backport-to-v3-3-test Backport to v3-3-test provider:amazon AWS/Amazon - related issues ready for maintainer review Set after triaging when all criteria pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants