Skip to content
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

More flake fixes. #7866

Merged
merged 4 commits into from
Nov 26, 2024
Merged

Conversation

rustyrussell
Copy link
Contributor

@rustyrussell rustyrussell commented Nov 25, 2024

Well, so far one xpay - askrene layer fix, and one "mfc thinks every multifundchannel has id 34" fix.

Changelog-None

@rustyrussell rustyrussell added the flake Known CI flakes label Nov 25, 2024
@rustyrussell rustyrussell added this to the v24.11 milestone Nov 25, 2024
They're not always 34 (aka '"').  This is a side-effect of ids
changing from u64 to strings quite a while ago.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Create lower-level versions of routines to create biases, layers,
constraints, etc and only save the ones called from the public APIs.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Changelog-None: persistent layers were only added in this release
No code changes.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
…loading.

Using jsonrpc_request_sync, layers are loaded before we finish init,
so we never can be asked to create a layer before we've loaded it
(xpay creates a layer immediately on startup).

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Changelog-None: persistent layers new this release.
@rustyrussell rustyrussell merged commit fba738b into ElementsProject:master Nov 26, 2024
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flake Known CI flakes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant