Address review feedback.

Signed-off-by: Peter Štibraný <[email protected]>

CHANGELOG.md

@@ -94,7 +94,7 @@
 * [BUGFIX] Ingester: fixed in-memory series count when replaying a corrupted WAL. #8295
 * [BUGFIX] Ingester: fix context cancellation handling when a query is busy looking up series in the TSDB index and `-blocks-storage.tsdb.head-postings-for-matchers-cache*` or `-blocks-storage.tsdb.block-postings-for-matchers-cache*` are in use. #8337
 * [BUGFIX] Querier: fix edge case where bucket indexes are sometimes cached forever instead of with the expected TTL. #8343
-* [BUGFIX] OTLP handler: fix errors returned by OTLP handler when used via httpgrpc tunelling. #8363
+* [BUGFIX] OTLP handler: fix errors returned by OTLP handler when used via httpgrpc tunneling. #8363
 
 ### Mixin
 

pkg/distributor/push_test.go

@@ -901,7 +901,6 @@ func TestOTLPPushHandlerErrorsAreReportedCorrectlyViaHttpgrpc(t *testing.T) {
 
 	srv, err := dskit_server.New(cfg)
 	require.NoError(t, err)
-	//srv.HTTP.
 
 	push := func(ctx context.Context, req *Request) error {
 		// Trigger conversion of incoming request to WriteRequest.
@@ -923,6 +922,7 @@ func TestOTLPPushHandlerErrorsAreReportedCorrectlyViaHttpgrpc(t *testing.T) {
 	require.NoError(t, err)
 
 	var wg sync.WaitGroup
+	wg.Add(1)
 	go func() { defer wg.Done(); _ = srv.Run() }()
 	t.Cleanup(func() {
 		srv.Stop()