celestiaorg / go-cnc

celestia-node REST client in Go.
Apache License 2.0
15 stars 8 forks source link

client: bump namespace size #55

Closed tuxcanfly closed 1 year ago

tuxcanfly commented 1 year ago

Overview

This PR updates the namespace id size from 8 bytes to 28 bytes to make it compatible with latest celestia node v0.11.0-rc1.

Checklist

tuxcanfly commented 1 year ago

Trying to figure out why this test case is failing:

2023/05/31 07:34:13 github.com/testcontainers/testcontainers-go - Connected to docker:
  Server Version: 20.10.22
  API Version: 1.41
  Operating System: Docker Desktop
  Total Memory: 7850 MB
    cnc_test.go:35:
                Error Trace:    /Users/tuxcanfly/Work/go-cnc/integration_test/cnc_test.go:35
                                                        /Users/tuxcanfly/.asdf/installs/golang/1.20.4/packages/pkg/mod/github.com/stretchr/testify@v1.8.2/suite/suite.go:154
                                                        /Users/tuxcanfly/Work/go-cnc/integration_test/cnc_test.go:47
                Error:          failed to execute docker compose up:
                Test:           TestIntegrationTestSuite
                Messages:       error: one or more wait strategies could not be applied to the running containers: Unable to apply wait strategy &{60000000000 26659 /balance 0x1044899f0 0x1045b0d00 false false <nil> GET <nil> 3s} to service bridge0 due to context deadline exceeded
                                stdout: <nil>
                                stderr: <nil>
=== RUN   TestIntegrationTestSuite/TestDataRoundTrip
    cnc_test.go:80:
                Error Trace:    /Users/tuxcanfly/Work/go-cnc/integration_test/cnc_test.go:80
                Error:          Received unexpected error:
                                Post "http://localhost:26659/submit_pfb": dial tcp [::1]:26659: connect: connection refused
                Test:           TestIntegrationTestSuite/TestDataRoundTrip
jcstein commented 1 year ago

Does bumping celestia-node to v0.11.0-rc1 fix the above? based on these versions for Arabica

tuxcanfly commented 1 year ago

Good call, celestia-node should be v0.11.0-rc1 for new namespace size.

Those errors were due to outdated script start-bridge.sh.

Also bumped celestia-app to v1.0.0-rc1.

tuxcanfly commented 1 year ago

There seems to be an issue with the bridge node keystore, as the wait script times out trying to query the balance, but the bridge node panics:

bridge0  | 2023/05/31 16:47:31 http: panic serving 192.167.10.1:56866: runtime error: invalid memory address or nil pointer dereference
bridge0  | goroutine 635 [running]:
bridge0  | net/http.(*conn).serve.func1()
bridge0  |      /usr/local/go/src/net/http/server.go:1854 +0xb0
bridge0  | panic({0x220ab20, 0x47a3160})
bridge0  |      /usr/local/go/src/runtime/panic.go:890 +0x248
bridge0  | github.com/celestiaorg/celestia-app/x/blob/types.(*KeyringSigner).GetSignerInfo(...)