FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ci: Raise `update-tox` timeout to 20 minutes by alexander-alderman-webb · Pull Request #6488 · getsentry/sentry-python · GitHub

ci: Raise update-tox timeout to 20 minutes - #6488

Merged
alexander-alderman-webb merged 1 commit into
masterfrom
webb/update-toxgen-timeout
Jun 3, 2026
Merged

ci: Raise update-tox timeout to 20 minutes#6488
alexander-alderman-webb merged 1 commit into
masterfrom
webb/update-toxgen-timeout

Conversation

Copy link
Copy Markdown
Contributor

Description

A run with an empty cache takes 14 minutes in getsentry/toxgen-testbed#1

Issues

Reminders

alexander-alderman-webb requested a review from a team as a code owner June 3, 2026 08:14

Copy link
Copy Markdown
Contributor

In this repo the caches should be populated, why is it taking so long? 👀

github-actions Bot commented Jun 3, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Codecov Results 📊

88635 passed | ⏭️ 6007 skipped | Total: 94642 | Pass Rate: 93.65% | Execution Time: 293m 37s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +283
Passed Tests 📈 +283
Failed Tests
Skipped Tests

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2332 uncovered lines.
✅ Project coverage is 89.91%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.91%    89.91%        —%
==========================================
  Files          191       191         —
  Lines        23115     23115         —
  Branches      7952      7952         —
==========================================
+ Hits         20783     20783         —
- Misses        2332      2332         —
- Partials      1309      1309         —

Generated by Codecov Action

Copy link
Copy Markdown
Contributor Author

No it's quicker in this repo, but 14 minutes is the current upper bound if all cache entries are missed.
This doesn't block the update at the moment but I want to avoid the weekly update failing in the future 😬.

alexander-alderman-webb merged commit a9aaff4 into master Jun 3, 2026
270 of 272 checks passed
alexander-alderman-webb deleted the webb/update-toxgen-timeout branch June 3, 2026 09:16
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL