https://bugzilla.redhat.com/show_bug.cgi?id=1096293
Bug ID: 1096293 Summary: `docker start` doesn't fail on started container Product: Red Hat Enterprise Linux 7 Version: 7.1 Component: docker Assignee: lsm5@redhat.com Reporter: ldoktor@redhat.com QA Contact: virt-bugs@redhat.com CC: admiller@redhat.com, golang@lists.fedoraproject.org, lsm5@redhat.com, mattdm@redhat.com, mgoldman@redhat.com, s@shk.io, vbatts@redhat.com Depends On: 1090071
+++ This bug was initially created as a clone of Bug #1090071 +++
Description of problem: On older (0.9) docker, the `docker start $running_container` failed. The new 0.10.2 version just prints the container name and proceeds.
Version-Release number of selected component (if applicable): docker-0.10.0-8.el7.x86_64 docker-io-0.10.0-2.fc20.x86_64
How reproducible: Always
Steps to Reproduce: 1. docker run -i -t -name test fedora bash 2. docker start test
Actual results: [root@t530 ~]# docker start test test [root@t530 ~]# echo &? 0
Expected results: [root@t530 ~]# docker start test Error: Cannot start container test: The container 429d709bc86c037b09e9f74dfab40aa33dc19bcf3109daf3f5b2fbbd3c6df297 is already running. 2014/04/22 15:29:28 Error: failed to start one or more containers [root@t530 ~]# echo &? 1
--- Additional comment from Lukas Doktor on 2014-05-05 03:38:57 EDT ---
Upstream Docker version 0.10.0, build dc9c28f/0.10.0 has the same issue (silently passes even thought the container was already running)
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1090071 [Bug 1090071] `docker start` doesn't fail on started container
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
Chris Evich cevich@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |cevich@redhat.com Version|7.1 |7.0 Severity|unspecified |high
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
Daniel Walsh dwalsh@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dwalsh@redhat.com
--- Comment #3 from Daniel Walsh dwalsh@redhat.com --- I don't think this is a bug but a request for enhancement. I don't see any documentation that says this is what docker should do.
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
--- Comment #4 from Chris Evich cevich@redhat.com ---
IMHO, the destinction would be if it could even be a problem to ignore the second 'start'. I could see this as splitting-hairs as well, but in that case it should probably at least be documented behavior somewhere.
I'm not a concurrency/parallel expert, but I'd guess as long as either double-start or double-stop reports an error it's fine. It's asking for trouble if both ignore the double call. In that case I'd call it a bug b/c: The start or stop needn't come from the same interface (i.e. CLI and REST, arriving at the "same" time). In other words, as long as one way (start or stop) ignores doubles, but the other way makes a stink, there are unlikely to be any races possible in customer use/calls.
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
--- Comment #5 from Lukas Doktor ldoktor@redhat.com --- The main issue here is inconsistency. Old docker behaved differently, than the current one. It would be nice to choose one way and stick with it.
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
--- Comment #6 from Daniel Walsh dwalsh@redhat.com --- Lukas. I agree I think this one should be fixed. I think docker start should tell you the container is already started.
But since the app is controlled by upstream, we only have the ability to send patches.
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
--- Comment #7 from Chris Evich cevich@redhat.com --- IMHO, the between-versions behavior inconsistency is nearly always a hard-sell, especially for something sooooo new. While having it change like this is frustrating for testing/development, it's somewhat to be expected. We'll have a much easier argument to make for this change if it can be pinned to a nastier problem to expect end-users to workaround. If we can't, then probably best close this and fixup our tests with version-dependant behavior.
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
Lokesh Mandvekar lsm5@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |mheon@redhat.com Assignee|lsm5@redhat.com |mheon@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
--- Comment #10 from Chris Evich cevich@redhat.com --- Confirmed fixed in docker-0.11.1-19.el7.x86_64 (or slightly earlier). Multiple starts, just keep printing the container's name for an already running container.
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
Matthew Heon mheon@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED Fixed In Version| |docker-0.11.1-19.el7
--- Comment #11 from Matthew Heon mheon@redhat.com --- Given that this change was intentionally made by upstream, we're not going to fix it. There is now documentation in the manpages and Docker web documentation which establishes that this is intended behaviour (included in docker-0.11.1-19.el7). This should clarify that this is not a bug and remove confusion.
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
--- Comment #12 from Chris Evich cevich@redhat.com --- Yep, no need to fix it now and updated documentation is great as well. Docker autotest has also been updated to reflect this behavior, so we're all good now. Thanks!
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
Chris Evich cevich@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |VERIFIED
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
Red Hat Bugzilla Rules Engine rule-engine@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |Extras
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
John Skeoch jskeoch@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Assignee|mheon@redhat.com |jrieden@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1096293
Libor Miksik lmiksik@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|VERIFIED |CLOSED CC| |lmiksik@redhat.com Resolution|--- |CURRENTRELEASE Last Closed| |2015-03-26 07:35:24
https://bugzilla.redhat.com/show_bug.cgi?id=1096293 Bug 1096293 depends on bug 1090071, which changed state.
Bug 1090071 Summary: `docker start` doesn't fail on started container https://bugzilla.redhat.com/show_bug.cgi?id=1090071
What |Removed |Added ---------------------------------------------------------------------------- Status|VERIFIED |CLOSED Resolution|--- |EOL
golang@lists.fedoraproject.org