test: mark test-tls-min-max-version as flaky

[JonasBa]

Ref: #56746

[anonrig]

cc @nodejs/platform-smartos

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/64714/

[lpinca]

The failure is a timeout so it might be #54918 again.

[anonrig]

The failure is a timeout so it might be #54918 again.

@lpinca For education purposes: How did you know that it was a timeout? The error code is -15, which I couldn't find any information about online. Is it the duration that gave it away?

[lpinca]

duration_ms: 300088.347
exitcode: -15
severity: fail
stack: |-
  timeout
...

[mcollina]

lgtm

I think this is our deadlock problem :(.

[lpinca]

Is the test failing consistently or is it one off (timeout only)? In the latter case I would not mark it flaky or we will slowing mark every test flaky until a fix for #54918 is found.

[lpinca]

I could not find any previous failure of this test on https://github.com/nodejs/reliability so I think this is premature, but I don't want to start another fight :)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.21%. Comparing base (08eeddf) to head (1120f7f).
Report is 426 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #56747   +/-   ##
=======================================
  Coverage   89.21%   89.21%           
=======================================
  Files         662      662           
  Lines      191968   191968           
  Branches    36955    36948    -7     
=======================================
+ Hits       171269   171271    +2     
- Misses      13535    13542    +7     
+ Partials     7164     7155    -9     

see 39 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[mcollina]

blocking for now. Happy to revise if this show up further.