Skip to content

feat: Allow Session Replay to retry payloads#1666

Merged
metal-messiah merged 61 commits into
mainfrom
sr-retry
Apr 30, 2026
Merged

feat: Allow Session Replay to retry payloads#1666
metal-messiah merged 61 commits into
mainfrom
sr-retry

Conversation

@metal-messiah

@metal-messiah metal-messiah commented Jan 2, 2026

Copy link
Copy Markdown
Member

Payloads failed with retryable codes will be re-harvested by the agent. Further session replay data will not be collected until the harvest retries successfully.

Overview

Related Issue(s)

https://new-relic.atlassian.net/browse/NR-489361

Testing

New tests have been added.

@github-actions

github-actions Bot commented Jan 2, 2026

Copy link
Copy Markdown

Supportability Metric Usage Changes:

No matching changes found

Supportability Metrics .md File Changes:

supportability_metrics.md was changed? false

New supportability metrics require changes to supportability_metrics.md, as well as a new PR to Angler. Please ensure an Angler PR is created and linked to this PR.

@github-actions

github-actions Bot commented Jan 2, 2026

Copy link
Copy Markdown

Asset Size Report

Merging this pull request will result in the following asset size changes:

Agent Asset Previous Size New Size Diff
lite loader 26.55 kB / 10.06 kB (gzip) 26.55 kB / 10.06 kB (gzip) 0% / 0% (gzip)
lite async-chunk 55.07 kB / 18.52 kB (gzip) 55.07 kB / 18.52 kB (gzip) 0% / 0% (gzip)
pro loader 62.4 kB / 21.93 kB (gzip) 62.4 kB / 21.93 kB (gzip) 0% / 0% (gzip)
pro async-chunk 91.24 kB / 28.92 kB (gzip) 91.44 kB / 28.96 kB (gzip) 0.22% / 0.13% (gzip)
spa loader 64.8 kB / 22.73 kB (gzip) 64.8 kB / 22.73 kB (gzip) 0% / 0% (gzip)
spa async-chunk 102.46 kB / 31.77 kB (gzip) 102.66 kB / 31.81 kB (gzip) 0.2% / 0.11% (gzip)

@codecov

codecov Bot commented Jan 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.82%. Comparing base (d9f6711) to head (94183f8).
⚠️ Report is 32 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1666      +/-   ##
==========================================
+ Coverage   88.76%   88.82%   +0.06%     
==========================================
  Files         209      209              
  Lines        7536     7546      +10     
  Branches     1557     1561       +4     
==========================================
+ Hits         6689     6703      +14     
+ Misses        754      751       -3     
+ Partials       93       92       -1     
Flag Coverage Δ
unit-tests 82.72% <100.00%> (+0.13%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

@github-actions

github-actions Bot commented Jan 2, 2026

Copy link
Copy Markdown

Static Badge

Last ran on April 28, 2026 18:27:07 CDT
Checking merge of (94183f8) into main (d9f6711)

@cwli24

cwli24 commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Is there a ticket for this?

@metal-messiah

Copy link
Copy Markdown
Member Author

Is there a ticket for this?

@cwli24 sorry just appended it to the PR. It's https://new-relic.atlassian.net/browse/NR-489361

Comment thread src/features/session_replay/aggregate/index.js Outdated
Comment thread src/features/session_replay/aggregate/index.js Outdated
Comment thread src/features/session_replay/constants.js Outdated
Comment thread src/features/session_replay/aggregate/index.js
@metal-messiah metal-messiah merged commit 9874ac5 into main Apr 30, 2026
33 of 34 checks passed
@metal-messiah metal-messiah deleted the sr-retry branch April 30, 2026 15:04
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.

6 participants