Skip to content

Require integration gates and streamline the CLN update workflow - #5

Closed
r1ckstardev wants to merge 3 commits into
btcpayserver:btcpaymasterfrom
r1ckstardev:jenny/cln-update-integration-tests
Closed

r1ckstardev wants to merge 3 commits into
btcpayserver:btcpaymasterfrom
r1ckstardev:jenny/cln-update-integration-tests

Conversation

@r1ckstardev

@r1ckstardev r1ckstardev commented Nov 4, 2026 •

Copy link
Copy Markdown

The CLN release procedure omitted the test-image PRs and integration gates in BTCPayServer.Lightning and BTCPay Server. Require both current-head integration runs against the published image before the deployment bump, and link the LND guide while keeping LND's startup suite distinct.

Consolidate the procedure from 196 lines on the base branch to 130 while adding those gates. Preserve linear source history and explicit-lease publication, verify all platform attestations, distinguish regtest coverage from other Compose environments, and replace contradictory wrong-tag deletion instructions with publication-aware recovery. Delegate generated Docker files to that repository's authoritative instructions.

Checked skill frontmatter, local link targets, downstream paths/services and the diff. Current-head CI on this documentation-only PR stops on pre-existing ShellCheck SC1090 in unchanged tools/docker-entrypoint.sh lines 93 and 114 (run).

Changelog-None

@r1ckstardev
r1ckstardev force-pushed the jenny/cln-update-integration-tests branch from 0b4935a to 54cd016 Compare November 4, 2026 15:38
r1ckstardev pushed a commit that referenced this pull request Nov 4, 2026
Three bare 100s (the BOLT #5 quote above the first one explains what
they are).  Taken from Rusty's branch.

Changelog-None

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019XZZWMuBhXrq41u9HhxVBM

(cherry picked from commit 132eefabe541c8bce2a2c10c80e1979650ec12ca)
@r1ckstardev r1ckstardev changed the title Require BTCPay integration tests for Core Lightning updates Require integration gates and streamline the CLN update workflow Nov 4, 2026
@NicolasDorier

Copy link
Copy Markdown
Member

The shared maintenance repository is now merged in btcpayserver/image-maintenance#1, and #6 moved the fork instructions there. The applicable integration, provenance and recovery corrections proposed here were incorporated into https://github.com/btcpayserver/image-maintenance/blob/master/docs/lightning.md and the shared conventions. Further procedure changes should target that repository rather than restore fork-local agent instructions.

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