kotovalexarian-likes-github/moby--moby
1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2022-11-09 12:21:53 -05:00
Code Releases Activity

Merge pull request #22575 from mountkin/fix-22536

Browse source 
fix LogConfig for containers created before docker 1.12 on daemon start
This commit is contained in:
Sebastiaan van Stijn 2016-06-13 15:11:01 +02:00 • committed by GitHub
commit 6ba3d63a8c
1 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
daemon/daemon.go
View file

@ -148,6 +148,16 @@ func (daemon *Daemon) restore() error {
logrus.Errorf("Failed to register container %s: %s", c.ID, err) logrus.Errorf("Failed to register container %s: %s", c.ID, err)
continue continue
} }
// The LogConfig.Type is empty if the container was created before docker 1.12 with default log driver.
// We should rewrite it to use the daemon defaults.
// Fixes https://github.com/docker/docker/issues/22536
if c.HostConfig.LogConfig.Type == "" {
if err := daemon.mergeAndVerifyLogConfig(&c.HostConfig.LogConfig); err != nil {
logrus.Errorf("Failed to verify log config for container %s: %q", c.ID, err)
continue
}
}
} }
var wg sync.WaitGroup var wg sync.WaitGroup
var mapLock sync.Mutex var mapLock sync.Mutex