Skip to content

Fix stale gossip retention Heisenbug when channel updates are slow. #1953

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

arik-so
Copy link
Contributor

@arik-so arik-so commented Jan 14, 2023

Debugs and fixes #1914.

@arik-so arik-so force-pushed the 2023-01-debug-gossip-heisenbug branch from 603922c to e9d4ae1 Compare January 14, 2023 15:28
@arik-so arik-so marked this pull request as ready for review January 14, 2023 15:29
@codecov-commenter
Copy link

codecov-commenter commented Jan 14, 2023

Codecov Report

Base: 90.73% // Head: 90.71% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (16deda0) compared to base (ac6e0b3).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1953      +/-   ##
==========================================
- Coverage   90.73%   90.71%   -0.02%     
==========================================
  Files          97       97              
  Lines       50548    50551       +3     
  Branches    50548    50551       +3     
==========================================
- Hits        45864    45857       -7     
- Misses       4684     4694      +10     
Impacted Files Coverage Δ
lightning/src/routing/gossip.rs 92.14% <100.00%> (+0.01%) ⬆️
lightning/src/ln/functional_tests.rs 96.93% <0.00%> (-0.13%) ⬇️
lightning/src/ln/monitor_tests.rs 99.45% <0.00%> (-0.11%) ⬇️
lightning/src/chain/channelmonitor.rs 90.99% <0.00%> (-0.10%) ⬇️
lightning/src/chain/onchaintx.rs 95.39% <0.00%> (+0.20%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@arik-so arik-so changed the title Debug #1914. Fix stale gossip retention Heisenbug when channel updates are slow. Jan 14, 2023
@arik-so arik-so added this to the 0.0.114 milestone Jan 14, 2023
@TheBlueMatt TheBlueMatt merged commit 1e81287 into lightningdevkit:main Jan 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

routing::gossip::tests::test_channel_timeouts is flaky
4 participants