Skip to content

Commit

Permalink
Merge pull request kubernetes-sigs#1546 from saschagrunert/oom
Browse files Browse the repository at this point in the history
Fix OOMKilled test assertions
  • Loading branch information
k8s-ci-robot authored Aug 5, 2024
2 parents 22c71cc + 22d7ff9 commit 8aed864
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions pkg/validate/container_linux.go
Original file line number Diff line number Diff line change
Expand Up @@ -159,12 +159,10 @@ var _ = framework.KubeDescribe("Container OOM", func() {
state := getContainerStatus(rc, containerID)

By("exit code is 137")
//nolint:ginkgolinter // This cannot be fixed right now since it introduces a behavioral change. See: https://github.com/cri-o/cri-o/issues/8411
Expect(state.ExitCode, int32(137))
Expect(state.ExitCode).To(BeEquivalentTo(137))

By("reason is OOMKilled")
//nolint:ginkgolinter // This cannot be fixed right now since it introduces a behavioral change. See: https://github.com/cri-o/cri-o/issues/8411
Expect(state.Reason, "OOMKilled")
Expect(state.Reason).To(Equal("OOMKilled"))
})
})
})
Expand Down

0 comments on commit 8aed864

Please sign in to comment.