metan-ucw / runltp-ng

Minimalistic LTP testrunner
11 stars 16 forks source link

Add ssh_opts option to backend #24

Closed b10n1k closed 3 years ago

b10n1k commented 3 years ago

This is a small addition which is helpful to pass extra parameters to the ssh connection.

Authored-by: Clemens Famulla-Conrad cfamullaconrad@suse.de

richiejp commented 3 years ago

I merged it, but I do not like the string escaping either. It would be best to put those settings which require it in a temporary config file and pass it to SSH with -F. Then we can remove the escaping @cfconrad @b10n1k

Unless of course there are some command line only settings.

cfconrad commented 3 years ago

Yes that's also an option. So you would go with a ssh-config-file: parameter parsed by run-ltp-ng and adding the -F directly?!

richiejp commented 3 years ago

I suppose we could replace ssh_opts with ssh_config_file. Because it keeps runltp-ng simple and handles everything.