Skip to content

splice: Take splice out of experimental mode!#9021

Open
ddustin wants to merge 4 commits intoElementsProject:masterfrom
ddustin:ddustin/splicing-release
Open

splice: Take splice out of experimental mode!#9021
ddustin wants to merge 4 commits intoElementsProject:masterfrom
ddustin:ddustin/splicing-release

Conversation

@ddustin
Copy link
Copy Markdown
Collaborator

@ddustin ddustin commented Apr 2, 2026

Turn on splicing by default and remove the experimental splicing feature!

@ddustin ddustin requested a review from cdecker as a code owner April 2, 2026 02:19
@ddustin ddustin force-pushed the ddustin/splicing-release branch 2 times, most recently from a38d635 to c170ea0 Compare April 2, 2026 02:24
@ddustin ddustin added this to the v26.04 milestone Apr 2, 2026
@ddustin ddustin force-pushed the ddustin/splicing-release branch 2 times, most recently from 53a144f to 119e87a Compare April 2, 2026 03:00
Fix leaking of the `abort_pkg` if our peer does not support splicing.
@ddustin ddustin force-pushed the ddustin/splicing-release branch 3 times, most recently from 9f31c41 to 878f6f3 Compare April 2, 2026 15:26
@ddustin ddustin requested review from rustyrussell April 2, 2026 19:18
@rustyrussell
Copy link
Copy Markdown
Contributor

It's good, but a bit overzealous. We need to silently accept it and run a full deprecation period. This avoids breaking people's configs when they upgrade (and by the time we do remove it, in 12 months, they don't have to worry about downgrading).

See this example:

static char *opt_set_hsm_password(struct lightningd *ld)
{
	int is_encrypted;

	/* Show deprecation warning */
	if (!opt_deprecated_ok(ld, "encrypted_hsm",
			      "Use --hsm-passphrase instead",
			      "v25.12", "v26.12"))
		return "--encrypted-hsm was removed, use --hsm-passphrase instead";

Then update doc/developers-guide/deprecations.md, and Changelog-Deprecated it.

Turn splicing on by default for channel feature sets!

Changelog-Deprecated Splicing is now enabled by default!
@ddustin ddustin force-pushed the ddustin/splicing-release branch from d9980c2 to dadedd3 Compare April 7, 2026 13:52
@ddustin
Copy link
Copy Markdown
Collaborator Author

ddustin commented Apr 7, 2026

It's good, but a bit overzealous. We need to silently accept it and run a full deprecation period. This avoids breaking people's configs when they upgrade (and by the time we do remove it, in 12 months, they don't have to worry about downgrading).

See this example:

static char *opt_set_hsm_password(struct lightningd *ld)
{
	int is_encrypted;

	/* Show deprecation warning */
	if (!opt_deprecated_ok(ld, "encrypted_hsm",
			      "Use --hsm-passphrase instead",
			      "v25.12", "v26.12"))
		return "--encrypted-hsm was removed, use --hsm-passphrase instead";

Then update doc/developers-guide/deprecations.md, and Changelog-Deprecated it.

Okay I think I got all the deprecation steps in. Cool system!

Added the command back in with the deprecation warning, adjusted deprecated-features.md and changed the commit message to say Changelog-Deprecated

@ddustin ddustin force-pushed the ddustin/splicing-release branch 7 times, most recently from bb440fd to 5e62761 Compare April 7, 2026 15:08
@daywalker90 daywalker90 force-pushed the ddustin/splicing-release branch from 5e62761 to 92bc86e Compare April 7, 2026 15:29
@ddustin ddustin force-pushed the ddustin/splicing-release branch 3 times, most recently from e5c2de7 to c406406 Compare April 7, 2026 17:37
@daywalker90 daywalker90 force-pushed the ddustin/splicing-release branch from c406406 to 6e0022f Compare April 7, 2026 17:53
When the doc generator fails, make the CI output the diff of how it failed.
@daywalker90 daywalker90 force-pushed the ddustin/splicing-release branch from 6e0022f to 4712f33 Compare April 7, 2026 18:29
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.

3 participants