[AIRFLOW-6402] fix ssh_operator default value

[ohadmata]

[AIRFLOW-6402] https://issues.apache.org/jira/browse/AIRFLOW-6402

Fix for ssh operator default value.
Right now I am getting Nonetype Exception (1.10.7)

[kaxil]

Can you also add a test for it so this can be avoided in future, please?

Test for this operator are at: https://github.com/apache/airflow/blob/master/tests/contrib/operators/test_ssh_operator.py

[dstandish]

this operator raises exception if no command is provided:

airflow/airflow/contrib/operators/ssh_operator.py

Lines 108 to 109 in 4984578

	if not self.command:
	raise AirflowException("SSH command not specified. Aborting.")

does it make sense to just pass empty string super().__init__(command='') when subclassing?

seems like it would make sense to make command not optional i.e. remove the default value of None

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[kaxil]

this operator raises exception if no command is provided:

airflow/airflow/contrib/operators/ssh_operator.py

Lines 108 to 109 in 4984578

	if not self.command:
	raise AirflowException("SSH command not specified. Aborting.")

does it make sense to just pass empty string super().__init__(command='') when subclassing?

seems like it would make sense to make command not optional i.e. remove the default value of None

Yes, I don't see why someone would use SSHOperator without a "command".

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[CRX-DS]

Should this TASK work now?:
t1 = SSHOperator(
task_id='t1',
bash_command='echo 5',
ssh_conn_id='myconnection1',
dag=dag
)