Remove the OTLP receiver legacy gRPC port(55680) references

open-telemetry · May 25, 2021 · 9f8d657 · 9f8d657
1 parent d5258c5
commit 9f8d657
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 8 deletions.
diff --git a/pkg/collector/parser/receiver_otlp.go b/pkg/collector/parser/receiver_otlp.go
@@ -28,7 +28,6 @@ const (
 	parserNameOTLP = "__otlp"
 
 	defaultOTLPGRPCPort       int32 = 4317
-	defaultOTLPGRPCLegacyPort int32 = 55680
 	defaultOTLPHTTPPort       int32 = 55681
 )
 
@@ -71,11 +70,6 @@ func (o *OTLPReceiverParser) Ports() ([]corev1.ServicePort, error) {
 					Port:       defaultOTLPGRPCPort,
 					TargetPort: intstr.FromInt(int(defaultOTLPGRPCPort)),
 				},
-				{
-					Name:       portName(fmt.Sprintf("%s-grpc-legacy", o.name), defaultOTLPGRPCLegacyPort),
-					Port:       defaultOTLPGRPCLegacyPort,
-					TargetPort: intstr.FromInt(int(defaultOTLPGRPCPort)), // we target the official port, not the legacy
-				},
 			},
 		},
 		{

diff --git a/pkg/collector/parser/receiver_otlp_test.go b/pkg/collector/parser/receiver_otlp_test.go
@@ -65,15 +65,14 @@ func TestOTLPExposeDefaultPorts(t *testing.T) {
 		seen       bool
 	}{
 		"otlp-grpc":        {portNumber: 4317},
-		"otlp-grpc-legacy": {portNumber: 55680},
 	}
 
 	// test
 	ports, err := builder.Ports()
 
 	// verify
 	assert.NoError(t, err)
-	assert.Len(t, ports, 2)
+	assert.Len(t, ports, 1)
 
 	for _, port := range ports {
 		r := expectedResults[port.Name]