Skip to content

Conversation

@arvindbr8
Copy link
Member

This is the next part of stabilizing the include-xdstp-name-in-lds-experimental flag that was introduced in #35 per the steps mentioned in #57 .

Testing:

Setup Proxyless Service Mesh setup using a gRPC Server

  1. default behavior (i.e., not specifying the flag which implies: include-xdstp-name-in-lds-experimental=true)

    1. dial target w/o authority:
      xds:///helloworld.default.svc.cluster.local:50051

      ./grpcurl --plaintext   -d '{"name": "world"}'   xds:///helloworld.default.svc.cluster.local:50051  helloworld.Greeter/SayHello
      {
      	 "message": "Hello from psm-grpc-server-57d8c669b9-wz6fh world"
      }
      
    2. dial target w/ authority:
      xds://traffic-director-global.xds.googleapis.com/helloworld.default.svc.cluster.local:50051

       ./grpcurl --plaintext   -d '{"name": "world"}'   xds://traffic-director-global.xds.googleapis.com/helloworld.default.svc.cluster.local:50051  helloworld.Greeter/SayHello
       {
            "message": "Hello from psm-grpc-server-57d8c669b9-gbd58 world"
       }
      
  2. Settinginclude-xdstp-name-in-lds-experimental=false and dial target: xds:///helloworld.default.svc.cluster.local:50051

    ./grpcurl --plaintext -d '{"name": "world"}' xds:///helloworld.default.svc.cluster.local:50051 helloworld.Greeter/SayHello
    {
    "message": "Hello from psm-grpc-server-57d8c669b9-wz6fh world"
    }

for more details: go/testing-xdstp-listener-resource-name-template-bootstrap-generator

@arvindbr8 arvindbr8 merged commit 4ba99d7 into GoogleCloudPlatform:master Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants