Skip to content

askrene: added reservations leak test under load#3

Closed
ScuttoZ wants to merge 4 commits intoLagrang3:reservation-leakfrom
ScuttoZ:test-pr8681
Closed

askrene: added reservations leak test under load#3
ScuttoZ wants to merge 4 commits intoLagrang3:reservation-leakfrom
ScuttoZ:test-pr8681

Conversation

@ScuttoZ
Copy link

@ScuttoZ ScuttoZ commented Mar 12, 2026

Important

25.09 FREEZE July 28TH: Non-bugfix PRs not ready by this date will wait for 25.12.

RC1 is scheduled on August 11th

The final release is scheduled for September 1st.

Checklist

Before submitting the PR, ensure the following tasks are completed. If an item is not applicable to your PR, please mark it as checked:

  • The changelog has been updated in the relevant commit(s) according to the guidelines.
  • Tests have been added or modified to reflect the changes.
  • Documentation has been reviewed and updated as needed.
  • Related issues have been listed and linked, including any that this PR closes.

Lagrang3 and others added 4 commits February 23, 2026 14:09
Log when a reservation removal failures during getroutes computation.
Failed reservation removals can lead to reservation leaks.

Changelog-None

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>
Changelog-None.

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>
Signed-off-by: Lagrang3 <lagrang3@protonmail.com>
@Lagrang3 Lagrang3 force-pushed the reservation-leak branch 2 times, most recently from f4196e3 to 9fd1ed6 Compare March 17, 2026 13:10
@Lagrang3
Copy link
Owner

@ScuttoZ, PR 8681 was recently merged. You might like to make another PR into Core Lightning to include the test you wrote.

@Lagrang3 Lagrang3 closed this Mar 18, 2026
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