Taragolis commented on PR #26162: URL: https://github.com/apache/airflow/pull/26162#issuecomment-1248032448
@mik-laj @potiuk Let's finalise discussion. We not ready to add auth to Amazon ECR by this way it described in the PR for different reasons: 1. We need to move auth to AWS from postgres provider to amazon 2. We can not use `import_string` for untrusted modules but right now but if we move `EcrDockerCredentialHelper` to aws I do not see any possibility to do that without specify it in somewhere in postgres hook. So might be some additional way to help user to integrate with ECR if it required? It could be some page in to documentation like this snippet https://github.com/apache/airflow/issues/9707#issuecomment-773353852 or create Operator which could update specific docker connection by ECR credentials WDYT guys? -- 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]
