-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
tstest/integration: TestAddPingRequest flakes #2079
Labels
T7 Build/test failure
Issue type
Comments
bradfitz
added a commit
that referenced
this issue
Jun 10, 2021
This fails pretty reliably with a lot of output now showing what's happening: TS_DEBUG_MAP=1 go test --failfast -v -run=Ping -race -count=20 ./tstest/integration --verbose-tailscaled I haven't dug into the details yet, though. Updates #2079
DentonGentry
added a commit
that referenced
this issue
Jun 14, 2021
Failing often now, we don't want people to get used to routinely ignoring test failures. Can be re-enabled when #2079 is resolved. Signed-off-by: Denton Gentry <dgentry@tailscale.com>
DentonGentry
added a commit
that referenced
this issue
Jun 15, 2021
Failing often now, we don't want people to get used to routinely ignoring test failures. Can be re-enabled when #2079 is resolved. Signed-off-by: Denton Gentry <dgentry@tailscale.com>
DentonGentry
added a commit
that referenced
this issue
Jun 15, 2021
Failing often now, we don't want people to get used to routinely ignoring test failures. Can be re-enabled when #2079 is resolved. Signed-off-by: Denton Gentry <dgentry@tailscale.com>
bradfitz
added a commit
that referenced
this issue
Jun 15, 2021
… is unique Updates #2079 Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The new
TestAddPingRequest
flakes sometimes, e.g. with:@simenghe, can you take a look?
The text was updated successfully, but these errors were encountered: