Change verbosity level for otelcol_exporter_queue_batch_send_size metric (#14279)

<!-- Issue number if applicable -->
#### Link to tracking issue
Fixes #14278

---------

Signed-off-by: Israel Blancas <[email protected]>

Author: iblancasa

.chloggen/14278.yaml

@@ -0,0 +1,25 @@
+# Use this changelog template to create an entry for release notes.
+
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: breaking
+
+# The name of the component, or a single word describing the area of concern, (e.g. receiver/otlp)
+component: pkg/exporterhelper
+
+# A brief description of the change.  Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: Change verbosity level for otelcol_exporter_queue_batch_send_size metric to detailed.
+
+# One or more tracking issues or pull requests related to the change
+issues: [14278]
+
+# (Optional) One or more lines of additional information to render under the primary note.
+# These lines will be padded with 2 spaces and then inserted directly into the document.
+# Use pipe (|) for multiline entries.
+subtext:
+
+# Optional: The change log or logs in which this entry should be included.
+# e.g. '[user]' or '[user, api]'
+# Include 'user' if the change is relevant to end users.
+# Include 'api' if there is a change to a library API.
+# Default: '[user]'
+change_logs: []

service/internal/metricviews/views.go

@@ -93,10 +93,15 @@ func DefaultViews(level configtelemetry.Level) []config.View {
 	// Batch exporter metrics
 	if level < configtelemetry.LevelDetailed {
 		scope := ptr("go.opentelemetry.io/collector/exporter/exporterhelper")
-		views = append(views, dropViewOption(&config.ViewSelector{
-			MeterName:      scope,
-			InstrumentName: ptr("otelcol_exporter_queue_batch_send_size_bytes"),
-		}))
+		views = append(views,
+			dropViewOption(&config.ViewSelector{
+				MeterName:      scope,
+				InstrumentName: ptr("otelcol_exporter_queue_batch_send_size_bytes"),
+			}),
+			dropViewOption(&config.ViewSelector{
+				MeterName:      scope,
+				InstrumentName: ptr("otelcol_exporter_queue_batch_send_size"),
+			}))
 	}
 
 	// Batch processor metrics

service/internal/metricviews/views_test.go

@@ -7,6 +7,7 @@ import (
 	"testing"
 
 	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 
 	"go.opentelemetry.io/collector/config/configtelemetry"
 )
@@ -21,17 +22,17 @@ func TestDefaultViews(t *testing.T) {
 		{
 			name:           "None",
 			level:          configtelemetry.LevelNone,
-			wantViewsCount: 16,
+			wantViewsCount: 17,
 		},
 		{
 			name:           "Basic",
 			level:          configtelemetry.LevelBasic,
-			wantViewsCount: 16,
+			wantViewsCount: 17,
 		},
 		{
 			name:           "Normal",
 			level:          configtelemetry.LevelNormal,
-			wantViewsCount: 13,
+			wantViewsCount: 14,
 		},
 		{
 			name:           "Detailed",
@@ -45,3 +46,70 @@ func TestDefaultViews(t *testing.T) {
 		})
 	}
 }
+
+func TestDefaultViews_BatchExporterMetrics(t *testing.T) {
+	tests := []struct {
+		name             string
+		level            configtelemetry.Level
+		shouldDropBucket bool
+		shouldDropBytes  bool
+	}{
+		{
+			name:             "basic level drops bucket and bytes",
+			level:            configtelemetry.LevelBasic,
+			shouldDropBucket: true,
+			shouldDropBytes:  true,
+		},
+		{
+			name:             "normal level drops bucket and bytes",
+			level:            configtelemetry.LevelNormal,
+			shouldDropBucket: true,
+			shouldDropBytes:  true,
+		},
+		{
+			name:             "detailed level does not drop bucket or bytes",
+			level:            configtelemetry.LevelDetailed,
+			shouldDropBucket: false,
+			shouldDropBytes:  false,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			views := DefaultViews(tt.level)
+
+			exporterHelperScope := "go.opentelemetry.io/collector/exporter/exporterhelper"
+			bucketMetricName := "otelcol_exporter_queue_batch_send_size"
+			bytesMetricName := "otelcol_exporter_queue_batch_send_size_bytes"
+
+			var foundBucketDrop, foundBytesDrop bool
+			for _, view := range views {
+				if view.Selector != nil {
+					if view.Selector.MeterName != nil && *view.Selector.MeterName == exporterHelperScope {
+						if view.Selector.InstrumentName != nil {
+							if *view.Selector.InstrumentName == bucketMetricName {
+								foundBucketDrop = true
+								// Verify it's a drop view
+								require.NotNil(t, view.Stream)
+								require.NotNil(t, view.Stream.Aggregation)
+								require.NotNil(t, view.Stream.Aggregation.Drop)
+							}
+							if *view.Selector.InstrumentName == bytesMetricName {
+								foundBytesDrop = true
+								// Verify it's a drop view
+								require.NotNil(t, view.Stream)
+								require.NotNil(t, view.Stream.Aggregation)
+								require.NotNil(t, view.Stream.Aggregation.Drop)
+							}
+						}
+					}
+				}
+			}
+
+			assert.Equal(t, tt.shouldDropBucket, foundBucketDrop,
+				"bucket metric drop view should be %v for level %v", tt.shouldDropBucket, tt.level)
+			assert.Equal(t, tt.shouldDropBytes, foundBytesDrop,
+				"bytes metric drop view should be %v for level %v", tt.shouldDropBytes, tt.level)
+		})
+	}
+}