Merge pull request #2160 from gaorong/victimContainerName

change default oom victim container name
2019-02-05 09:27:02 -08:00 · 2019-02-05 09:27:02 -08:00 · 0995283c32
commit 0995283c32
parent 9b2d88cd62 407289527c
2 changed files with 4 additions and 3 deletions
--- a/utils/oomparser/oomparser.go
+++ b/utils/oomparser/oomparser.go
@ -102,6 +102,7 @@ func (self *OomParser) StreamOoms(outStream chan<- *OomInstance) {
 		if in_oom_kernel_log {
 			oomCurrentInstance := &OomInstance{
 				ContainerName:       "/",
+				VictimContainerName: "/",
 				TimeOfDeath:         msg.Timestamp,
 			}
 			for msg := range kmsgEntries {
--- a/utils/oomparser/oomparser_test.go
+++ b/utils/oomparser/oomparser_test.go
@ -266,7 +266,7 @@ func TestStreamOOMs(t *testing.T) {
 				ProcessName:         "badsysprogram",
 				TimeOfDeath:         testTime,
 				ContainerName:       "/",
-				VictimContainerName: "",
+				VictimContainerName: "/",
 			}},
 		},
 		{ // Multiple OOMs