[shipping] Use cumulative metrics in shipping service to be consistent with the other services of the demo (#2503)

* Use cumulative metrics to be consistent with the other apps of the demo

* Use cumulative metrics to be consistent with the other apps of the demo

* Add OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE to shipping

---------

Co-authored-by: Juliano Costa <julianocosta89@outlook.com>
Co-authored-by: Juliano Costa <juliano.costa@datadoghq.com>

Author: cyrille-leclerc

CHANGELOG.md

@@ -83,21 +83,24 @@ the release.
   ([#2392](https://github.com/open-telemetry/opentelemetry-demo/pull/2392))
 * [prometheus / grafana] Add Linux monitoring dashboard
   ([#2395](https://github.com/open-telemetry/opentelemetry-demo/pull/2395))
+* [prometheus /grafana] Add alerting demo through the `CartAddItemHighLatency`
+  alert rule
+  ([#2401](https://github.com/open-telemetry/opentelemetry-demo/pull/2401))
+* [cart] Enable automatic generation of `service.instance.id`
+  ([#2402](https://github.com/open-telemetry/opentelemetry-demo/pull/2402))
 * [grafana] Update OpenSearch logs index in APM Dashboards
   ([#2419](https://github.com/open-telemetry/opentelemetry-demo/pull/2419))
 * [frontend-proxy] Add redirects for web UI paths to ensure proper asset loading
   ([#2476](https://github.com/open-telemetry/opentelemetry-demo/pull/2476))
-* [cart] Enable automatic generation of `service.instance.id`
-  ([#2402](https://github.com/open-telemetry/opentelemetry-demo/pull/2402))
-* [prometheus /grafana] Add alerting demo through the `CartAddItemHighLatency`
-  alert rule
-  ([#2401](https://github.com/open-telemetry/opentelemetry-demo/pull/2401))
 * [chore] Bump dependent images
   ([#2477](https://github.com/open-telemetry/opentelemetry-demo/pull/2477))
 * [email] Add memory leak scenario to email service
   ([#2481](https://github.com/open-telemetry/opentelemetry-demo/pull/2481))
 * [checkout] Add graceful shutdown to checkout service
   ([#2491](https://github.com/open-telemetry/opentelemetry-demo/pull/2491))
+* [shipping] Use cumulative metrics in shipping service to be consistent
+  with the other services of the demo
+  ([#2503](https://github.com/open-telemetry/opentelemetry-demo/pull/2503))
 
 ## 2.0.2
 

docker-compose.minimal.yml

@@ -494,9 +494,10 @@ services:
     environment:
       - SHIPPING_PORT
       - QUOTE_ADDR
-      - OTEL_EXPORTER_OTLP_ENDPOINT=http://${OTEL_COLLECTOR_HOST}:${OTEL_COLLECTOR_PORT_GRPC}
+      - OTEL_EXPORTER_OTLP_ENDPOINT
       - OTEL_RESOURCE_ATTRIBUTES
       - OTEL_SERVICE_NAME=shipping
+      - OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE
     healthcheck:
       test: ["CMD-SHELL", "timeout 1 bash -c '>/dev/tcp/localhost/${SHIPPING_PORT}'"]
       start_period: 10s

docker-compose.yml

@@ -586,6 +586,7 @@ services:
       - OTEL_EXPORTER_OTLP_ENDPOINT
       - OTEL_RESOURCE_ATTRIBUTES
       - OTEL_SERVICE_NAME=shipping
+      - OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE
     healthcheck:
       test: ["CMD-SHELL", "timeout 1 bash -c '>/dev/tcp/localhost/${SHIPPING_PORT}'"]
       start_period: 10s

src/shipping/src/telemetry_conf.rs

@@ -42,7 +42,6 @@ fn init_meter_provider() -> opentelemetry_sdk::metrics::SdkMeterProvider {
         .with_resource(get_resource())
         .with_periodic_exporter(
             opentelemetry_otlp::MetricExporter::builder()
-                .with_temporality(opentelemetry_sdk::metrics::Temporality::Delta)
                 .with_tonic()
                 .build()
                 .expect("Failed to initialize metric exporter"),