Merge pull request #39342 from jccomputing/detach

dockerng: fix documentation for detach's default value

doc/man/salt.7

@@ -98664,10 +98664,10 @@ Attach TTYs
 Example: \fBtty=True\fP
 .TP
 .B detach
-True
+False
 If \fBTrue\fP, run \fBcommand\fP in the background (daemon mode)
 .sp
-Example: \fBdetach=False\fP
+Example: \fBdetach=True\fP
 .TP
 .B user
 User under which to run docker

salt/modules/dockerng.py

@@ -2836,10 +2836,10 @@ def create(image,
 
         Example: ``tty=True``
 
-    detach : True
+    detach : False
         If ``True``, run ``command`` in the background (daemon mode)
 
-        Example: ``detach=False``
+        Example: ``detach=True``
 
     user
         User under which to run docker

salt/states/dockerng.py

@@ -980,7 +980,7 @@ def running(name,
                 - image: bar/baz:latest
                 - tty: True
 
-    detach : True
+    detach : False
         If ``True``, run the container's command in the background (daemon
         mode)
 
@@ -989,7 +989,7 @@ def running(name,
             foo:
               dockerng.running:
                 - image: bar/baz:latest
-                - detach: False
+                - detach: True
 
     user
         User under which to run docker