[I] SSHHook require remote_host in __init__ but SSHOperator set remote_host after __init__ [airflow]

["dispensable (via GitHub)" <gi...@apache.org>]

dispensable opened a new issue, #55663:
URL: https://github.com/apache/airflow/issues/55663

   ### Apache Airflow Provider(s)
   
   ssh
   
   ### Versions of Apache Airflow Providers
   
   4.1.3
   
   ### Apache Airflow version
   
   3.0.6
   
   ### Operating System
   
   debian bookworm
   
   ### Deployment
   
   Official Apache Airflow Helm Chart
   
   ### Deployment details
   
   _No response_
   
   ### What happened
   
   in SSHHook https://github.com/apache/airflow/blob/8634ce0d3fc030aa783527ea34e2d9bd4209ceed/providers/ssh/src/airflow/providers/ssh/hooks/ssh.py#L232 requires remote_host.
   
   in SSHOperator https://github.com/apache/airflow/blob/8634ce0d3fc030aa783527ea34e2d9bd4209ceed/providers/ssh/src/airflow/providers/ssh/operators/ssh.py#L130 if ssh_conn_id set, remote_host been passed after SSHook __init__
   
   if you use ssh_conn_id in SSHOperator and set remote_host, you still got `Missing required remote_host` error
   
   ### What you think should happen instead
   
   ssh_conn_id and remote_host should init a correct SSHHook
   
   ### How to reproduce
   
   just use SSHOperator with ssh_conn_id(no remote_host attr set) + remote_host like doc shows
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [x] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [x] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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: commits-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] SSHHook/SFTPHook require remote_host during __init__ but operator set remote_host arg after __init__ [airflow]

["potiuk (via GitHub)" <gi...@apache.org>]

potiuk closed issue #55663: SSHHook/SFTPHook require remote_host during __init__ but operator set remote_host arg after __init__
URL: https://github.com/apache/airflow/issues/55663


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] SSHHook require remote_host in __init__ but SSHOperator set remote_host after __init__ [airflow]

["boring-cyborg[bot] (via GitHub)" <gi...@apache.org>]

boring-cyborg[bot] commented on issue #55663:
URL: https://github.com/apache/airflow/issues/55663#issuecomment-3290758125

   Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org