Skip to content
This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

Support for split market subsystem #61

Merged
merged 2 commits into from
Oct 13, 2021
Merged

Conversation

cerblue
Copy link
Contributor

@cerblue cerblue commented Oct 12, 2021

This change adds two new optional parameters to receive multiaddr and auth token to market process. If omitted, there is no change in behavior. If provided, all Filecoin API calls other than Filecoin.SectorsSummary is sent to market process endpoint. This resolves the problem of sealing-sectors-limit settings having no effect on systems with split-out market process because of the following error:

2021-10-12T23:02:24.325Z        ERROR        bidbot/service        service/service.go:313        fail to get number of sealing sectors, continuing: getting lotus sectors summary: fatal error calling 'Filecoin.SectorsSummary': panic in rpc method 'Filecoin.SectorsSummary': runtime error: invalid memory address or nil pointer dereference

This call fails because Filecoin.SectorsSummary API call is supported by the miner process.

@merlinran merlinran self-requested a review October 13, 2021 12:42
Copy link
Contributor

@merlinran merlinran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix thanks @cerblue ! Exactly what we wanted to do.

@merlinran merlinran merged commit 4a7c013 into textileio:main Oct 13, 2021
@cerblue cerblue deleted the split-market branch October 13, 2021 15:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants