web3labs / crux

Data Privacy for Quorum Blockchains
https://www.web3labs.com
Apache License 2.0
53 stars 28 forks source link

Server does not implement chimera.ClientServer (missing Delete method) #33

Closed drandreaskrueger closed 5 years ago

drandreaskrueger commented 6 years ago

Please switch on the "github issues" for your https://github.com/blk-io/quorum-examples until then, I am reporting it here.

git clone https://github.com/blk-io/quorum-examples.git blk-io_quorum-examples
cd blk-io_quorum-examples/
vagrant up

fails:

...
==> default: github.com/ethereum/go-ethereum/whisper/whisperv2
==> default: Cloning into 'crux'...
==> default: Note: checking out 'tags/v1.0.0'.
==> default: 
==> default: You are in 'detached HEAD' state. You can look around, make experimental
==> default: changes and commit them, and you can discard any commits you make in this
==> default: state without impacting any branches by performing another checkout.
==> default: 
==> default: If you want to create a new branch to retain commits you create, you may
==> default: do so (now or later) by using -b with the checkout command again. Example:
==> default: 
==> default:   git checkout -b <new-branch-name>
==> default: HEAD is now at 1a124b3... Update the gRPC address and add logs when connected to other nodes (#23)
==> default: go get -u github.com/golang/dep/cmd/dep
==> default: go get -u golang.org/x/tools/cmd/goimports
==> default: go get -u github.com/wadey/gocovmerge
==> default: (cd /home/vagrant/crux/.GOPATH/src/github.com/blk-io/crux && ./bin/dep ensure)
==> default: # github.com/blk-io/crux/vendor/github.com/jsimonetti/berkeleydb
==> default: bdb.c: In function ‘go_db_cursor’:
==> default: bdb.c:78:38: warning: passing argument 3 of ‘dbp->cursor’ from incompatible pointer type [-Wincompatible-pointer-types]
==> default:          ret = dbp->cursor(dbp, NULL, dbcp, 0);
==> default:                                       ^
==> default: bdb.c:78:38: note: expected ‘DBC ** {aka struct __dbc **}’ but argument is of type ‘DBC * {aka struct __dbc *}’
==> default: # github.com/blk-io/crux/server
==> default: .GOPATH/src/github.com/blk-io/crux/server/proto_server.go:23:43: cannot use &s (type *Server) as type chimera.ClientServer in argument to chimera.RegisterClientServer:
==> default:    *Server does not implement chimera.ClientServer (missing Delete method)
==> default: .GOPATH/src/github.com/blk-io/crux/server/proto_server.go:78:43: cannot use &s (type *Server) as type chimera.ClientServer in argument to chimera.RegisterClientServer:
==> default:    *Server does not implement chimera.ClientServer (missing Delete method)
==> default: .GOPATH/src/github.com/blk-io/crux/server/proto_server.go:115:43: cannot use &s (type *Server) as type chimera.ClientServer in argument to chimera.RegisterClientServer:
==> default:    *Server does not implement chimera.ClientServer (missing Delete method)
==> default: Makefile:16: recipe for target 'build' failed
==> default: make: *** [build] Error 2
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.
drandreaskrueger commented 6 years ago

and please switch on the raft RPC api in https://github.com/blk-io/quorum-examples/blob/68610ee8ff9aa187d3ba76c92ed2c991c0b59e7b/examples/7nodes/raft-start.sh#L11

Puneetha17 commented 5 years ago

The Vagrant build is fixed, thanks!

drandreaskrueger commented 5 years ago

@Puneetha17 thanks.

and please switch on the raft RPC api in https://github.com/blk-io/quorum-examples/blob/68610ee8ff9aa187d3ba76c92ed2c991c0b59e7b/examples/7nodes/raft-start.sh#L11

Puneetha17 commented 5 years ago

Done

drandreaskrueger commented 5 years ago

nope, @Puneetha17, it is still not switched on:

--rpcapi admin,db,eth,debug,miner,net,shh,txpool,personal,web3,quorum

-->

--rpcapi admin,db,eth,debug,miner,net,shh,txpool,personal,web3,quorum,raft

in https://github.com/blk-io/quorum-examples/blob/master/examples/7nodes/raft-start.sh

newest version in master is 68610ee, no?

Puneetha17 commented 5 years ago

My bad, sorry. Fixed it now.