Bug 239341 - Enable --treat-failing-as-flaky for the JSC post-commit bots
Summary: Enable --treat-failing-as-flaky for the JSC post-commit bots
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Angelos Oikonomopoulos
URL:
Keywords: InRadar
Depends on:
Blocks: 239342
  Show dependency treegraph
 
Reported: 2022-04-14 09:15 PDT by Angelos Oikonomopoulos
Modified: 2022-04-28 07:41 PDT (History)
4 users (show)

See Also:


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

Note You need to log in before you can comment on or make changes to this bug.
Description Angelos Oikonomopoulos 2022-04-14 09:15:37 PDT
Enable --treat-failing-as-flaky for the JSC post-commit bots
Comment 1 Angelos Oikonomopoulos 2022-04-14 09:16:43 PDT
Created attachment 457634 [details]
Patch
Comment 2 Angelos Oikonomopoulos 2022-04-14 09:40:41 PDT
Created attachment 457638 [details]
Patch
Comment 3 Ryan Haddad 2022-04-21 15:41:57 PDT
rdar://92099399
Comment 4 Angelos Oikonomopoulos 2022-04-25 05:25:18 PDT
Created attachment 458261 [details]
Patch
Comment 5 Angelos Oikonomopoulos 2022-04-25 05:28:11 PDT
Created attachment 458262 [details]
Patch
Comment 6 EWS 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].
Comment 7 Angelos Oikonomopoulos 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.
Comment 8 Jonathan Bedard 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.