Move forward: stop using cython as not enough optimizations

* Extra type declarations using subclassing
* Add chroot info in call logs

Author: penguinolog

.travis.yml

@@ -21,22 +21,7 @@ _python:
     python: "3.8"
 
 _helpers:
-  - &install_cython pip install --upgrade Cython
   - &build_package python setup.py bdist_wheel
-  - &install_built pip install exec_helpers --no-index -f dist
-  - &test_no_cov  py.test -vvv test
-  - &test_cythonized
-    stage: Test cythonized
-    install:
-      - *upgrade_python_toolset
-      - *install_test_deps
-      - *install_deps
-      - *install_cython
-    script:
-      - *build_package
-      - *install_built
-      - *test_no_cov
-    after_success: skip
 
   - &static_analysis
     stage: Static analysis
@@ -109,14 +94,7 @@ jobs:
       <<: *python36
     - stage: test
       <<: *python37
-#    - stage: test
-#      <<: *python38
-
-    - <<: *test_cythonized
-      <<: *python36
-    - <<: *test_cythonized
-      <<: *python37
-    - <<: *test_cythonized
+    - stage: test
       <<: *python38
 
     - stage: deploy
@@ -130,8 +108,6 @@ jobs:
       install:
         - *upgrade_python_toolset
         - *install_deps
-      script:
-        - ./tools/run_docker.sh "exec_helpers"
       before_deploy:
         - *build_package
       deploy:

azure-pipelines.yml

@@ -99,79 +99,3 @@ jobs:
     parameters: {name: 'Python_37', python: '3.7', architecture: 'x64', kind: 'native'}
   - template: .azure_pipelines/run_tests.yml
     parameters: {name: 'Python_38', python: '3.8', architecture: 'x64', kind: 'native'}
-
-  - template: .azure_pipelines/run_tests.yml
-    parameters: {name: 'Python_36', python: '3.6', architecture: 'x64', kind: 'cython'}
-  - template: .azure_pipelines/run_tests.yml
-    parameters: {name: 'Python_36', python: '3.6', architecture: 'x86', kind: 'cython'}
-  - template: .azure_pipelines/run_tests.yml
-    parameters: {name: 'Python_37', python: '3.7', architecture: 'x64', kind: 'cython'}
-  - template: .azure_pipelines/run_tests.yml
-    parameters: {name: 'Python_37', python: '3.7', architecture: 'x86', kind: 'cython'}
-  - template: .azure_pipelines/run_tests.yml
-    parameters: {name: 'Python_38', python: '3.8', architecture: 'x64', kind: 'cython'}
-  - template: .azure_pipelines/run_tests.yml
-    parameters: {name: 'Python_38', python: '3.8', architecture: 'x86', kind: 'cython'}
-
-  - job: 'Build_and_deploy'
-    dependsOn:
-      - Python_36_x64_native
-      - Python_37_x64_native
-      - Python_38_x64_native
-
-      - Python_36_x64_cython
-      - Python_36_x86_cython
-      - Python_37_x64_cython
-      - Python_37_x86_cython
-      - Python_38_x64_cython
-      - Python_38_x86_cython
-    condition: and(succeeded(), startsWith(variables['Build.SourceBranch'], 'refs/tags/'))
-    pool:
-      vmIMage: 'VS2017-Win2016'
-    strategy:
-      maxParallel: 6
-      matrix:
-        Python36_x64:
-          python.version: '3.6'
-          python.architecture: 'x64'
-        Python36_x86:
-          python.version: '3.6'
-          python.architecture: 'x86'
-        Python37_x64:
-          python.version: '3.7'
-          python.architecture: 'x64'
-        Python37_x86:
-          python.version: '3.7'
-          python.architecture: 'x86'
-        Python38_x64:
-          python.version: '3.8'
-          python.architecture: 'x64'
-        Python38_x86:
-          python.version: '3.8'
-          python.architecture: 'x86'
-
-    steps:
-      - task: UsePythonVersion@0
-        inputs:
-          versionSpec: '$(python.version)'
-          architecture: '$(python.architecture)'
-
-      - script: |
-          python -m pip install --upgrade pip
-          pip install -U setuptools
-          pip install -r build_requirements.txt
-        displayName: 'Install dependencies'
-
-      - script: |
-          python setup.py bdist_wheel
-        displayName: 'Build'
-
-      - task: TwineAuthenticate@0
-        displayName: 'Twine Authenticate '
-        inputs:
-          externalFeeds: PyPI
-
-      - script: |
-          pip install -U twine
-          twine upload -r PyPI --config-file $(PYPIRC_PATH) dist/exec_helpers-* --skip-existing
-        displayName: 'Deploy'

exec_helpers/_ssh_client_base.py

@@ -506,7 +506,7 @@ def __del__(self) -> None:
 
     def __enter__(self) -> "SSHClientBase":  # pylint: disable=useless-super-delegation
         """Get context manager."""
-        return super().__enter__()  # type: ignore
+        return super().__enter__()
 
     def __exit__(self, exc_type: typing.Any, exc_val: typing.Any, exc_tb: typing.Any) -> None:
         """Exit context manager.
@@ -1042,7 +1042,10 @@ def get_result(remote: "SSHClientBase") -> exec_result.ExecResult:
             cmd_for_log: str = remote._mask_command(cmd=command, log_mask_re=log_mask_re)
 
             remote.logger.log(
-                level=logging.INFO if verbose else logging.DEBUG, msg=f"Executing command:\n{cmd_for_log!r}\n"
+                level=log_level,
+                msg=f"Executing command"
+                f"{'' if not remote._chroot_path else f' (with chroot to: {remote._chroot_path})'}:\n"
+                f"{cmd_for_log!r}\n",
             )
             async_result: SshExecuteAsyncResult = remote._execute_async(
                 command,
@@ -1068,6 +1071,7 @@ def get_result(remote: "SSHClientBase") -> exec_result.ExecResult:
         prep_expected: typing.Tuple[typing.Union[int, proc_enums.ExitCodes], ...] = proc_enums.exit_codes_to_enums(
             expected
         )
+        log_level: int = logging.INFO if verbose else logging.DEBUG
 
         futures: typing.Dict["SSHClientBase", "concurrent.futures.Future[exec_result.ExecResult]"] = {
             remote: get_result(remote) for remote in set(remotes)