Skip to content

test: local test bridge node #4717

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

Merged
merged 17 commits into from
May 13, 2025
Merged

test: local test bridge node #4717

merged 17 commits into from
May 13, 2025

Conversation

rootulp
Copy link
Collaborator

@rootulp rootulp commented Apr 30, 2025

Closes #4682

Testing

Prerequisites

  1. Installed celestia-node v0.23.0-rc0 which includes Cian's PR to bump node to app v4.

Steps

In one terminal

# Test just app v4
./scripts/single-node.sh

# Test app v1 through v4
./scripts/single-node-all-upgrades.sh 

In another terminal

./scripts/single-bridge-node.sh

@rootulp rootulp self-assigned this Apr 30, 2025
@rootulp
Copy link
Collaborator Author

rootulp commented Apr 30, 2025

Currently hitting

Error: node: failed to start: fetching block by hash : rpc error: code = Unimplemented desc = unknown service tendermint.rpc.grpc.BlockAPI

which should be resolved by #4580

@chatton
Copy link
Collaborator

chatton commented May 6, 2025

@rootulp did you end up getting around the unregistered BlockApi?

@rootulp
Copy link
Collaborator Author

rootulp commented May 6, 2025

Put this on pause to address other issues. Just resumed and hit this again on the bridge node

Error: node: failed to start: fetching block by hash : rpc error: code = Unimplemented desc = unknown service tendermint.rpc.grpc.BlockAPI

rootulp added 5 commits May 6, 2025 15:30
```
Error: node: failed to start: fetching block by hash : rpc error: code =
Unknown desc = nil block meta for block hash []
```
@rootulp
Copy link
Collaborator Author

rootulp commented May 7, 2025

Error: node: failed to start: fetching block by hash : rpc error: code = Unknown desc = nil block meta for block hash []

@rootulp rootulp requested review from sysrex and evan-forbes May 12, 2025 19:31
@rootulp rootulp marked this pull request as ready for review May 12, 2025 19:31
@rootulp rootulp requested a review from a team as a code owner May 12, 2025 19:31
@rootulp rootulp requested review from Manav-Aggarwal and removed request for a team May 12, 2025 19:31
@rootulp rootulp removed the request for review from Manav-Aggarwal May 12, 2025 19:31
Copy link
Member

@evan-forbes evan-forbes left a comment

Choose a reason for hiding this comment

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

utack

@rootulp rootulp merged commit 7d21b0a into celestiaorg:main May 13, 2025
24 of 26 checks passed
@rootulp rootulp deleted the rp/local-test branch May 13, 2025 14:14
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.

Local test with celestia-app v4 and celestia-node
3 participants