Couple of items from the PR feedback

Author: ryanrolds

pkg/utils/api_conversion/trace_test.go

@@ -71,7 +71,7 @@ var _ = Describe("Trace utils", func() {
 					TargetSpecifier: &envoy_config_core_v3.GrpcService_EnvoyGrpc_{
 						EnvoyGrpc: &envoy_config_core_v3.GrpcService_EnvoyGrpc{
 							ClusterName: clusterName,
-							Authority:   "authority",
+							Authority:   authority,
 						},
 					},
 				},

projects/gloo/api/external/envoy/config/trace/v3/opentelemetry.proto

@@ -37,8 +37,10 @@ message OpenTelemetryConfig {
   // listener + the namespace of the Gateway object
   string service_name = 3;
 
+  // Optional. Current only gRPC is supported, but are intentionally using a oneof
+  // to allow for future support of HTTP. This structure matches Envoy's.
   oneof service_type {
-    // Optional, if set the communication to the collector will be over gRPC.
+    // Optional gRPC transport options
     GrpcService grpc_service = 4;
   }
 }