Repository navigation
fix(ci): Update test timeout - #23504
iamkirkbater wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Summary
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title clearly and concisely describes the timeout change in the CI test. |
| Description check | |
The description explains the timeout increase, its rationale, scope, testing impact, and planned CI validation. It includes the required template sections. Several checklist items remain unchecked, bu… |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
Full details: Description check
-
Explanation
The description explains the timeout increase, its rationale, scope, testing impact, and planned CI validation. It includes the required template sections. Several checklist items remain unchecked, but the description is otherwise sufficiently complete for this test-only change.
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
Comment @coderabbitai help to get the list of available commands.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #23504 +/- ##
==========================================
- Coverage 52.01% 51.96% -0.05%
==========================================
Files 2914 2914
Lines 183753 183753
==========================================
- Hits 95574 95496 -78
- Misses 79835 79892 +57
- Partials 8344 8365 +21
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
🚀 Build Images ReadyImages are ready for commit 90b3523. To use with deploy scripts: export MAIN_IMAGE_TAG=5.1.x-305-g90b3523429 |
Description
Updates the test timeout to align with other Scanner e2e test timeouts.
This job is consistently timing out, according to Claude it's persistently hitting exponential backoffs due to the architecture. Hopefully this gives some breathing room so that this isn't a test that consistently fails.
Additionally, according to claude any refactoring to make the logic itself work would require a rearchitecture of the code itself, which is largely out of scope here.
I'm happy to put that context into a followup ticket if desired.
User-facing documentation
Testing and quality
Automated testing
How I validated my change
Validate CI runs.