[exporters/otlp/otlpmetric] Wrap upload metrics error to provide addi…

…tional context (#3974)

* Wrap upload metrics error to provide additional context

* Add PR number to CHANGELOG

* Update CHANGELOG.md

Co-authored-by: Damien Mathieu <42@dmathieu.com>

---------

Co-authored-by: Damien Mathieu <42@dmathieu.com>
Co-authored-by: Chester Cheung <cheung.zhy.csu@gmail.com>

CHANGELOG.md

@@ -21,6 +21,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
   This adds an implementation requirement to set the interface default behavior for unimplemented methods. (#3916)
 - Move No-Op implementation from `go.opentelemetry.io/otel/metric` into its own package `go.opentelemetry.io/otel/metric/noop`. (#3941)
   - `metric.NewNoopMeterProvider` is replaced with `noop.NewMeterProvider`
+- Wrap `UploadMetrics` error in `go.opentelemetry.io/otel/exporters/otlp/otlpmetric/` to improve error message when encountering generic grpc errors. (#3974)
 
 ### Fixed
 

exporters/otlp/otlpmetric/internal/exporter.go

@@ -58,7 +58,7 @@ func (e *exporter) Export(ctx context.Context, rm *metricdata.ResourceMetrics) e
 	e.clientMu.Unlock()
 	if upErr != nil {
 		if err == nil {
-			return upErr
+			return fmt.Errorf("failed to upload metrics: %w", upErr)
 		}
 		// Merge the two errors.
 		return fmt.Errorf("failed to upload incomplete metrics (%s): %w", err, upErr)