[6.2.z] cherry-pick test rex connect by ip

[pondrejk]

Cherry-pick from #4648
Related issue: #4624
Test results:

Run custom template on host connected via ip ... ok

----------------------------------------------------------------------
Ran 1 test in 304.554s

OK

 nosetests -v tests/foreman/cli/test_remoteexecution.py:RemoteExecutionTestCase.test_positive_run_default_job_template_by_ip
Run default template on host connected via ip ... ok

----------------------------------------------------------------------
Ran 1 test in 299.689s

OK

[mention-bot]

@pondrejk, thanks for your PR! By analyzing the history of the files in this pull request, we identified @abalakh to be a potential reviewer.

[renzon]

ACK

[oshtaier]

Maybe we should put next statements into try...finally block to return settings value no matter what result we get?

[pondrejk]

@oshtaier addressed, ok with this form?

[coveralls]

Coverage remained the same at 40.146% when pulling 0849141 on pondrejk:remote_execution_connect_by_ip-6.2.z into c4b6303 on SatelliteQE:6.2.z.

[coveralls]

Coverage remained the same at 40.146% when pulling a365a21 on pondrejk:remote_execution_connect_by_ip-6.2.z into c4b6303 on SatelliteQE:6.2.z.

[pondrejk]

Hello, after some research and discussion with @rplevka, I decided to take a different approach to these tests (using per-host setting instead of global setting). Therefore I put do_not_merge. Also this can help us to get more rex tests running in automation until the problems from #4578 are resolved

[pondrejk]

@renzon @oshtaier ready for review again, I enable the remote_execution_connect_by_ip per host, I added more tests -- these are by_ip version of existing tests. Tests:

py.test tests/foreman/cli/test_remoteexecution.py -k _by_ip 
...
=================== 5 passed, 1 skipped, 15 deselected in 1128.98 seconds ====================

[codecov]

Codecov Report

Merging #4657 into 6.2.z will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            6.2.z    #4657      +/-   ##
==========================================
- Coverage   40.31%   40.28%   -0.03%     
==========================================
  Files          87       87              
  Lines        7477     7477              
==========================================
- Hits         3014     3012       -2     
- Misses       4463     4465       +2

Impacted Files	Coverage Δ
robottelo/decorators/func_locker.py	92.12% <0%> (-1.58%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 08abc9f...ab0c355. Read the comment docs.

[renzon]

ACK

[rplevka]

ACK