jscheffl commented on code in PR #68115: URL: https://github.com/apache/airflow/pull/68115#discussion_r3369285819
########## providers/ssh/docs/operators/ssh_remote_job.rst: ########## @@ -164,6 +164,13 @@ Parameters * ``remote_os`` (str, optional): Remote OS type (``"auto"``, ``"posix"``, ``"windows"``). Default: ``"auto"`` * ``skip_on_exit_code`` (int or list, optional): Exit code(s) that should cause task to skip instead of fail +* ``conn_timeout`` (int, optional): SSH connection timeout in seconds +* ``banner_timeout`` (float, optional): Seconds to wait for the SSH banner. Default: 30.0 +* ``conn_retry_attempts`` (int, optional): How many times to attempt the initial SSH connection for + submission and cleanup before failing. Default: 5. Raise this for large fan-outs where the remote + ``sshd`` transiently refuses connections (see :ref:`High Fan-out <howto/operator:SSHRemoteJobOperator:fanout>`) +* ``cleanup_retries`` (int, optional): How many times to retry remote directory cleanup before giving up + and leaving the directory in place. Default: 3 Review Comment: Can we please also advise to use `max_active_tis_per_dag` in the docs and link to https://airflow.apache.org/docs/apache-airflow/stable/authoring-and-scheduling/dynamic-task-mapping.html#placing-limits-on-mapped-tasks preventing overload of backends? -- 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]
