Taragolis opened a new pull request, #28363: URL: https://github.com/apache/airflow/pull/28363
Right now `DockerHook` only partially uses in `DockerOperator` and `DockerSwarmOperator`. This PR refactor some parts 1. Drop raising errors for most of the missing arguments during constructing hook. In Docker SDK all of them optional. 2. Move out all expensive call from `__init__`, e.g. obtain connection. 3. `docker_conn_id` now optional. If user not required login to docker Registry this field might be set to None. Previously this functional sits in `DockerOperator` 4. Cache connection to docker server/host. 5. Move out constructing TLSConfig from `DockerOperator` to `DockerHook` This kind of integration will allow in the future move all complex logic from Operators to Hook and make them reusable. Also this PR pre-requirements for https://github.com/apache/airflow/pull/26162 -- 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]
