Skip to content

Test penalty transaction accounting from both the punisher and the cheater peer perspective#8860

Open
enaples wants to merge 4 commits intoElementsProject:masterfrom
enaples:test-penalty-accounting
Open

Test penalty transaction accounting from both the punisher and the cheater peer perspective#8860
enaples wants to merge 4 commits intoElementsProject:masterfrom
enaples:test-penalty-accounting

Conversation

@enaples
Copy link
Copy Markdown
Collaborator

@enaples enaples commented Jan 22, 2026

Important

26.04 FREEZE March 11th: Non-bugfix PRs not ready by this date will wait for 26.06.

RC1 is scheduled on March 23rd

The final release is scheduled for April 15th.

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:

  • [ x] The changelog has been updated in the relevant commit(s) according to the guidelines.
  • [ x] Tests have been added or modified to reflect the changes.
  • [ x] Documentation has been reviewed and updated as needed.
  • [ x] Related issues have been listed and linked, including any that this PR closes.
  • [ x] Important All PRs must consider how to reverse any persistent changes for tools/lightning-downgrade

Requested Info

It it correct that, from the bookkeeper perspective, the penalty transaction is recorded on the channel account and not on the wallet account?

Changelog-None

@enaples enaples added this to the v26.04 milestone Mar 13, 2026
@sangbida sangbida modified the milestones: v26.04, 26.06 Mar 16, 2026
@madelinevibes madelinevibes added the QA Blockstream QA team have reproduced, or a test has been created! Look for the linked PR/Issue label May 4, 2026
@cdecker cdecker force-pushed the test-penalty-accounting branch from 37639d4 to 33351a5 Compare May 4, 2026 10:12
@cdecker
Copy link
Copy Markdown
Member

cdecker commented May 4, 2026

Rebased on top of master to see if it passes.

@madelinevibes madelinevibes added Status::Ready for Review The work has been completed and is now awaiting evaluation or approval. PLEASE clear CI 🫠 labels May 4, 2026
@enaples enaples force-pushed the test-penalty-accounting branch from 33351a5 to 53342b1 Compare May 6, 2026 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PLEASE clear CI 🫠 QA Blockstream QA team have reproduced, or a test has been created! Look for the linked PR/Issue Status::Ready for Review The work has been completed and is now awaiting evaluation or approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants