You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/12/22 21:53:19 UTC

[GitHub] [airflow] syedahsn commented on pull request #28484: Add execution role parameter to AddStepsOperator

syedahsn commented on PR #28484:
URL: https://github.com/apache/airflow/pull/28484#issuecomment-1363382860

   Made a minor change with how the new parameter gets used in the hook. Because we are using `@mock_emr` decorator in some of the tests, it applies boto3 validation rules on the parameters passed, which include no `None` values, as well as a minimum length (otherwise I could have done a simpler `ExecutionRoleArn=execution_role_arn or ""`). 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org