From 70ded758e81d574e408762aa4323bc9e70958268 Mon Sep 17 00:00:00 2001 From: Brett McBride Date: Fri, 20 Jan 2023 10:09:05 +1100 Subject: [PATCH 1/2] reduce quoteservice spans slim autoinstrumentation beta4 was updated to work without psr15, which was responsible for most of the spans --- src/quoteservice/Dockerfile | 2 +- src/quoteservice/composer.json | 8 +------- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/src/quoteservice/Dockerfile b/src/quoteservice/Dockerfile index 5606f176b2..9ebb48df6f 100644 --- a/src/quoteservice/Dockerfile +++ b/src/quoteservice/Dockerfile @@ -18,7 +18,7 @@ RUN chmod +x /usr/local/bin/install-php-extensions \ && install-php-extensions \ opcache \ protobuf \ - open-telemetry/opentelemetry-php-instrumentation@1.0.0beta1 + open-telemetry/opentelemetry-php-instrumentation@1.0.0beta2 WORKDIR /var/www COPY --from=build /tmp/vendor/ ./vendor/ diff --git a/src/quoteservice/composer.json b/src/quoteservice/composer.json index 182fd91c73..a1edfc3bca 100644 --- a/src/quoteservice/composer.json +++ b/src/quoteservice/composer.json @@ -9,8 +9,7 @@ "monolog/monolog": "2.8.0", "open-telemetry/sdk": "1.0.0beta1", "open-telemetry/exporter-otlp": "1.0.0beta1", - "open-telemetry/opentelemetry-auto-slim": "1.0.0beta3", - "open-telemetry/opentelemetry-auto-psr15": "1.0.0beta2", + "open-telemetry/opentelemetry-auto-slim": "1.0.0beta4", "guzzlehttp/guzzle": "7.4.5", "php-di/php-di": "6.4.0", "php-di/slim-bridge": "3.2.0", @@ -22,10 +21,5 @@ "psr-4": { "App\\": "src/" } - }, - "config": { - "allow-plugins": { - "phpstan/extension-installer": true - } } } From e552a763f5134805bf94cdeaa334dfb14e04fdd9 Mon Sep 17 00:00:00 2001 From: Brett McBride Date: Fri, 20 Jan 2023 10:12:11 +1100 Subject: [PATCH 2/2] update changelog --- CHANGELOG.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index e9e15f6d4c..0dc6156796 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -198,3 +198,5 @@ significant modifications will be credited to OpenTelemetry Authors. ([#691](https://github.com/open-telemetry/opentelemetry-demo/pull/691)) * Fix payment service version to support temporality environment variable ([#693](https://github.com/open-telemetry/opentelemetry-demo/pull/693)) +* Reduce spans generated from quote service +([#702](https://github.com/open-telemetry/opentelemetry-demo/pull/702))