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

handle our view change in collator protocol validator side #4197

Conversation

kishansagathiya
Copy link
Contributor

Changes

network bridge will send a network bridge update message to collator protocol when it see new active leaves.
this PR handles that network bridge event for our view change

Tests

go test -tags integration github.com/ChainSafe/gossamer

Issues

#4156

network bridge will send a network bridge update message to collator protocol
when it see new active leaves.

this commit handles that network bridge event for our view change
Copy link
Member

@edwardmack edwardmack left a comment

Choose a reason for hiding this comment

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

Nice work, just some questions regarding what TODOs are included in the scope if this issue.

dot/parachain/collator-protocol/validator_side.go Outdated Show resolved Hide resolved
dot/parachain/network-bridge/receiver.go Show resolved Hide resolved
@kishansagathiya kishansagathiya merged commit e6a79a3 into kishan/feat/network-bridge-rx Sep 30, 2024
15 of 22 checks passed
@kishansagathiya kishansagathiya deleted the kishan/feat/collator-protocol-handle-view branch September 30, 2024 06:17
kishansagathiya added a commit that referenced this pull request Oct 4, 2024
- handled active leaves update message
- handled block finalized message
- relay network protocol messages
- handle view update message for receiver side of the network bridge #3864 
- decode messages to wire message (PR #4188), Fixes #4108
- handle our view change in collator protocol validator side (PR #4197), Issue #4156
- Handle network bridge messages (UpdateAuthorityIDs and NewGossipTopology) Fixes #3862 
- process network events for receiver side of network bridge Issue Fixes #3863 
-  process overseer signals for network bridge Fixes #3861
edwardmack added a commit that referenced this pull request Oct 4, 2024
statement distribution skeleton (#4169)

Statement Distribution subsystem is responsible for distributing signed statements that we have generated and forwarding statements generated by our peers.
This commit just introduces a skeleton for it.
- implements the subsystem interface
- registers the subsystem with overseer
- add the processMessage method with messages to be handled

Issue #3583

cleanup remove un-used code

add comments

regenerate mocks

feat(dot/parachain): receiver side of network bridge (#3955)

- handled active leaves update message
- handled block finalized message
- relay network protocol messages
- handle view update message for receiver side of the network bridge #3864
- decode messages to wire message (PR #4188), Fixes #4108
- handle our view change in collator protocol validator side (PR #4197), Issue #4156
- Handle network bridge messages (UpdateAuthorityIDs and NewGossipTopology) Fixes #3862
- process network events for receiver side of network bridge Issue Fixes #3863
-  process overseer signals for network bridge Fixes #3861
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.

3 participants