[all] Remove more unused params (#22971)

Alex Boten · web-flow · commit b20a2527704d · 2023-05-31T10:34:22.000+02:00
Linked issue: #20424 Signed-off-by: Alex Boten <aboten@lightstep.com>
diff --git a/exporter/awsemfexporter/datapoint.go b/exporter/awsemfexporter/datapoint.go
@@ -26,7 +26,7 @@ var (
 	summaryMetricCalculator = aws.NewMetricCalculator(calculateSummaryDelta)
 )
 
-func calculateSummaryDelta(prev *aws.MetricValue, val interface{}, timestampMs time.Time) (interface{}, bool) {
+func calculateSummaryDelta(prev *aws.MetricValue, val interface{}, _ time.Time) (interface{}, bool) {
 	metricEntry := val.(summaryMetricEntry)
 	summaryDelta := metricEntry.sum
 	countDelta := metricEntry.count
@@ -98,7 +98,7 @@ type summaryMetricEntry struct {
 }
 
 // CalculateDeltaDatapoints retrieves the NumberDataPoint at the given index and performs rate/delta calculation if necessary.
-func (dps numberDataPointSlice) CalculateDeltaDatapoints(i int, instrumentationScopeName string, detailedMetrics bool) ([]dataPoint, bool) {
+func (dps numberDataPointSlice) CalculateDeltaDatapoints(i int, instrumentationScopeName string, _ bool) ([]dataPoint, bool) {
 	metric := dps.NumberDataPointSlice.At(i)
 	labels := createLabels(metric.Attributes(), instrumentationScopeName)
 	timestampMs := unixNanoToMilliseconds(metric.Timestamp())
@@ -138,7 +138,7 @@ func (dps numberDataPointSlice) CalculateDeltaDatapoints(i int, instrumentationS
 }
 
 // CalculateDeltaDatapoints retrieves the HistogramDataPoint at the given index.
-func (dps histogramDataPointSlice) CalculateDeltaDatapoints(i int, instrumentationScopeName string, detailedMetrics bool) ([]dataPoint, bool) {
+func (dps histogramDataPointSlice) CalculateDeltaDatapoints(i int, instrumentationScopeName string, _ bool) ([]dataPoint, bool) {
 	metric := dps.HistogramDataPointSlice.At(i)
 	labels := createLabels(metric.Attributes(), instrumentationScopeName)
 	timestamp := unixNanoToMilliseconds(metric.Timestamp())
diff --git a/exporter/awsemfexporter/emf_exporter.go b/exporter/awsemfexporter/emf_exporter.go
@@ -166,7 +166,7 @@ func (emf *emfExporter) listPushers() []cwlogs.Pusher {
 }
 
 // shutdown stops the exporter and is invoked during shutdown.
-func (emf *emfExporter) shutdown(ctx context.Context) error {
+func (emf *emfExporter) shutdown(_ context.Context) error {
 	for _, emfPusher := range emf.listPushers() {
 		returnError := emfPusher.ForceFlush()
 		if returnError != nil {
diff --git a/exporter/awsemfexporter/emf_exporter_test.go b/exporter/awsemfexporter/emf_exporter_test.go
@@ -39,7 +39,7 @@ type mockPusher struct {
 	mock.Mock
 }
 
-func (p *mockPusher) AddLogEntry(logEvent *cwlogs.Event) error {
+func (p *mockPusher) AddLogEntry(_ *cwlogs.Event) error {
 	args := p.Called(nil)
 	errorStr := args.String(0)
 	if errorStr != "" {
diff --git a/exporter/azuremonitorexporter/factory.go b/exporter/azuremonitorexporter/factory.go
@@ -52,7 +52,7 @@ func createDefaultConfig() component.Config {
 }
 
 func (f *factory) createTracesExporter(
-	ctx context.Context,
+	_ context.Context,
 	set exporter.CreateSettings,
 	cfg component.Config,
 ) (exporter.Traces, error) {
@@ -67,7 +67,7 @@ func (f *factory) createTracesExporter(
 }
 
 func (f *factory) createLogsExporter(
-	ctx context.Context,
+	_ context.Context,
 	set exporter.CreateSettings,
 	cfg component.Config,
 ) (exporter.Logs, error) {
@@ -82,7 +82,7 @@ func (f *factory) createLogsExporter(
 }
 
 func (f *factory) createMetricsExporter(
-	ctx context.Context,
+	_ context.Context,
 	set exporter.CreateSettings,
 	cfg component.Config,
 ) (exporter.Metrics, error) {
diff --git a/exporter/azuremonitorexporter/logexporter.go b/exporter/azuremonitorexporter/logexporter.go
@@ -18,7 +18,7 @@ type logExporter struct {
 	logger           *zap.Logger
 }
 
-func (exporter *logExporter) onLogData(context context.Context, logData plog.Logs) error {
+func (exporter *logExporter) onLogData(_ context.Context, logData plog.Logs) error {
 	resourceLogs := logData.ResourceLogs()
 	logPacker := newLogPacker(exporter.logger)
 
diff --git a/exporter/azuremonitorexporter/metricexporter.go b/exporter/azuremonitorexporter/metricexporter.go
@@ -19,7 +19,7 @@ type metricExporter struct {
 	packer           *metricPacker
 }
 
-func (exporter *metricExporter) onMetricData(context context.Context, metricData pmetric.Metrics) error {
+func (exporter *metricExporter) onMetricData(_ context.Context, metricData pmetric.Metrics) error {
 	resourceMetrics := metricData.ResourceMetrics()
 
 	for i := 0; i < resourceMetrics.Len(); i++ {
diff --git a/exporter/azuremonitorexporter/traceexporter.go b/exporter/azuremonitorexporter/traceexporter.go
@@ -51,7 +51,7 @@ func (v *traceVisitor) visit(
 	return true
 }
 
-func (exporter *traceExporter) onTraceData(context context.Context, traceData ptrace.Traces) error {
+func (exporter *traceExporter) onTraceData(_ context.Context, traceData ptrace.Traces) error {
 	spanCount := traceData.SpanCount()
 	if spanCount == 0 {
 		return nil
diff --git a/exporter/clickhouseexporter/exporter_logs_test.go b/exporter/clickhouseexporter/exporter_logs_test.go
@@ -144,7 +144,7 @@ type testClickhouseDriver struct {
 	recorder recorder
 }
 
-func (t *testClickhouseDriver) Open(name string) (driver.Conn, error) {
+func (t *testClickhouseDriver) Open(_ string) (driver.Conn, error) {
 	return &testClickhouseDriverConn{
 		recorder: t.recorder,
 	}, nil
@@ -169,7 +169,7 @@ func (*testClickhouseDriverConn) Begin() (driver.Tx, error) {
 	return &testClickhouseDriverTx{}, nil
 }
 
-func (*testClickhouseDriverConn) CheckNamedValue(v *driver.NamedValue) error {
+func (*testClickhouseDriverConn) CheckNamedValue(_ *driver.NamedValue) error {
 	return nil
 }
 
@@ -190,7 +190,7 @@ func (t *testClickhouseDriverStmt) Exec(args []driver.Value) (driver.Result, err
 	return nil, t.recorder(t.query, args)
 }
 
-func (t *testClickhouseDriverStmt) Query(args []driver.Value) (driver.Rows, error) {
+func (t *testClickhouseDriverStmt) Query(_ []driver.Value) (driver.Rows, error) {
 	return nil, nil
 }
 
diff --git a/exporter/clickhouseexporter/exporter_metrics.go b/exporter/clickhouseexporter/exporter_metrics.go
@@ -46,7 +46,7 @@ func (e *metricsExporter) start(ctx context.Context, _ component.Host) error {
 }
 
 // shutdown will shut down the exporter.
-func (e *metricsExporter) shutdown(ctx context.Context) error {
+func (e *metricsExporter) shutdown(_ context.Context) error {
 	if e.client != nil {
 		return e.client.Close()
 	}
diff --git a/exporter/datadogexporter/internal/hostmetadata/internal/ec2/ec2.go b/exporter/datadogexporter/internal/hostmetadata/internal/ec2/ec2.go
@@ -74,7 +74,7 @@ func GetHostInfo(logger *zap.Logger) (hostInfo *HostInfo) {
 	return
 }
 
-func (hi *HostInfo) GetHostname(logger *zap.Logger) string {
+func (hi *HostInfo) GetHostname(_ *zap.Logger) string {
 	if isDefaultHostname(hi.EC2Hostname) {
 		return hi.InstanceID
 	}
@@ -108,7 +108,7 @@ func (p *Provider) fillHostInfo() {
 	p.once.Do(func() { p.hostInfo = *GetHostInfo(p.logger) })
 }
 
-func (p *Provider) Source(ctx context.Context) (source.Source, error) {
+func (p *Provider) Source(_ context.Context) (source.Source, error) {
 	p.fillHostInfo()
 	if p.hostInfo.InstanceID == "" {
 		return source.Source{}, fmt.Errorf("instance ID is unavailable")
diff --git a/exporter/datadogexporter/internal/hostmetadata/internal/ecs/provider.go b/exporter/datadogexporter/internal/hostmetadata/internal/ecs/provider.go
@@ -27,7 +27,7 @@ type Provider struct {
 }
 
 // OnECSFargate determines if the application is running on ECS Fargate.
-func (p *Provider) OnECSFargate(ctx context.Context) (bool, error) {
+func (p *Provider) OnECSFargate(_ context.Context) (bool, error) {
 	if p.missingEndpoint {
 		// No ECS metadata endpoint, therefore not on ECS Fargate
 		return false, nil
diff --git a/exporter/datadogexporter/internal/hostmetadata/internal/gcp/provider.go b/exporter/datadogexporter/internal/hostmetadata/internal/gcp/provider.go
@@ -73,7 +73,7 @@ func (p *Provider) Source(context.Context) (source.Source, error) {
 	return source.Source{Kind: source.HostnameKind, Identifier: fmt.Sprintf("%s.%s", name, cloudAccount)}, nil
 }
 
-func (p *Provider) ClusterName(ctx context.Context) (string, error) {
+func (p *Provider) ClusterName(_ context.Context) (string, error) {
 	return p.detector.GKEClusterName()
 }
 
diff --git a/exporter/datadogexporter/internal/testutil/test_utils.go b/exporter/datadogexporter/internal/testutil/test_utils.go
@@ -97,7 +97,7 @@ type validateAPIKeyResponse struct {
 	Valid bool `json:"valid"`
 }
 
-func validateAPIKeyEndpoint(w http.ResponseWriter, r *http.Request) {
+func validateAPIKeyEndpoint(w http.ResponseWriter, _ *http.Request) {
 	res := validateAPIKeyResponse{Valid: true}
 	resJSON, _ := json.Marshal(res)
 
@@ -108,7 +108,7 @@ func validateAPIKeyEndpoint(w http.ResponseWriter, r *http.Request) {
 	}
 }
 
-func validateAPIKeyEndpointInvalid(w http.ResponseWriter, r *http.Request) {
+func validateAPIKeyEndpointInvalid(w http.ResponseWriter, _ *http.Request) {
 	res := validateAPIKeyResponse{Valid: false}
 	resJSON, _ := json.Marshal(res)
 
@@ -123,7 +123,7 @@ type metricsResponse struct {
 	Status string `json:"status"`
 }
 
-func metricsEndpoint(w http.ResponseWriter, r *http.Request) {
+func metricsEndpoint(w http.ResponseWriter, _ *http.Request) {
 	res := metricsResponse{Status: "ok"}
 	resJSON, _ := json.Marshal(res)
 
@@ -135,7 +135,7 @@ func metricsEndpoint(w http.ResponseWriter, r *http.Request) {
 	}
 }
 
-func metricsV2Endpoint(w http.ResponseWriter, r *http.Request) {
+func metricsV2Endpoint(w http.ResponseWriter, _ *http.Request) {
 	res := metricsResponse{Status: "ok"}
 	resJSON, _ := json.Marshal(res)
 
@@ -218,15 +218,15 @@ type MockSourceProvider struct {
 	Src source.Source
 }
 
-func (s *MockSourceProvider) Source(ctx context.Context) (source.Source, error) {
+func (s *MockSourceProvider) Source(_ context.Context) (source.Source, error) {
 	return s.Src, nil
 }
 
 type MockStatsProcessor struct {
 	In []pb.ClientStatsPayload
 }
 
-func (s *MockStatsProcessor) ProcessStats(in pb.ClientStatsPayload, lang, tracerVersion string) {
+func (s *MockStatsProcessor) ProcessStats(in pb.ClientStatsPayload, _, _ string) {
 	s.In = append(s.In, in)
 }
 
diff --git a/exporter/datadogexporter/logs_exporter.go b/exporter/datadogexporter/logs_exporter.go
@@ -69,7 +69,7 @@ func newLogsExporter(ctx context.Context, params exporter.CreateSettings, cfg *C
 var _ consumer.ConsumeLogsFunc = (*logsExporter)(nil).consumeLogs
 
 // consumeLogs is implementation of cosumer.ConsumeLogsFunc
-func (exp *logsExporter) consumeLogs(ctx context.Context, ld plog.Logs) (err error) {
+func (exp *logsExporter) consumeLogs(_ context.Context, ld plog.Logs) (err error) {
 	defer func() { err = exp.scrubber.Scrub(err) }()
 	if exp.cfg.HostMetadata.Enabled {
 		// start host metadata with resource attributes from
diff --git a/exporter/datadogexporter/traces_exporter_test.go b/exporter/datadogexporter/traces_exporter_test.go
@@ -38,10 +38,10 @@ func TestMain(m *testing.M) {
 type testlogger struct{}
 
 // Trace implements Logger.
-func (testlogger) Trace(v ...interface{}) {}
+func (testlogger) Trace(_ ...interface{}) {}
 
 // Tracef implements Logger.
-func (testlogger) Tracef(format string, params ...interface{}) {}
+func (testlogger) Tracef(_ string, _ ...interface{}) {}
 
 // Debug implements Logger.
 func (testlogger) Debug(v ...interface{}) { fmt.Println("DEBUG", fmt.Sprint(v...)) }
diff --git a/exporter/pulsarexporter/jaeger_marshaler.go b/exporter/pulsarexporter/jaeger_marshaler.go
@@ -21,7 +21,7 @@ type jaegerMarshaler struct {
 
 var _ TracesMarshaler = (*jaegerMarshaler)(nil)
 
-func (j jaegerMarshaler) Marshal(traces ptrace.Traces, topic string) ([]*pulsar.ProducerMessage, error) {
+func (j jaegerMarshaler) Marshal(traces ptrace.Traces, _ string) ([]*pulsar.ProducerMessage, error) {
 	batches, err := jaeger.ProtoFromTraces(traces)
 	if err != nil {
 		return nil, err
diff --git a/exporter/pulsarexporter/pdata_marshaler.go b/exporter/pulsarexporter/pdata_marshaler.go
@@ -15,7 +15,7 @@ type pdataLogsMarshaler struct {
 	encoding  string
 }
 
-func (p pdataLogsMarshaler) Marshal(ld plog.Logs, topic string) ([]*pulsar.ProducerMessage, error) {
+func (p pdataLogsMarshaler) Marshal(ld plog.Logs, _ string) ([]*pulsar.ProducerMessage, error) {
 	bts, err := p.marshaler.MarshalLogs(ld)
 	if err != nil {
 		return nil, err
@@ -43,7 +43,7 @@ type pdataMetricsMarshaler struct {
 	encoding  string
 }
 
-func (p pdataMetricsMarshaler) Marshal(ld pmetric.Metrics, topic string) ([]*pulsar.ProducerMessage, error) {
+func (p pdataMetricsMarshaler) Marshal(ld pmetric.Metrics, _ string) ([]*pulsar.ProducerMessage, error) {
 	bts, err := p.marshaler.MarshalMetrics(ld)
 	if err != nil {
 		return nil, err
@@ -71,7 +71,7 @@ type pdataTracesMarshaler struct {
 	encoding  string
 }
 
-func (p pdataTracesMarshaler) Marshal(td ptrace.Traces, topic string) ([]*pulsar.ProducerMessage, error) {
+func (p pdataTracesMarshaler) Marshal(td ptrace.Traces, _ string) ([]*pulsar.ProducerMessage, error) {
 	bts, err := p.marshaler.MarshalTraces(td)
 	if err != nil {
 		return nil, err
diff --git a/exporter/signalfxexporter/exporter.go b/exporter/signalfxexporter/exporter.go
@@ -238,7 +238,7 @@ func (se *signalfxExporter) pushLogs(ctx context.Context, ld plog.Logs) error {
 	return err
 }
 
-func (se *signalfxExporter) shutdown(ctx context.Context) error {
+func (se *signalfxExporter) shutdown(_ context.Context) error {
 	if se.cancelFn != nil {
 		se.cancelFn()
 	}
diff --git a/exporter/signalfxexporter/internal/hostmetadata/host_others.go b/exporter/signalfxexporter/internal/hostmetadata/host_others.go
@@ -8,10 +8,10 @@
 
 package hostmetadata // import "github.com/open-telemetry/opentelemetry-collector-contrib/exporter/signalfxexporter/internal/hostmetadata"
 
-func fillPlatformSpecificOSData(info *hostOS) error {
+func fillPlatformSpecificOSData(_ *hostOS) error {
 	return nil
 }
 
-func fillPlatformSpecificCPUData(info *hostCPU) error {
+func fillPlatformSpecificCPUData(_ *hostCPU) error {
 	return nil
 }
diff --git a/exporter/tencentcloudlogserviceexporter/logs_exporter.go b/exporter/tencentcloudlogserviceexporter/logs_exporter.go
@@ -34,7 +34,7 @@ type logServiceLogsSender struct {
 }
 
 func (s *logServiceLogsSender) pushLogsData(
-	ctx context.Context,
+	_ context.Context,
 	md plog.Logs) error {
 	var err error
 	clsLogs := convertLogs(md)
diff --git a/extension/awsproxy/extension.go b/extension/awsproxy/extension.go
@@ -23,7 +23,7 @@ type xrayProxy struct {
 
 var _ extension.Extension = (*xrayProxy)(nil)
 
-func (x xrayProxy) Start(ctx context.Context, host component.Host) error {
+func (x xrayProxy) Start(_ context.Context, host component.Host) error {
 	go func() {
 		if err := x.server.ListenAndServe(); !errors.Is(err, http.ErrServerClosed) && err != nil {
 			host.ReportFatalError(err)
diff --git a/extension/basicauthextension/extension.go b/extension/basicauthextension/extension.go
@@ -62,7 +62,7 @@ func newServerAuthExtension(cfg *Config) (auth.Server, error) {
 	), nil
 }
 
-func (ba *basicAuth) serverStart(ctx context.Context, host component.Host) error {
+func (ba *basicAuth) serverStart(_ context.Context, _ component.Host) error {
 	var rs []io.Reader
 
 	if ba.htpasswd.File != "" {
diff --git a/processor/datadogprocessor/processor.go b/processor/datadogprocessor/processor.go
@@ -61,7 +61,7 @@ func newProcessor(ctx context.Context, logger *zap.Logger, config component.Conf
 }
 
 // Start implements the component.Component interface.
-func (p *datadogProcessor) Start(ctx context.Context, host component.Host) error {
+func (p *datadogProcessor) Start(_ context.Context, host component.Host) error {
 	var datadogs []exporter.Metrics
 loop:
 	for k, exp := range host.GetExporters()[component.DataTypeMetrics] {
diff --git a/processor/datadogprocessor/processor_test.go b/processor/datadogprocessor/processor_test.go
@@ -209,9 +209,9 @@ type mockHost struct {
 	Exporters map[component.DataType]map[component.ID]component.Component
 }
 
-func (m *mockHost) ReportFatalError(err error) {}
+func (m *mockHost) ReportFatalError(_ error) {}
 
-func (m *mockHost) GetFactory(kind component.Kind, componentType component.Type) component.Factory {
+func (m *mockHost) GetFactory(_ component.Kind, _ component.Type) component.Factory {
 	return nil
 }
 
diff --git a/processor/logstransformprocessor/factory.go b/processor/logstransformprocessor/factory.go
@@ -34,7 +34,7 @@ func createDefaultConfig() component.Config {
 }
 
 func createLogsProcessor(
-	ctx context.Context,
+	_ context.Context,
 	set processor.CreateSettings,
 	cfg component.Config,
 	nextConsumer consumer.Logs) (processor.Logs, error) {
diff --git a/processor/logstransformprocessor/processor.go b/processor/logstransformprocessor/processor.go
@@ -62,7 +62,7 @@ func (ltp *logsTransformProcessor) Capabilities() consumer.Capabilities {
 	return consumer.Capabilities{MutatesData: true}
 }
 
-func (ltp *logsTransformProcessor) Shutdown(ctx context.Context) error {
+func (ltp *logsTransformProcessor) Shutdown(_ context.Context) error {
 	ltp.logger.Info("Stopping logs transform processor")
 	pipelineErr := ltp.pipe.Stop()
 	ltp.converter.Stop()
@@ -72,7 +72,7 @@ func (ltp *logsTransformProcessor) Shutdown(ctx context.Context) error {
 	return pipelineErr
 }
 
-func (ltp *logsTransformProcessor) Start(ctx context.Context, host component.Host) error {
+func (ltp *logsTransformProcessor) Start(ctx context.Context, _ component.Host) error {
 
 	// There is no need for this processor to use storage
 	err := ltp.pipe.Start(storage.NewNopClient())
@@ -116,7 +116,7 @@ func (ltp *logsTransformProcessor) Start(ctx context.Context, host component.Hos
 	return nil
 }
 
-func (ltp *logsTransformProcessor) ConsumeLogs(ctx context.Context, ld plog.Logs) error {
+func (ltp *logsTransformProcessor) ConsumeLogs(_ context.Context, ld plog.Logs) error {
 	// Add the logs to the chain
 	return ltp.fromConverter.Batch(ld)
 }
diff --git a/processor/schemaprocessor/transformer.go b/processor/schemaprocessor/transformer.go
diff --git a/receiver/activedirectorydsreceiver/scraper.go b/receiver/activedirectorydsreceiver/scraper.go
diff --git a/receiver/prometheusexecreceiver/factory.go b/receiver/prometheusexecreceiver/factory.go
diff --git a/receiver/prometheusexecreceiver/receiver.go b/receiver/prometheusexecreceiver/receiver.go
diff --git a/receiver/prometheusreceiver/internal/transaction.go b/receiver/prometheusreceiver/internal/transaction.go
diff --git a/receiver/purefareceiver/internal/scraper.go b/receiver/purefareceiver/internal/scraper.go
diff --git a/receiver/purefbreceiver/internal/scraper.go b/receiver/purefbreceiver/internal/scraper.go
diff --git a/receiver/rabbitmqreceiver/factory.go b/receiver/rabbitmqreceiver/factory.go
diff --git a/receiver/rabbitmqreceiver/scraper.go b/receiver/rabbitmqreceiver/scraper.go
diff --git a/receiver/sqlserverreceiver/factory_others.go b/receiver/sqlserverreceiver/factory_others.go
diff --git a/receiver/windowseventlogreceiver/receiver_others.go b/receiver/windowseventlogreceiver/receiver_others.go
diff --git a/receiver/windowsperfcountersreceiver/factory_others.go b/receiver/windowsperfcountersreceiver/factory_others.go

Original file line number	Diff line number	Diff line change
`@@ -166,7 +166,7 @@ func (emf *emfExporter) listPushers() []cwlogs.Pusher {`
`166`	`166`	`}`
`167`	`167`
`168`	`168`	`// shutdown stops the exporter and is invoked during shutdown.`
`169`		`-func (emf *emfExporter) shutdown(ctx context.Context) error {`
	`169`	`+func (emf *emfExporter) shutdown(_ context.Context) error {`
`170`	`170`	`for _, emfPusher := range emf.listPushers() {`
`171`	`171`	`returnError := emfPusher.ForceFlush()`
`172`	`172`	`if returnError != nil {`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ type mockPusher struct {`
`39`	`39`	`mock.Mock`
`40`	`40`	`}`
`41`	`41`
`42`		`-func (p mockPusher) AddLogEntry(logEvent cwlogs.Event) error {`
	`42`	`+func (p mockPusher) AddLogEntry(_ cwlogs.Event) error {`
`43`	`43`	`args := p.Called(nil)`
`44`	`44`	`errorStr := args.String(0)`
`45`	`45`	`if errorStr != "" {`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ type logExporter struct {`
`18`	`18`	`logger *zap.Logger`
`19`	`19`	`}`
`20`	`20`
`21`		`-func (exporter *logExporter) onLogData(context context.Context, logData plog.Logs) error {`
	`21`	`+func (exporter *logExporter) onLogData(_ context.Context, logData plog.Logs) error {`
`22`	`22`	`resourceLogs := logData.ResourceLogs()`
`23`	`23`	`logPacker := newLogPacker(exporter.logger)`
`24`	`24`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ type metricExporter struct {`
`19`	`19`	`packer *metricPacker`
`20`	`20`	`}`
`21`	`21`
`22`		`-func (exporter *metricExporter) onMetricData(context context.Context, metricData pmetric.Metrics) error {`
	`22`	`+func (exporter *metricExporter) onMetricData(_ context.Context, metricData pmetric.Metrics) error {`
`23`	`23`	`resourceMetrics := metricData.ResourceMetrics()`
`24`	`24`
`25`	`25`	`for i := 0; i < resourceMetrics.Len(); i++ {`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ func (v *traceVisitor) visit(`
`51`	`51`	`return true`
`52`	`52`	`}`
`53`	`53`
`54`		`-func (exporter *traceExporter) onTraceData(context context.Context, traceData ptrace.Traces) error {`
	`54`	`+func (exporter *traceExporter) onTraceData(_ context.Context, traceData ptrace.Traces) error {`
`55`	`55`	`spanCount := traceData.SpanCount()`
`56`	`56`	`if spanCount == 0 {`
`57`	`57`	`return nil`
Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,7 @@ type testClickhouseDriver struct {`
`144`	`144`	`recorder recorder`
`145`	`145`	`}`
`146`	`146`
`147`		`-func (t *testClickhouseDriver) Open(name string) (driver.Conn, error) {`
	`147`	`+func (t *testClickhouseDriver) Open(_ string) (driver.Conn, error) {`
`148`	`148`	`return &testClickhouseDriverConn{`
`149`	`149`	`recorder: t.recorder,`
`150`	`150`	`}, nil`
`@@ -169,7 +169,7 @@ func (*testClickhouseDriverConn) Begin() (driver.Tx, error) {`
`169`	`169`	`return &testClickhouseDriverTx{}, nil`
`170`	`170`	`}`
`171`	`171`
`172`		`-func (testClickhouseDriverConn) CheckNamedValue(v driver.NamedValue) error {`
	`172`	`+func (testClickhouseDriverConn) CheckNamedValue(_ driver.NamedValue) error {`
`173`	`173`	`return nil`
`174`	`174`	`}`
`175`	`175`
`@@ -190,7 +190,7 @@ func (t *testClickhouseDriverStmt) Exec(args []driver.Value) (driver.Result, err`
`190`	`190`	`return nil, t.recorder(t.query, args)`
`191`	`191`	`}`
`192`	`192`
`193`		`-func (t *testClickhouseDriverStmt) Query(args []driver.Value) (driver.Rows, error) {`
	`193`	`+func (t *testClickhouseDriverStmt) Query(_ []driver.Value) (driver.Rows, error) {`
`194`	`194`	`return nil, nil`
`195`	`195`	`}`
`196`	`196`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ func (e *metricsExporter) start(ctx context.Context, _ component.Host) error {`
`46`	`46`	`}`
`47`	`47`
`48`	`48`	`// shutdown will shut down the exporter.`
`49`		`-func (e *metricsExporter) shutdown(ctx context.Context) error {`
	`49`	`+func (e *metricsExporter) shutdown(_ context.Context) error {`
`50`	`50`	`if e.client != nil {`
`51`	`51`	`return e.client.Close()`
`52`	`52`	`}`
Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ func GetHostInfo(logger zap.Logger) (hostInfo HostInfo) {`
`74`	`74`	`return`
`75`	`75`	`}`
`76`	`76`
`77`		`-func (hi HostInfo) GetHostname(logger zap.Logger) string {`
	`77`	`+func (hi HostInfo) GetHostname(_ zap.Logger) string {`
`78`	`78`	`if isDefaultHostname(hi.EC2Hostname) {`
`79`	`79`	`return hi.InstanceID`
`80`	`80`	`}`
`@@ -108,7 +108,7 @@ func (p *Provider) fillHostInfo() {`
`108`	`108`	`p.once.Do(func() { p.hostInfo = *GetHostInfo(p.logger) })`
`109`	`109`	`}`
`110`	`110`
`111`		`-func (p *Provider) Source(ctx context.Context) (source.Source, error) {`
	`111`	`+func (p *Provider) Source(_ context.Context) (source.Source, error) {`
`112`	`112`	`p.fillHostInfo()`
`113`	`113`	`if p.hostInfo.InstanceID == "" {`
`114`	`114`	`return source.Source{}, fmt.Errorf("instance ID is unavailable")`