Container.exec_run returns None as exit_code if stream or socket

Signed-off-by: Frank Sachsenheim <[email protected]>

Author: funkyfuture

docker/models/containers.py

@@ -152,7 +152,8 @@ def exec_run(self, cmd, stdout=True, stderr=True, stdin=False, tty=False,
         Returns:
             (tuple): A tuple of (exit_code, output)
                 exit_code: (int):
-                    Exit code for the executed command
+                    Exit code for the executed command or ``None`` if
+                    either ``stream```or ``socket`` is ``True``.
                 output: (generator or str):
                     If ``stream=True``, a generator yielding response chunks.
                     If ``socket=True``, a socket object for the connection.
@@ -170,10 +171,11 @@ def exec_run(self, cmd, stdout=True, stderr=True, stdin=False, tty=False,
         exec_output = self.client.api.exec_start(
             resp['Id'], detach=detach, tty=tty, stream=stream, socket=socket
         )
-        exit_code = 0
-        if stream is False:
-            exit_code = self.client.api.exec_inspect(resp['Id'])['ExitCode']
-        return (exit_code, exec_output)
+        if socket or stream:
+            return None, exec_output
+        else:
+            return (self.client.api.exec_inspect(resp['Id'])['ExitCode'],
+                    exec_output)
 
     def export(self):
         """