limit pytest version to <8.1

[skshetry]

Changes in pytest==8.1.0 breaks flaky plugin.

See box/flaky#198.

It's breaking the CI.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.22%. Comparing base (257ea5f) to head (d76efa6).

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10334      +/-   ##
==========================================
- Coverage   90.60%   90.22%   -0.39%     
==========================================
  Files         500      500              
  Lines       38591    38591              
  Branches     5573     5564       -9     
==========================================
- Hits        34965    34818     -147     
- Misses       2983     3081      +98     
- Partials      643      692      +49     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[shcheklein]

Will we detect when there is a fix available?

[skshetry]

The change is intentional in pytest, so it will likely have to be fixed in flaky. In that case, there will be updates in the above issue.

[shcheklein]

yep, but how will we know to bump it again? will there be a dependabot PR?

[skshetry]

I will revert this PR once the issue gets fixed.

And, yes there will be dependabot PR for newer pytest releases. But the problem is not with pytest, but with flaky.

[shcheklein]

But the problem is not with pytest, but with flaky.

yep, I got that part. Was to double check that we have a way to get back to it via some automated PR. Thanks @skshetry .

[The-Compiler]

FWIW, given that flaky hasn't seen a release since 2020 and nobody reacted to an issue asking if it was still active, this is unlikely to improve in the forseeable future. You might want to look at migrating to e.g. pytest-rerunfailures instead.

[skshetry]

Thanks @The-Compiler for the suggestion. I have replaced flaky with pytest-rerunfailures in #10335. 🙂