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

chore: merge releases back into master #8638

Merged
merged 31 commits into from
May 11, 2022
Merged

chore: merge releases back into master #8638

merged 31 commits into from
May 11, 2022

Conversation

arajasek
Copy link
Contributor

No description provided.

vyzo and others added 30 commits April 12, 2022 11:32
Index provider integration uses a gossipsub topic to announce changes to
the advertised content. The topic name was fixed to the default topic
which is `/indexer/ingest/mainnet`.

In the case of lotus, the gossipsub validators enforce a list of topics
the instance is permitted to join by setting subscription filter option
when `PubSub` instance is constructed via DI.

Having the fixed topic name meant that any SP starting up a node on a
network other than `mainnet` would have to override the default config
to avoid the node crashing when index provider is enabled.

Instead of a fixed default, the changes here infer the allowed indexer
topic name from network name automatically if the topic configuration is
left empty.

Fixes #8510
deps: ffi: update ffi that includes the log fix
@arajasek arajasek requested a review from a team as a code owner May 11, 2022 00:03
@magik6k magik6k enabled auto-merge May 11, 2022 00:07
@magik6k magik6k disabled auto-merge May 11, 2022 00:07
@magik6k magik6k merged commit 03f4124 into master May 11, 2022
@magik6k magik6k deleted the asr/merge-releases branch May 11, 2022 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants