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

Disable payment channel manager by default #12718

Open
rvagg opened this issue Nov 25, 2024 · 3 comments
Open

Disable payment channel manager by default #12718

rvagg opened this issue Nov 25, 2024 · 3 comments
Labels
good first issue Good for newcomers

Comments

@rvagg
Copy link
Member

rvagg commented Nov 25, 2024

On node startup, even for a light node, the payment channel manager starts up and watches the chain for paych-related messages to react to. Payment channels get minimal (to no) use currently on mainnet so we should be safe to introduce a configuration option to disable the payment channel manager and turn it on by default.

See

lotus/node/builder_chain.go

Lines 120 to 126 in 0d1d7c2

// Service: Payment channels
Override(new(paychmgr.PaychAPI), From(new(modules.PaychAPI))),
Override(new(*paychmgr.Store), modules.NewPaychStore),
Override(new(*paychmgr.Manager), modules.NewManager),
Override(HandlePaymentChannelManagerKey, modules.HandlePaychManager),
Override(SettlePaymentChannelsKey, settler.SettlePaymentChannels),

@rvagg rvagg added the good first issue Good for newcomers label Nov 25, 2024
@github-project-automation github-project-automation bot moved this to 📌 Triage in FilOz Nov 25, 2024
@f8-ptrk
Copy link
Contributor

f8-ptrk commented Nov 25, 2024

you mean turn it off by default?

[edit]

its not quite clear what you want as an outcome - it seems its already enabled by default, is it? turning it off by default and intro an option to enable it would make sense

@rvagg
Copy link
Member Author

rvagg commented Nov 25, 2024

Yeah, sorry, bad wording - turn it off by default, stop it humming in the background.

A few of us are also discussing how we might trim the code down around the paych functionality, removing most, or even all of it from Lotus.

@f8-ptrk
Copy link
Contributor

f8-ptrk commented Nov 28, 2024

removing it from boost was detrimental to the project. what was left of the existing storage market on filecoin got killed. even years later there is no more build in way to pay for retrieval. its a feature that once worked and i am not sure what the reason was/is to remove it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Status: 📌 Triage
Development

No branches or pull requests

2 participants