Skip to content

Fix idle snap timeout for unused snap#2010

Merged
FrederikBolding merged 1 commit intomainfrom
fb/fix-idle-snap-timeout-for-unused-snap
Dec 4, 2023
Merged

Fix idle snap timeout for unused snap#2010
FrederikBolding merged 1 commit intomainfrom
fb/fix-idle-snap-timeout-for-unused-snap

Conversation

@FrederikBolding
Copy link
Copy Markdown
Member

Fixes an issue where a snap that never had an incoming RPC request would never be torn down by the idle timeout.

@FrederikBolding FrederikBolding requested a review from a team as a code owner December 4, 2023 13:32
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 4, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (59da0ea) 96.08% compared to head (da5c01d) 96.08%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2010   +/-   ##
=======================================
  Coverage   96.08%   96.08%           
=======================================
  Files         269      269           
  Lines        6259     6261    +2     
  Branches     1009     1009           
=======================================
+ Hits         6014     6016    +2     
  Misses        245      245           

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

@FrederikBolding FrederikBolding force-pushed the fb/fix-idle-snap-timeout-for-unused-snap branch from eae95ab to da5c01d Compare December 4, 2023 13:48
@FrederikBolding FrederikBolding merged commit 390d788 into main Dec 4, 2023
@FrederikBolding FrederikBolding deleted the fb/fix-idle-snap-timeout-for-unused-snap branch December 4, 2023 14: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.

2 participants