-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: Add missing gomega Eventually intervals #14388
Conversation
Gomega time interval between Eventually retries is 10ms. Use 1 second to clean up the test logs like these: ... 15:31:27 23:31:26 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:28 23:31:28 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready 15:31:28 23:31:28 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready ... Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
test-me-please |
Please see #14384. Is there any benefit to printing the message in the first place? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with @pchaigno. These msgs aren't of any value. We can still print a msg when it fails. Either way, we can either do it as a followup or in this PR.
I've added labels for backports into v1.8 and v1.9 given this pull request fixes spam messages introduced by #14286, which was backported to those two branches. |
Gomega time interval between Eventually retries is 10ms. Use 1 second
to clean up the test logs like these:
...
15:31:27 23:31:26 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:27 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:28 23:31:27 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:28 23:31:28 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
15:31:28 23:31:28 STEP: Cilium DaemonSet not ready yet: only 1 of 2 desired pods are ready
...
Signed-off-by: Jarno Rajahalme jarno@covalent.io