WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 239341
Enable --treat-failing-as-flaky for the JSC post-commit bots
https://bugs.webkit.org/show_bug.cgi?id=239341
Summary
Enable --treat-failing-as-flaky for the JSC post-commit bots
Angelos Oikonomopoulos
Reported
2022-04-14 09:15:37 PDT
Enable --treat-failing-as-flaky for the JSC post-commit bots
Attachments
Patch
(1.77 KB, patch)
2022-04-14 09:16 PDT
,
Angelos Oikonomopoulos
no flags
Details
Formatted Diff
Diff
Patch
(4.63 KB, patch)
2022-04-14 09:40 PDT
,
Angelos Oikonomopoulos
no flags
Details
Formatted Diff
Diff
Patch
(4.14 KB, patch)
2022-04-25 05:25 PDT
,
Angelos Oikonomopoulos
no flags
Details
Formatted Diff
Diff
Patch
(5.03 KB, patch)
2022-04-25 05:28 PDT
,
Angelos Oikonomopoulos
no flags
Details
Formatted Diff
Diff
Show Obsolete
(3)
View All
Add attachment
proposed patch, testcase, etc.
Angelos Oikonomopoulos
Comment 1
2022-04-14 09:16:43 PDT
Created
attachment 457634
[details]
Patch
Angelos Oikonomopoulos
Comment 2
2022-04-14 09:40:41 PDT
Created
attachment 457638
[details]
Patch
Ryan Haddad
Comment 3
2022-04-21 15:41:57 PDT
rdar://92099399
Angelos Oikonomopoulos
Comment 4
2022-04-25 05:25:18 PDT
Created
attachment 458261
[details]
Patch
Angelos Oikonomopoulos
Comment 5
2022-04-25 05:28:11 PDT
Created
attachment 458262
[details]
Patch
EWS
Comment 6
2022-04-26 09:57:21 PDT
Committed
r293430
(
249990@main
): <
https://commits.webkit.org/249990@main
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 458262
[details]
.
Angelos Oikonomopoulos
Comment 7
2022-04-28 05:17:12 PDT
This has been picked up by the build.webkit.org after yesterday's restart, and there's already been a run with a flaky test:
https://build.webkit.org/#/builders/31/builds/3567
This is correctly captured in the per-test results: ❯ curl -sL '
http://results.webkit.org/api/results/javascriptcore-tests/stress/new-largeish-contiguous-array-with-size.js.ram-size-10000000?architecture=mips&uuid=165109908000
' | jq . [ { "configuration": { "architecture": "mips", "is_simulator": false, "platform": "linux", "style": "release", "version": 4019000 }, "results": [ { "actual": "PASS", "expected": "PASS", "flakiness_num_passes": 7, "flakiness_num_tries": 8, "start_time": 1651099268, "uuid": 165109908000 } ] } ] However, when I look at the "suite" results, I get: ❯ curl -sL '
http://results.webkit.org/api/results/javascriptcore-tests?architecture=mips&limit=40
' | jq -C . | grep build-number | grep 356 "build-number": "3560", "build-number": "3561", "build-number": "3563", "build-number": "3564", "build-number": "3568", "build-number": "3569", I.e. the result for the build number (3567) is missing. That said, I see that e.g.
https://build.webkit.org/#/builders/31/builds/3565
, which didn't have any flaky tests, is also missing, so I'm not sure what to think. FWIW I've locally verified again that when I submit a report with the current ToT (+ some flakiness injection) my local resultsdb can return the result just fine.
Jonathan Bedard
Comment 8
2022-04-28 07:41:30 PDT
This feels like we dropped processing the build, but I would need to investigate to verify that. Since we have an example, I can put this in the staging instance to check.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug