vincenzopalazzo
/
rgb-cln
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

lightningd: don't timeout plugins if init is slow! 

This is a minimal fix: we wait until all plugins reply from init before
continuing.  Really large or busy nodes can have other things monopolize
lightningd, then the timer goes off and we blame the plugin (which has
responded, we just haven't read it yet!).

The real answer is to have some timeouts only advance when we're idle,
or have them low-priority so we only activate them when we're idle (this
doesn't apply to all timers: some are probably important!).  But
this is a minimal fix for -rc3.

Fixes: https://github.com/ElementsProject/lightning/issues/5736
Changelog-Fixed: plugins: on large/slow nodes we could blame plugins for failing to answer init in time, when we were just slow.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
This commit is contained in:
Rusty Russell 2022-11-26 12:59:53 +10:30 committed by Christian Decker
parent 3d311c96b1
commit 626998efce
2 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lightningd/plugin.c
View File

@ -180,6 +180,9 @@ static void check_plugins_initted(struct plugins *plugins)
for (size_t i = 0; i < tal_count(plugin_cmds); i++)
plugin_cmd_all_complete(plugins, plugin_cmds[i]);
tal_free(plugin_cmds);
if (plugins->startup)
io_break(plugins);
}
struct command_result *plugin_register_all_complete(struct lightningd *ld,
@ -1943,6 +1946,11 @@ void plugins_config(struct plugins *plugins)
plugin_config(p);
}
/* Wait for them to configure, before continuing: large
* nodes can take a while to startup! */
if (plugins->startup)
io_loop_with_timers(plugins->ld);
plugins->startup = false;
}

2
tests/test_plugin.py
View File

@ -1942,6 +1942,8 @@ def test_plugin_fail(node_factory):
time.sleep(2)
# It should clean up!
assert 'failcmd' not in [h['command'] for h in l1.rpc.help()['help']]
# Can happen *before* the 'Server started with public key'
l1.daemon.logsearch_start = 0
l1.daemon.wait_for_log(r': exited during normal operation')
l1.rpc.plugin_start(plugin)