Update flaky test issue template (#16296)

This commit is contained in:
Isaiah Becker-Mayer
2022-09-12 22:20:50 +00:00
committed by GitHub
parent a1cfea0e36
commit 122e3abe6d
+10 -39
View File
@@ -1,17 +1,17 @@
---
name: Flaky Test ❄
about: Report a flaky unit or integration test
title: '`TestName` flakiness'
title: "`TestName` flakiness"
labels: flaky tests
---
## Before submitting a new issue
Before filing a new issue, search https://github.com/gravitational/teleport/issues?q=is%3Aissue+is%3Aopen+label%3A%22flaky+tests%22 for
the flaky test's name to see if the same issue has already been reported. If so, double check the `Relevant snippet` to confirm that
the specific failure is the same. If it is, please react 👍 to this issue (so we can sort issues by reactions). If you have a link to the
full log of the test run, please add it to the `Link(s) to logs` section below. If instead you've encountered a new failure, edit the issue
and copy paste in a new `Failure` section below the existing one(s).
Before filing a new issue, search https://github.com/gravitational/teleport/issues?q=is%3Aissue+label%3A%22flaky+tests%22+ for
the flaky test's name to see if the same issue has already been reported.
If so, double check the `Relevant snippet` to confirm that the specific failure is the same. If it is, please react 👍 to this issue (so we can
sort issues by reactions). Add a comment with the `Link(s) to logs` url (see section below). If you've encountered a new failure for the same test, also copy-paste the `Relevant snippet` (see section below).
If there's no existing issue for this test, continue to file a new one:
@@ -30,28 +30,9 @@ If there's no existing issue for this test, continue to file a new one:
**replace everything below here with your snippet**
Be extra helpful by adding the relevant snippet of the logs that show where the failure takes place.
See the examples below.
##### `check` style test, ctrl-F for "FAIL:"
```
FAIL: services_test.go:159: ServicesSuite.TestSemaphoreContention
/workspace/lib/services/suite/suite.go:1222:
c.Assert(err, check.IsNil)
... value *trace.TraceErr =
ERROR REPORT:
Original Error: *trace.LimitExceededError too much contention on semaphore connection/alice
Stack Trace:
/workspace/lib/services/local/presence.go:760 github.com/gravitational/teleport/lib/services/local.(*PresenceService).AcquireSemaphore
/workspace/lib/services/semaphore.go:245 github.com/gravitational/teleport/lib/services.AcquireSemaphoreLock
/workspace/lib/services/suite/suite.go:1221 github.com/gravitational/teleport/lib/services/suite.(*ServicesTestSuite).SemaphoreContention.func1
/opt/go/src/runtime/asm_amd64.s:1571 runtime.goexit
User Message: too much contention on semaphore connection/alice ("too much contention on semaphore connection/alice")
```
##### `testing` style test, ctrl-F for "Test:"
Be extra helpful by adding the relevant snippet of the logs that show where the failure takes place. Typically ctrl+F searching for "fail:" and/
or "test:" will bring you to the right spot. The example below shows what a typical failure looks like (though sometimes no such log will exist,
for example in the case of a timeout).
```
tsh_test.go:667:
@@ -59,14 +40,4 @@ User Message: too much contention on semaphore connection/alice ("too much conte
Error: Received unexpected error:
exit code 1
Test: TestSSHAccessRequest
```
## Update the Flaky Test Tracker meta-issue
After submitting this issue, add a comment to our `Flaky Test Tracker` issue (https://github.com/gravitational/teleport/issues/9492) in the format:
`TestName`: https://github.com/gravitational/teleport/issues/<issue-number>
See https://github.com/gravitational/teleport/issues/9492#issuecomment-1158937319 for an example.
**Delete this section, submit the issue, and add the comment**
```