Skip to content

Validate EmrAddStepsOperator job flow arguments after rendering - #70448

Closed
haseebmalik18 wants to merge 1 commit into
apache:mainfrom
haseebmalik18:fix-emr-add-steps-template-validation
Closed

Validate EmrAddStepsOperator job flow arguments after rendering#70448
haseebmalik18 wants to merge 1 commit into
apache:mainfrom
haseebmalik18:fix-emr-add-steps-template-validation

Conversation

@haseebmalik18

Copy link
Copy Markdown
Contributor

job_flow_id and job_flow_name on EmrAddStepsOperator are template fields, so Jinja renders them after __init__ runs. The mutual-exclusion check lived in __init__, which meant it validated the raw {{ ... }} expressions instead of the actual values. This moves the check into execute() so it runs after rendering.

Part of #70296.


@shahar1

Copy link
Copy Markdown
Contributor

Closing as false positive according to the up-to-date definition, see details here:
#70505
Feel free to take on the remaining operators :)

@shahar1shahar1 closed this Jul 27, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-toolsarea:providersbackport-to-v3-3-testBackport to v3-3-testprovider:amazonAWS/Amazon - related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@haseebmalik18@shahar1