Skip to content

Commit

Permalink
dashboard/app: split Reported-and-tested-by tag
Browse files Browse the repository at this point in the history
Kernel community asks to split it for tooling purposes:
https://lore.kernel.org/all/[email protected]/
  • Loading branch information
dvyukov committed Jul 17, 2024
1 parent 215bec2 commit 8b5d608
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 5 deletions.
10 changes: 6 additions & 4 deletions dashboard/app/jobs_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -258,7 +258,8 @@ patch: %[3]v
syzbot has tested the proposed patch and the reproducer did not trigger any issue:
Reported-and-tested-by: syzbot+%[email protected]
Reported-by: syzbot+%[email protected]
Tested-by: syzbot+%[email protected]
Tested on:
Expand All @@ -271,7 +272,7 @@ compiler: compiler1
patch: %[2]v
Note: testing is done by a robot and is best-effort only.
`, extBugID, patchLink, kernelConfigLink, logLink))
`, extBugID, extBugID, patchLink, kernelConfigLink, logLink))
c.checkURLContents(patchLink, []byte(sampleGitPatch))
c.checkURLContents(kernelConfigLink, build.KernelConfig)
}
Expand Down Expand Up @@ -372,7 +373,8 @@ func TestJobWithoutPatch(t *testing.T) {
syzbot has tested the proposed patch and the reproducer did not trigger any issue:
Reported-and-tested-by: syzbot+%[email protected]
Reported-by: syzbot+%[email protected]
Tested-by: syzbot+%[email protected]
Tested on:
Expand All @@ -384,7 +386,7 @@ compiler: compiler2
Note: no patches were applied.
Note: testing is done by a robot and is best-effort only.
`, extBugID, kernelConfigLink))
`, extBugID, extBugID, kernelConfigLink))
c.checkURLContents(kernelConfigLink, testBuild.KernelConfig)
}

Expand Down
7 changes: 6 additions & 1 deletion dashboard/app/templates/mail_test_result.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,12 @@ Error text is too large and was truncated, full error text is at:
{{else}}
syzbot has tested the proposed patch and the reproducer did not trigger any issue:

Reported-and-tested-by: {{.CreditEmail}}
{{/* Note: kernel prefers separate reported/tested tags rather than a single
Reported-and-tested-by, see:
https://lore.kernel.org/all/[email protected]/
*/ -}}
Reported-by: {{.CreditEmail}}
Tested-by: {{.CreditEmail}}
{{end}}
Tested on:

Expand Down

0 comments on commit 8b5d608

Please sign in to comment.