Hi @feng-tao , I have
- Added instance checking for argument `ssh_hook`. If `ssh_hook` given by user 
is not of class "SSHHook", it would be handled properly.
- Updated the tests for both operators accordingly and both can pass.

The CI is failing due to:
- test_poke 
(tests.contrib.sensors.test_aws_redshift_cluster_sensor.TestAwsRedshiftClusterSensor)
 ... ERROR
- test_poke_cluster_not_found 
(tests.contrib.sensors.test_aws_redshift_cluster_sensor.TestAwsRedshiftClusterSensor)
 ... ERROR
- test_poke_false 
(tests.contrib.sensors.test_aws_redshift_cluster_sensor.TestAwsRedshiftClusterSensor)
 ... ERROR

None of these test cases were touched in this PR. Seems it's exceptions like 
`Unable to locate credentials`. Not sure what is happening.

[ Full content available at: 
https://github.com/apache/incubator-airflow/pull/3793 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to