TESTS: Fix the ssh configuration - II

[aplopez]

The previous patch fixed the environment for the multihost tests, but system tests also need the fix.

Previous patch: e556bfd

These two patches need to be removed once FreeIPA has updated their tool ipa-client-install to use our new tool sss_ssh_knownhosts.

FreeIPA's PR: freeipa/freeipa#7254

[thalman]

Since this is workaround that will be removed soon, I'm fine with that.

Otherwise I would appreciate better regex like s/^Global...

Also with sed you can use any character in expression as a separator so it is then more readable (You do not have to escape all backslashes - like -e 's#/etc/#/opt/etc/#')

ACK

[aplopez]

Since this is workaround that will be removed soon, I'm fine with that.

Otherwise I would appreciate better regex like s/^Global...

Me too, but that expression would not work. FreeIPA generates the GlobalKnownHostsFile line indented because it is preceded by another one to which it is related. Not using the ^ guarantees it will match wherever it is placed in the line.

Also with sed you can use any character in expression as a separator so it is then more readable (You do not have to escape all backslashes - like -e 's#/etc/#/opt/etc/#')

Nice! I didn't know this. Thanks.

[alexey-tikhonov]

Pushed PR: #7247

• master
  • c858d57 - TESTS: Fix the ssh configuration - II