Remove unnecessary job "initserverpidfile"

That job was a hacky solution to a real race condition. This removes the
hack without re-introducing the race.

Signed-off-by: Solomon Hykes <solomon@docker.com>
Replace Server.IsRunning with Engine.IsShutdown

Signed-off-by: Solomon Hykes <solomon@docker.com>
Extra testing for engine.Shutdown

Signed-off-by: Solomon Hykes <solomon@docker.com>
Remove last trace of Daemon->Server dependency

This removes a shim `daemon.Server` interface which was used to start
separating Daemon from Server *gradually*, without getting cyclic
dependency errors.

Now that the last Daemon->Server dependency has been removed, we can
finally remove the shim. Yay!

Signed-off-by: Solomon Hykes <solomon@docker.com>
Remove unused field from engine.Job

Signed-off-by: Solomon Hykes <solomon@docker.com>
Engine.Shutdown only waits 5 seconds for active jobs to complete

Signed-off-by: Solomon Hykes <solomon@docker.com>
Update CONTRIBUTING.md

Added link to community guidelines
Make it clear that JSON array format should be used if CMD is used as default arguments

Signed-off-by: Xuecong Liao <satorulogic@gmail.com>
Revert `rm -f` deprecation use SIGKILL instead

`rm -f` was originally deprecated in favor of `rm --stop/--kill` since `rm
-f` was sending SIGTERM and potentially very slow.
Instead this will bring back `rm -f` but use SIGKILL isntead

Docker-DCO-1.1-Signed-off-by: Brian Goff <cpuguy83@gmail.com> (github: cpuguy83)
Fix several minor issues with Debian sysvinit script

- add `set -e` to make failing commands bail the script
- remove trailing `exit 0` which is just extraneous anyhow
- adjust `status_of_proc` options to pass in `$DOCKER_DESC` so we get consistently styled messages like `Docker is running` or `Docker is not running` or `could not access PID file for Docker`

Signed-off-by: Andrew Page <admwiggin@gmail.com>
Merge pull request #7453 from shykes/new-subsystem-maintainers

Add maintainers to daemon/ and graph/
Merge pull request #7374 from cpuguy83/revert_rm_f_deprecation

Revert --stop/--kill and use SIGKILL instead
Merge pull request #7476 from tianon/sysvinit-status-of-proc

Fix several minor issues with Debian sysvinit script
Merge pull request #7458 from jamtur01/cg

Update CONTRIBUTING.md
Merge pull request #7460 from suzaku/fix_docs

Make it clear that JSON array format should be used if CMD is used as default arguments
Move "info" to daemon/info.go

Signed-off-by: Solomon Hykes <solomon@docker.com>
Merge pull request #7278 from Toebez/patch-1

Update nodejs_web_app.md
Move "pull" and "push" to graph/pull.go and graph/push.go

This is part of the ongoing effort to remove the deprecated server/
package, and generally cleanup and simplify the codebase.

Signed-off-by: Solomon Hykes <solomon@docker.com>
Updated openSUSE's installation instructions

Cover network configuration and reference the official openSUSE image
inside of the examples.

Docker-DCO-1.1-Signed-off-by: Flavio Castelli <fcastelli@suse.com> (github: flavio)
Use only unavailable image when load from Tarball

Docker-DCO-1.1-Signed-off-by: Satoshi Amemiya <satoshi_amemiya@voyagegroup.com> (github: rail44)