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 (covered different combinations of arguments `ssh_hook` and `ssh_conn_id`). 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 exception like `Unable to locate credentials`. Not sure what is happening. Similar issue is happening to a few other PRs. [ Full content available at: https://github.com/apache/incubator-airflow/pull/3793 ] This message was relayed via gitbox.apache.org for [email protected]
