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

报 call triple server error = service was not found upstream even though it should have been there: service not found; service discovery error #523

Closed
nicktming opened this issue Nov 16, 2022 · 5 comments
Labels
good first issue Good for newcomers wontfix This will not be worked on

Comments

@nicktming
Copy link
Contributor

nicktming commented Nov 16, 2022

syntax = "proto3";

import "k8s.io/api/core/v1/generated.proto";

package test;

option go_package = "./;test";

service TestInterface {
rpc GetTest (TestRequest) returns (TestResponse) {
}
}

message TestRequest {
string uid = 1;
Test test = 2;
}

message TestResponse {
optional bool success = 1;
}

message Test {
optional string name = 1;
optional k8s.io.api.core.v1.Pod Pod = 2;
}

What happened:

当proto3引用proto2文件起的dubbo triple服务 p.call 报 client call err: call triple server error = service was not found upstream even though it should have been there: service not found; service discovery error

What you expected to happen:

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

@AlexStocks
Copy link
Contributor

AlexStocks commented Nov 18, 2022

image

根本原因:k8s 用了 gogo 组织的 gogoprotobuf 编译 pb,而 dubbogo 的 triple 使用的是go官方使用的 protobuf,导致协议不通。

@AlexStocks AlexStocks added good first issue Good for newcomers wontfix This will not be worked on labels Nov 19, 2022
@mark4z
Copy link
Member

mark4z commented Nov 26, 2022

使用本地导入的protoset有可能解决该问题

@mark4z
Copy link
Member

mark4z commented Nov 29, 2022

该问题是由于gprc reflection无法识别gogoprotoc生成的pb。
可能的解决方案,对于此种情况,为triple的filter增加一个配置的路径,用来加载protoset文件。这样对于不兼容的情况,不使用gprc reflection来避免问题。
但同时,需要保留gprc reflection的能力。

可以参考的项目:https://github.com/fullstorydev/grpcurl(拥有以上两种能力)
image

@mark4z
Copy link
Member

mark4z commented Nov 29, 2022

cc @jielun1995

@shawnh2 shawnh2 mentioned this issue Feb 25, 2023
3 tasks
@mark4z
Copy link
Member

mark4z commented Jun 19, 2023

close via #548

@mark4z mark4z closed this as completed Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants