Jarek Potiuk
Jarek Potiuk
> > 2. not sure if we have tasks running as sub-processes (that prevented step-by-step debugging between tasks) - but I guess this should be possible to run all task...
I think this is good idea, but the ECRDockerCredentialHelper should be part of Amazon provider, and likely there should be an ECRDockerOperator that uses it by default. This is very...
> @potiuk FYI: The same problem exists in Postgres hook: > > Yeah it could be. I put it into docker provider for reduce complexity of dependency. > And also...
> Why I move this part inside of DockerHook hook: > docker_conn_id not only use in DockerOperator also it use in DockerSwarmOperator and @task.docker task-flow decorator which won't work with...
Wrong box to open :). This is not part of it. Both of the examples you mentioned @Taragolis are clearly AWS ones they need no fix as they are in...
> > Wrong box to open :). This is not part of it. > > But it do almost the same the same thing that `PostgresHook.get_iam_token` and `MySqlHook.get_iam_token` - grant...
Why not using the oppoertunity to add the auth mechanism now? I do not think this needs a super-generic solution that will work for all hooks. I think there are...
Yep. I want to see how your proposal will look like after rebase, but yeah - the smaller set of changes, the better for now.
This could be done as a config entry in Airflow. On one hand it is not ideal (because users would not be able to modify it) but on the other...