opensergo / opensergo-java-sdk

OpenSergo Java SDK
Apache License 2.0
41 stars 24 forks source link

Add support of Traffic-routing spec #15

Closed 123liuziming closed 1 year ago

123liuziming commented 1 year ago

Describe what this PR does / why we need it

Does this pull request fix one issue?

refrence to issue 16

Describe how you did it

Add xds proto so that OpenSergo can listen the route configuration

Describe how to verify it

Special notes for reviews

123liuziming commented 1 year ago

and the traffic-routing proto files of opensergo should be added into directory src/main/proto/ Therer is no traffic-routing proto, we just reuse the proto in xds

sczyh30 commented 1 year ago

and the traffic-routing proto files of opensergo should be added into directory src/main/proto/

Actually we just leveraged the original "route configuration" from xDS RDS proto (and there are some extension fields inside it, which may satisfy our requirement)

sczyh30 commented 1 year ago

cc @123liuziming @panxiaojun233 We may need to add some description for our design later.

jnan806 commented 1 year ago

cc @sczyh30 @123liuziming @panxiaojun233 I will merge it with squash and merge

sczyh30 commented 1 year ago

Nice work. Thanks!