Remove unnecessary exclude rules from golangci (#3441)

Signed-off-by: Bogdan Drutu <[email protected]>

.golangci.yml

@@ -119,18 +119,14 @@ issues:
       linters:
         # See https://github.com/golangci/golangci-lint/issues/537#issuecomment-545170007
         - structcheck
-    - path: receiver/otlpreceiver/internal/marshal_jsonpb\.go
-      linters:
-        - gocritic
-        - gosimple
     - text: "G404:"
       linters:
         - gosec
     - text: "G402:"
       linters:
         - gosec
-    # Exclude internal and testbed packages for now.
-    - path: ".*internal.*|.*testbed.*"
+    # Exclude internal package for now.
+    - path: ".*internal.*"
       text: "should have comment|should be of the form"
       linters:
         - revive

testbed/testbed/test_bed.go

@@ -46,8 +46,9 @@ func SaveResults(resultsSummary TestResultsSummary) {
 
 const testBedEnableEnvVarName = "RUN_TESTBED"
 
+// GlobalConfig global config for testbed.
 var GlobalConfig = struct {
-	// Relative path to default agent executable to test.
+	// DefaultAgentExeRelativeFile relative path to default agent executable to test.
 	// Can be set in the contrib repo to use a different executable name.
 	// Set this before calling DoTestMain().
 	//
@@ -58,7 +59,7 @@ var GlobalConfig = struct {
 	// that is passed to the TestCase.
 	DefaultAgentExeRelativeFile string
 }{
-	// The default exe that is produced by Makefile "otelcol" target relative
+	// DefaultAgentExeRelativeFile the default exe that is produced by Makefile "otelcol" target relative
 	// to testbed/tests directory.
 	DefaultAgentExeRelativeFile: "../../bin/otelcol_{{.GOOS}}_{{.GOARCH}}",
 }