otelconf: skip flaky exporter tests on windows (#7469)

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Co-authored-by: Robert Pająk <pellared@hotmail.com>

Author: bacherfl

.lycheeignore

@@ -1,2 +1,4 @@
 http://localhost
 http://169.254.169.254/metadata/instance/compute
+https://10.0.0.0
+https://localhost:4317

otelconf/v0.3.0/log_test.go

@@ -12,6 +12,7 @@ import (
 	"os"
 	"path/filepath"
 	"reflect"
+	"runtime"
 	"strings"
 	"testing"
 	"time"
@@ -706,6 +707,10 @@ func TestLogProcessor(t *testing.T) {
 }
 
 func Test_otlpGRPCLogExporter(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		// TODO (#7446): Fix the flakiness on Windows.
+		t.Skip("Test is flaky on Windows.")
+	}
 	type args struct {
 		ctx        context.Context
 		otlpConfig *OTLP
@@ -841,10 +846,15 @@ func startGRPCLogsCollector(t *testing.T, listener net.Listener, serverOptions [
 	c := &grpcLogsCollector{}
 
 	collogpb.RegisterLogsServiceServer(srv, c)
-	go func() { _ = srv.Serve(listener) }()
+
+	errCh := make(chan error, 1)
+	go func() { errCh <- srv.Serve(listener) }()
 
 	t.Cleanup(func() {
-		srv.Stop()
+		srv.GracefulStop()
+		if err := <-errCh; err != nil && !errors.Is(err, grpc.ErrServerStopped) {
+			assert.NoError(t, err)
+		}
 	})
 }
 

otelconf/v0.3.0/metric_test.go

@@ -14,6 +14,7 @@ import (
 	"os"
 	"path/filepath"
 	"reflect"
+	"runtime"
 	"strings"
 	"testing"
 	"time"
@@ -1398,6 +1399,10 @@ func TestPrometheusIPv6(t *testing.T) {
 }
 
 func Test_otlpGRPCMetricExporter(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		// TODO (#7446): Fix the flakiness on Windows.
+		t.Skip("Test is flaky on Windows.")
+	}
 	type args struct {
 		ctx        context.Context
 		otlpConfig *OTLPMetric
@@ -1548,10 +1553,15 @@ func startGRPCMetricCollector(t *testing.T, listener net.Listener, serverOptions
 	c := &grpcMetricCollector{}
 
 	v1.RegisterMetricsServiceServer(srv, c)
-	go func() { _ = srv.Serve(listener) }()
+
+	errCh := make(chan error, 1)
+	go func() { errCh <- srv.Serve(listener) }()
 
 	t.Cleanup(func() {
-		srv.Stop()
+		srv.GracefulStop()
+		if err := <-errCh; err != nil && !errors.Is(err, grpc.ErrServerStopped) {
+			assert.NoError(t, err)
+		}
 	})
 }
 

otelconf/v0.3.0/trace_test.go

@@ -12,6 +12,7 @@ import (
 	"os"
 	"path/filepath"
 	"reflect"
+	"runtime"
 	"strings"
 	"testing"
 	"time"
@@ -854,6 +855,10 @@ func TestSampler(t *testing.T) {
 }
 
 func Test_otlpGRPCTraceExporter(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		// TODO (#7446): Fix the flakiness on Windows.
+		t.Skip("Test is flaky on Windows.")
+	}
 	type args struct {
 		ctx        context.Context
 		otlpConfig *OTLP
@@ -989,10 +994,15 @@ func startGRPCTraceCollector(t *testing.T, listener net.Listener, serverOptions
 	c := &grpcTraceCollector{}
 
 	v1.RegisterTraceServiceServer(srv, c)
-	go func() { _ = srv.Serve(listener) }()
+
+	errCh := make(chan error, 1)
+	go func() { errCh <- srv.Serve(listener) }()
 
 	t.Cleanup(func() {
-		srv.Stop()
+		srv.GracefulStop()
+		if err := <-errCh; err != nil && !errors.Is(err, grpc.ErrServerStopped) {
+			assert.NoError(t, err)
+		}
 	})
 }