Browse Source

Merge pull request #9246 from estesp/add-oom-event

Proposal: Add "OOM killed" event based on OOM state information
James Turnbull 10 years ago
parent
commit
8722b6db4b

+ 6 - 0
daemon/monitor.go

@@ -154,6 +154,9 @@ func (m *containerMonitor) Start() error {
 
 
 		if m.shouldRestart(exitStatus.ExitCode) {
 		if m.shouldRestart(exitStatus.ExitCode) {
 			m.container.SetRestarting(&exitStatus)
 			m.container.SetRestarting(&exitStatus)
+			if exitStatus.OOMKilled {
+				m.container.LogEvent("oom")
+			}
 			m.container.LogEvent("die")
 			m.container.LogEvent("die")
 			m.resetContainer(true)
 			m.resetContainer(true)
 
 
@@ -170,6 +173,9 @@ func (m *containerMonitor) Start() error {
 			continue
 			continue
 		}
 		}
 		m.container.ExitCode = exitStatus.ExitCode
 		m.container.ExitCode = exitStatus.ExitCode
+		if exitStatus.OOMKilled {
+			m.container.LogEvent("oom")
+		}
 		m.container.LogEvent("die")
 		m.container.LogEvent("die")
 		m.resetContainer(true)
 		m.resetContainer(true)
 		return err
 		return err

+ 1 - 1
docs/sources/reference/api/docker_remote_api_v1.17.md

@@ -1375,7 +1375,7 @@ polling (using since).
 
 
 Docker containers will report the following events:
 Docker containers will report the following events:
 
 
-    create, destroy, die, export, kill, pause, restart, start, stop, unpause
+    create, destroy, die, export, kill, oom, pause, restart, start, stop, unpause
 
 
 and Docker images will report:
 and Docker images will report:
 
 

+ 1 - 1
docs/sources/reference/commandline/cli.md

@@ -761,7 +761,7 @@ For example:
 
 
 Docker containers will report the following events:
 Docker containers will report the following events:
 
 
-    create, destroy, die, export, kill, pause, restart, start, stop, unpause
+    create, destroy, die, export, kill, oom, pause, restart, start, stop, unpause
 
 
 and Docker images will report:
 and Docker images will report: