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

ui: admin ui vist produces proto: no encoder for Error string [GetProperties] message #10484

Closed
glycerine opened this issue Nov 6, 2016 · 3 comments
Labels
C-question A question rather than an issue. No code/spec/doc change needed. O-community Originated from the community
Milestone

Comments

@glycerine
Copy link
Contributor

I'm still seeing the two proto lines of #8605 happening when I visit the admin http interface... only once though. Not a bunch. Still, it may indicate a residual error.

$ ./cockroach start
build:      beta-20161103 @ 2016/11/03 15:03:54 (go1.7.3)
admin:      http://localhost:8080
sql:        postgresql://root@localhost:26257?sslmode=disable
logs:       cockroach-data/logs
store[0]:   path=cockroach-data
status:     initialized new cluster
clusterID:  {b8a215a3-d6fb-431e-9010-e90598003ae3}
nodeID:     1
proto: no encoder for Error string [GetProperties]
proto: no encoder for Code int [GetProperties]
@tamird
Copy link
Contributor

tamird commented Nov 6, 2016

A relevant discussion is ongoing in
gogo/protobuf#178 (comment).

Looks like this happens on the databases page due to
#9689. Specifically, it
seems that the manner in which grpc-gateway
https://github.com/grpc-ecosystem/grpc-gateway handles errors causes this
error to appear.

On Sat, Nov 5, 2016 at 9:00 PM, Jason E. Aten, Ph.D. <
[email protected]> wrote:

I'm still seeing the two proto lines of #8605
#8605 happening when I
visit the admin http interface... only once though. Not a bunch. Still, it
may indicate a residual error.

$ ./cockroach start
build: beta-20161103 @ 2016/11/03 15:03:54 (go1.7.3)
admin: http://localhost:8080
sql: postgresql://root@localhost:26257?sslmode=disable
logs: cockroach-data/logs
store[0]: path=cockroach-data
status: initialized new cluster
clusterID: {b8a215a3-d6fb-431e-9010-e90598003ae3}
nodeID: 1
proto: no encoder for Error string [GetProperties]
proto: no encoder for Code int [GetProperties]


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#10484, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABdsPDYIGwTgz5rV-M8A2Q9CTcJL2ewnks5q7SamgaJpZM4Kqbqx
.

@petermattis petermattis changed the title admin ui vist produces proto: no encoder for Error string [GetProperties] message ui: admin ui vist produces proto: no encoder for Error string [GetProperties] message Feb 23, 2017
@petermattis petermattis added this to the 1.0 milestone Feb 23, 2017
@dianasaur323
Copy link
Contributor

Closing due to inactivity, and that the associated issue #9689 has been closed. Please re-open if you disagree.

@tamird
Copy link
Contributor

tamird commented Apr 5, 2017

Thanks @dianasaur323. For posterity, this was actually fixed in grpc-ecosystem/grpc-gateway#256.

@jordanlewis jordanlewis added C-question A question rather than an issue. No code/spec/doc change needed. O-community Originated from the community and removed O-deprecated-community-questions labels Apr 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-question A question rather than an issue. No code/spec/doc change needed. O-community Originated from the community
Projects
None yet
Development

No branches or pull requests

5 participants