Bring back deployment e2e commented out due to journald limitting problem
@@ -301,10 +301,7 @@ var _ = g.Describe("[Feature:DeploymentConfig] deploymentconfigs", func() {
g.By(fmt.Sprintf("checking the logs for substrings\n%s", out))
o.Expect(out).To(o.ContainSubstring("deployment-test-1 to 2"))
o.Expect(out).To(o.ContainSubstring("--> pre: Success"))
- // FIXME: In some cases the last log messages is lost because of the journald rate
- // limiter bug. For this test it should be enough to verify the deployment is marked
- // as complete. We should uncomment this once the rate-limiter issues are fixed.
- // o.Expect(out).To(o.ContainSubstring("--> Success"))
+ o.Expect(out).To(o.ContainSubstring("--> Success"))
g.By("verifying the deployment is marked complete and scaled to zero")
o.Expect(waitForLatestCondition(oc, "deployment-test", deploymentRunTimeout, deploymentReachedCompletion)).NotTo(o.HaveOccurred())
@@ -354,10 +351,7 @@ var _ = g.Describe("[Feature:DeploymentConfig] deploymentconfigs", func() {
o.Expect(out).To(o.ContainSubstring(fmt.Sprintf("deployment-test-%d up to 1", i+2)))
o.Expect(out).To(o.ContainSubstring("--> pre: Success"))
o.Expect(out).To(o.ContainSubstring("test pre hook executed"))
- // FIXME: In some cases the last log messages is lost because of the journald rate
- // limiter bug. For this test it should be enough to verify the deployment is marked
- // as complete. We should uncomment this once the rate-limiter issues are fixed.
- // o.Expect(out).To(o.ContainSubstring("--> Success"))
+ o.Expect(out).To(o.ContainSubstring("--> Success"))
}
})
})
@@ -556,10 +550,7 @@ var _ = g.Describe("[Feature:DeploymentConfig] deploymentconfigs", func() {
o.Expect(out).To(o.ContainSubstring("--> Reached 50%"))
o.Expect(out).To(o.ContainSubstring("Halfway"))
o.Expect(out).To(o.ContainSubstring("Finished"))
- // FIXME: In some cases the last log messages is lost because of the journald rate
- // limiter bug. For this test it should be enough to verify the deployment is marked
- // as complete. We should uncomment this once the rate-limiter issues are fixed.
- // o.Expect(out).To(o.ContainSubstring("--> Success"))
+ o.Expect(out).To(o.ContainSubstring("--> Success"))
})
})
