Correctly wraps errors instead of wrapping error strings (#4076)

Signed-off-by: Bogdan Drutu <[email protected]>
open-telemetry · Sep 20, 2021 · 37f476a · 37f476a
1 parent 1f5dd9f
commit 37f476a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/service/internal/builder/exporters_builder.go b/service/internal/builder/exporters_builder.go
@@ -251,7 +251,7 @@ func buildExporter(
 				// Could not create because this exporter does not support this data type.
 				return nil, exporterTypeMismatchErr(cfg, requirement.requiredBy, dataType)
 			}
-			return nil, fmt.Errorf("error creating %v exporter: %v", cfg.ID(), err)
+			return nil, fmt.Errorf("error creating %v exporter: %w", cfg.ID(), err)
 		}
 
 		// Check if the factory really created the exporter.

diff --git a/service/internal/builder/pipelines_builder.go b/service/internal/builder/pipelines_builder.go
@@ -199,7 +199,7 @@ func (pb *pipelinesBuilder) buildPipeline(ctx context.Context, pipelineCfg *conf
 		}
 
 		if err != nil {
-			return nil, fmt.Errorf("error creating processor %q in pipeline %q: %v",
+			return nil, fmt.Errorf("error creating processor %q in pipeline %q: %w",
 				procID, pipelineCfg.Name, err)
 		}
 

diff --git a/service/parserprovider/file.go b/service/parserprovider/file.go
@@ -38,7 +38,7 @@ func (fl *fileProvider) Get(context.Context) (*configparser.ConfigMap, error) {
 
 	cp, err := configparser.NewConfigMapFromFile(fileName)
 	if err != nil {
-		return nil, fmt.Errorf("error loading config file %q: %v", fileName, err)
+		return nil, fmt.Errorf("error loading config file %q: %w", fileName, err)
 	}
 
 	return cp, nil