Merge pull request #11 from thinkingabouther/fix-resource-in-metrics

fix: add OTEL resource info to metrics
krakend · Apr 9, 2024 · 306bbfc · 306bbfc
2 parents 497adaa + ec591e0
commit 306bbfc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/example/README.md b/example/README.md
@@ -6,7 +6,7 @@ In the `docker_compose` folder there is the environment to run the observability
 
 ```
 make image
-cd docker-compose && make up
+cd docker_compose && make up
 ```
 
 ## Run local krakend instances and other services in docker compose

diff --git a/state/state.go b/state/state.go
@@ -83,6 +83,7 @@ func NewWithVersion(serviceName string, cfg *OTELStateConfig, version string,
 	var meterProvider metric.MeterProvider = noopmetric.NewMeterProvider()
 	var sdkMeterProvider *sdkmetric.MeterProvider
 	if len(metricOpts) > 0 {
+		metricOpts = append(metricOpts, sdkmetric.WithResource(res))
 		sdkMeterProvider = sdkmetric.NewMeterProvider(metricOpts...)
 		meterProvider = sdkMeterProvider
 	}