nrkno / sofie-core

Sofie Core: A Part of the Sofie TV Studio Automation System
https://github.com/nrkno/Sofie-TV-automation/
MIT License
124 stars 40 forks source link

fix: add a way to take a heap snapshot (patch for release49) #1165

Closed nytamin closed 3 months ago

nytamin commented 4 months ago

About the Contributor

This pull request is posted on behalf of the NRK.

Type of Contribution

This is a hack, to allow for memory leak troubleshooting.

New Behavior

This PR adds a GET method that returns a NodeJs heap snapshot, which can be inspected in the Chrome devtools.

Testing Instructions

  1. Go to Settings / Core System Settings
  2. At the bottom of the page, you find a new button under Memory troublshooting

Time Frame

This is urgent for us, please review and merge it as soon as possible.

Other Information

Status

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 0% with 67 lines in your changes are missing coverage. Please review.

Project coverage is 59.02%. Comparing base (a04c695) to head (2138abd). Report is 19 commits behind head on release49.

Files Patch % Lines
meteor/server/api/heapSnapshot.ts 0.00% 66 Missing :warning:
meteor/server/main.ts 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## release49 #1165 +/- ## ============================================= - Coverage 59.07% 59.02% -0.06% ============================================= Files 473 474 +1 Lines 71613 71697 +84 Branches 3724 3728 +4 ============================================= + Hits 42308 42320 +12 - Misses 29258 29332 +74 + Partials 47 45 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.