Do not share subprocess lock between instances

Signed-off-by: Aleksei Stepanov <penguinolog@gmail.com>

(cherry picked from commit 82da78d)
Signed-off-by: Aleksei Stepanov <penguinolog@gmail.com>

Author: penguinolog

doc/source/Subprocess.rst

@@ -21,6 +21,7 @@ API: Subprocess
         .. versionchanged:: 3.1.0 Not singleton anymore. Only lock is shared between all instances.
         .. versionchanged:: 3.2.0 Logger can be enforced.
         .. versionchanged:: 4.1.0 support chroot
+        .. versionchanged:: 4.3.0 Lock is not shared anymore: allow parallel call of different instances
 
     .. py:attribute:: log_mask_re
 

exec_helpers/async_api/subprocess_runner.py

@@ -30,7 +30,6 @@
 from exec_helpers import _log_templates
 from exec_helpers import _subprocess_helpers
 from exec_helpers import exceptions
-from exec_helpers import metaclasses  # pylint: disable=unused-import
 from exec_helpers import subprocess_runner
 from exec_helpers.async_api import api
 from exec_helpers.async_api import exec_result
@@ -64,7 +63,7 @@ def stdout(self) -> typing.Optional[asyncio.StreamReader]:  # type: ignore
         return super(SubprocessExecuteAsyncResult, self).stdout
 
 
-class Subprocess(api.ExecHelper, metaclass=metaclasses.SingleLock):
+class Subprocess(api.ExecHelper):
     """Subprocess helper with timeouts and lock-free FIFO."""
 
     __slots__ = ()
@@ -85,6 +84,8 @@ def __init__(
 
         .. versionchanged:: 3.1.0 Not singleton anymore. Only lock is shared between all instances.
         .. versionchanged:: 3.2.0 Logger can be enforced.
+        .. versionchanged:: 4.1.0 support chroot
+        .. versionchanged:: 4.3.0 Lock is not shared anymore: allow parallel call of different instances.
         """
         super(Subprocess, self).__init__(logger=logger, log_mask_re=log_mask_re)
 

exec_helpers/metaclasses.py

@@ -35,7 +35,6 @@
 from exec_helpers import api
 from exec_helpers import exceptions
 from exec_helpers import exec_result
-from exec_helpers import metaclasses  # pylint: disable=unused-import
 
 
 # noinspection PyTypeHints
@@ -63,7 +62,7 @@ def stdout(self) -> typing.Optional[typing.IO]:  # type: ignore
         return super(SubprocessExecuteAsyncResult, self).stdout
 
 
-class Subprocess(api.ExecHelper, metaclass=metaclasses.SingleLock):
+class Subprocess(api.ExecHelper):
     """Subprocess helper with timeouts and lock-free FIFO."""
 
     def __init__(
@@ -85,6 +84,8 @@ def __init__(
         .. versionchanged:: 1.2.0 log_mask_re regex rule for masking cmd
         .. versionchanged:: 3.1.0 Not singleton anymore. Only lock is shared between all instances.
         .. versionchanged:: 3.2.0 Logger can be enforced.
+        .. versionchanged:: 4.1.0 support chroot
+        .. versionchanged:: 4.3.0 Lock is not shared anymore: allow parallel call of different instances.
         """
         super(Subprocess, self).__init__(logger=logger, log_mask_re=log_mask_re)
 

exec_helpers/subprocess_runner.py

@@ -68,7 +68,6 @@ def _extension(modpath):
     _extension("exec_helpers.constants"),
     _extension("exec_helpers.exceptions"),
     _extension("exec_helpers.exec_result"),
-    _extension("exec_helpers.metaclasses"),
     _extension("exec_helpers.proc_enums"),
     _extension("exec_helpers.ssh_auth"),
     _extension("exec_helpers.ssh_client"),