Skip to content

[worker] Keep legacy cache failures non-fatal - #4295

Open
sjchmiela wants to merge 1 commit into
mainfrom
stanley/cache-manager-error-reporting
Open

[worker] Keep legacy cache failures non-fatal#4295
sjchmiela wants to merge 1 commit into
mainfrom
stanley/cache-manager-error-reporting

Conversation

@sjchmiela

@sjchmiela sjchmiela commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Add Sentry logging to legacy build cache.

@sjchmiela
sjchmiela force-pushed the stanley/cache-manager-error-reporting branch from cfea81d to 1f4aa4f Compare August 27, 2026 13:56
@sjchmiela
sjchmiela requested a lite review from Copilot August 27, 2026 13:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.35%. Comparing base (b287d7a) to head (9635507).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4295      +/-   ##
==========================================
+ Coverage   64.34%   64.35%   +0.02%     
==========================================
  Files        1047     1047              
  Lines       48324    48325       +1     
  Branches    10191    10191              
==========================================
+ Hits        31087    31093       +6     
+ Misses      17121    17116       -5     
  Partials      116      116              

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

🚀 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.

@sjchmiela
sjchmiela force-pushed the stanley/cache-manager-error-reporting branch from 1f4aa4f to 9635507 Compare August 27, 2026 14:07
@sjchmiela
sjchmiela requested a review from AbbanMustafa August 27, 2026 14:07
@sjchmiela sjchmiela added the no changelog PR that doesn't require a changelog entry label Aug 27, 2026
@github-actions

Copy link
Copy Markdown

⏩ The changelog entry check has been skipped since the "no changelog" label is present.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog PR that doesn't require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants