-
Notifications
You must be signed in to change notification settings - Fork 912
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
plugin: rescan restarts plugin on update #4609
plugin: rescan restarts plugin on update #4609
Conversation
d89b1dd
to
d40abca
Compare
This comment has been minimized.
This comment has been minimized.
d40abca
to
0cb7b95
Compare
Just a test flake on tests/test_misc.py::test_funding_reorg_private |
0d441bd
to
13af6ae
Compare
This adds a `u32 checksum` field to the plugin struct that is used to identify if a plugin is outdated and needs to be restarted on `rescan`. Note: Only affects non-important plugins. Changelog-Added: Plugin: Restart plugin on `rescan` when binary was changed.
13af6ae
to
bdcf254
Compare
eaafd6e
to
657518b
Compare
657518b
to
1909b1f
Compare
Okay, same unrelated testflake. Other than that: done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ack 1909b1f
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ack 1909b1f
This adds a
u32 checksum
field to the plugin struct that is usedto identify if a plugin is outdated and needs to be restarted on
rescan
.Useful for developing and debugging plugins.
Note: only affects non-important plugins.
Changelog-Added: Plugin: Restarts a plugin on
rescan
when its binary was changed.