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

Inconsistent CRO merge behavior #270

Closed
magnified103 opened this issue Nov 27, 2024 · 0 comments · Fixed by #272
Closed

Inconsistent CRO merge behavior #270

magnified103 opened this issue Nov 27, 2024 · 0 comments · Fixed by #272

Comments

@magnified103
Copy link
Contributor

Steps to reproduce:

  • Checkout the tree https://github.com/topology-foundation/ts-topology/tree/bugfix/inconsistent-merge
cd ts-topology/examples/chat
yarn
yarn build
pnpm exec tsx tests/index.test.ts

Result:

topology::network ::broadcastMessage: Error: PublishError.NoPeersSubscribedToTopic
    at @chainsafe/libp2p-gossipsub.publish (/Users/magnified/projects/ts-topology/node_modules/@chainsafe/libp2p-gossipsub/src/index.ts:2263:13)
    at async TopologyNetworkNode.broadcastMessage (file:///Users/magnified/projects/ts-topology/packages/network/dist/src/node.js:172:13)
merging vertex b9fcd76b07784ba73d4e8d59741ea8508f7656346f0c40d2c837b13a275b26df
merged vertex b9fcd76b07784ba73d4e8d59741ea8508f7656346f0c40d2c837b13a275b26df
merging vertex 02465e287e3d086f12c6edd856953ca5ad0f01d6707bf8e410b4a601314c1ca5
merged vertex ee075937c2a6c8ccf8d94fb2a130c596d3dbcc32910b6e744ad55c3e41b41ad6
^C

Behavior: one node tries to merge the root hash 02465e287e3d086f12c6edd856953ca5ad0f01d6707bf8e410b4a601314c1ca5 during the sync, and it creates a stale vertex ee075937c2a6c8ccf8d94fb2a130c596d3dbcc32910b6e744ad55c3e41b41ad6 during that process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant