ashb edited a comment on pull request #19894:
URL: https://github.com/apache/airflow/pull/19894#issuecomment-989992102


   This hasn't worked 
https://github.com/apache/airflow/runs/4472124262?check_suite_focus=true -- and 
I've added `sts:AssumeRole` to the instance profile permission, but even after 
15 minutes (more than enough time for IAM to propagate changes) I can't even 
get `aws sts assume-role` to work form one of the AWS self-hosted runners.
   
   Hmmmm.
   
   Oh the trust relationship conditions is why ManualRunnerRole can't assume 
that role (only Github OIDC can). But anyway, since we are running on AWS 
hosted runners with an instance role we don't need any of this. As it's 
blocking main I'm just going to remove the cred section entirely, it's not 
needed.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to