Skip to content
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

Flaky test: should enable pods to ping each other via hostname #135

Closed
Tracked by #145
tenzen-y opened this issue May 10, 2023 · 3 comments · Fixed by #153
Closed
Tracked by #145

Flaky test: should enable pods to ping each other via hostname #135

tenzen-y opened this issue May 10, 2023 · 3 comments · Fixed by #153

Comments

@tenzen-y
Copy link
Member

JobSet when dns hostnames is enabled [It] should enable pods to ping each other via hostname
/home/prow/go/src/sigs.k8s.io/jobset/test/e2e/e2e_test.go:63
  [FAILED] Timed out after 120.001s.
  Expected
      <bool>: false
  to equal
      <bool>: true
  In [It] at: /home/prow/go/src/sigs.k8s.io/jobset/test/util/util.go:61 @ 05/10/23 16:26:41.124

https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_jobset/123/pull-jobset-test-e2e-main-1-24/1656333510748672000

@tenzen-y
Copy link
Member Author

ref: #123

@danielvegamyhre
Copy link
Contributor

danielvegamyhre commented May 10, 2023

We should consider increasing the timeout from 2min to 5min perhaps, and/or adding a longer sleep at the end the end of the bash script here so the final pod to become ready has time to ping the other pods successfully before they complete.

@tenzen-y
Copy link
Member Author

We should consider increasing the timeout from 2min to 5min perhaps, and/or adding a longer sleep at the end the end of the bash script here so the final pod to become ready has time to ping the other pods successfully before they complete.

Agree. I think this flaky test isn't caused by JobSet. Maybe, there are root causes in Headress Service (k/k).

cc: @ahg-g

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants