[GitHub] [airflow] ashb commented on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

[GitBox <gi...@apache.org>]

ashb commented on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization
URL: https://github.com/apache/airflow/pull/6715#issuecomment-561101212
 
 
   Yeah, we did think about that as a solution (which means we should probably add a comment/"architecture decision record" somewhere) but the issue is that I know of a link I want to add to EMR that isn't fully static: Linking from EmrAddStepOperator to the job logs in S3, which involves signing a URL that will only be valid for 15minutes. (This uses the Airflow servers AWS credentials, rather than needing the user to have the right permissions, which doesn't really work with S3 easily.)
   
   Because of this only allowing static links in the DB would not work for this use case

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services