Merge branch 'open-telemetry:main' into main

weslenteche · web-flow · commit 8c766fd014ea · 2025-07-21T21:37:59.000-03:00
diff --git a/.github/workflows/split_monorepo.yaml b/.github/workflows/split_monorepo.yaml
@@ -15,6 +15,8 @@ permissions:
 jobs:
   gitsplit:
     runs-on: ubuntu-latest
+    permissions:
+      contents: write # required for pushing changes
     steps:
       - name: checkout
         run: git clone "$GITHUB_SERVER_URL/$GITHUB_REPOSITORY" "$GITHUB_WORKSPACE" && cd "$GITHUB_WORKSPACE" && git checkout $GITHUB_SHA
diff --git a/src/Aws/.gitattributes b/src/Aws/.gitattributes
@@ -8,6 +8,7 @@
 /.phan export-ignore
 /.php-cs-fixer.php export-ignore
 /coverage.clover export-ignore
+/examples export-ignore
 /phpstan.neon.dist export-ignore
 /phpstan.neon.dist export-ignore
 /phpunit.xml.dist export-ignore
diff --git a/src/Aws/composer.json b/src/Aws/composer.json
@@ -33,7 +33,7 @@
     },
     "autoload": {
         "psr-4": {
-            "OpenTelemetry\\Aws\\": "src/"
+            "OpenTelemetry\\Contrib\\Aws\\": "src/"
         }
     },
     "autoload-dev": {
diff --git a/src/Aws/examples/AwsClientApp/bin/app b/src/Aws/examples/AwsClientApp/bin/app
diff --git a/src/Aws/examples/AwsClientApp/composer.json b/src/Aws/examples/AwsClientApp/composer.json
diff --git a/src/Aws/examples/AwsClientApp/src/main.php b/src/Aws/examples/AwsClientApp/src/main.php
@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright The OpenTelemetry Authors
  *
@@ -23,8 +24,8 @@
 use Aws\S3\S3Client;
 use GuzzleHttp\Client;
 use OpenTelemetry\API\Trace\SpanKind;
-use OpenTelemetry\Aws\Xray\IdGenerator;
-use OpenTelemetry\Aws\Xray\Propagator;
+use OpenTelemetry\Contrib\Aws\Xray\IdGenerator;
+use OpenTelemetry\Contrib\Aws\Xray\Propagator;
 use OpenTelemetry\Contrib\OtlpGrpc\Exporter as OTLPExporter;
 use OpenTelemetry\SDK\Trace\SpanProcessor\SimpleSpanProcessor;
 use OpenTelemetry\SDK\Trace\TracerProvider;
diff --git a/src/Aws/examples/README.md b/src/Aws/examples/README.md
diff --git a/src/Aws/examples/collector/config.yaml b/src/Aws/examples/collector/config.yaml
diff --git a/src/Aws/examples/sampleApp2/SampleApp2.php b/src/Aws/examples/sampleApp2/SampleApp2.php
@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright The OpenTelemetry Authors
  *
@@ -21,8 +22,8 @@
 
 require __DIR__ . '/../../../vendor/autoload.php';
 
-use OpenTelemetry\Aws\Xray\IdGenerator;
-use OpenTelemetry\Aws\Xray\Propagator;
+use OpenTelemetry\Contrib\Aws\Xray\IdGenerator;
+use OpenTelemetry\Contrib\Aws\Xray\Propagator;
 use OpenTelemetry\Contrib\OtlpGrpc\Exporter as OTLPExporter;
 use OpenTelemetry\SDK\Trace\PropagationMap;
 use OpenTelemetry\SDK\Trace\SpanProcessor\SimpleSpanProcessor;
diff --git a/src/Aws/examples/sampleApp2/Service1.php b/src/Aws/examples/sampleApp2/Service1.php
@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright The OpenTelemetry Authors
  *
@@ -21,8 +22,8 @@
 
 require __DIR__ . '/../../../vendor/autoload.php';
 
-use OpenTelemetry\Aws\Xray\IdGenerator;
-use OpenTelemetry\Aws\Xray\Propagator;
+use OpenTelemetry\Contrib\Aws\Xray\IdGenerator;
+use OpenTelemetry\Contrib\Aws\Xray\Propagator;
 use OpenTelemetry\Contrib\OtlpGrpc\Exporter as OTLPExporter;
 use OpenTelemetry\SDK\Trace\PropagationMap;
 use OpenTelemetry\SDK\Trace\SpanProcessor\SimpleSpanProcessor;
@@ -45,13 +46,13 @@ public function useService()
     {
         $Exporter = new OTLPExporter();
         $map = new PropagationMap();
-        
+
         // Create a tracer object that uses the AWS X-Ray ID Generator to
         // generate trace IDs in the correct format
         $tracer = (new TracerProvider(null, null, new IdGenerator()))
             ->addSpanProcessor(new SimpleSpanProcessor($Exporter))
             ->getTracer('io.opentelemetry.contrib.php');
-        
+
         // Extract the SpanContext from the carrier
         $spanContext = Propagator::extract($this->carrier, $map);
 
diff --git a/src/Aws/examples/sampleApp2/Service2.php b/src/Aws/examples/sampleApp2/Service2.php
@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright The OpenTelemetry Authors
  *
@@ -21,8 +22,8 @@
 
 require __DIR__ . '/../../../vendor/autoload.php';
 
-use OpenTelemetry\Aws\Xray\IdGenerator;
-use OpenTelemetry\Aws\Xray\Propagator;
+use OpenTelemetry\Contrib\Aws\Xray\IdGenerator;
+use OpenTelemetry\Contrib\Aws\Xray\Propagator;
 use OpenTelemetry\Contrib\OtlpGrpc\Exporter as OTLPExporter;
 use OpenTelemetry\SDK\Trace\PropagationMap;
 use OpenTelemetry\SDK\Trace\SpanProcessor\SimpleSpanProcessor;
@@ -45,13 +46,13 @@ public function useService()
     {
         $Exporter = new OTLPExporter();
         $map = new PropagationMap();
-        
+
         // Create a tracer object that uses the AWS X-Ray ID Generator to
         // generate trace IDs in the correct format
         $tracer = (new TracerProvider(null, null, new IdGenerator()))
             ->addSpanProcessor(new SimpleSpanProcessor($Exporter))
             ->getTracer('io.opentelemetry.contrib.php');
-        
+
         // Extract the SpanContext from the carrier
         $spanContext = Propagator::extract($this->carrier, $map);
 
diff --git a/src/Aws/src/AwsSdkInstrumentation.php b/src/Aws/src/AwsSdkInstrumentation.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace OpenTelemetry\Aws;
+namespace OpenTelemetry\Contrib\Aws;
 
 use Aws\CommandInterface;
 use Aws\Middleware;
diff --git a/src/Aws/src/Ec2/Detector.php b/src/Aws/src/Ec2/Detector.php
@@ -18,7 +18,7 @@
  * limitations under the License.
  */
 
-namespace OpenTelemetry\Aws\Ec2;
+namespace OpenTelemetry\Contrib\Aws\Ec2;
 
 use OpenTelemetry\SDK\Common\Attribute\Attributes;
 use OpenTelemetry\SDK\Resource\ResourceDetectorInterface;
diff --git a/src/Aws/src/Ecs/DataProvider.php b/src/Aws/src/Ecs/DataProvider.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace OpenTelemetry\Aws\Ecs;
+namespace OpenTelemetry\Contrib\Aws\Ecs;
 
 class DataProvider
 {
diff --git a/src/Aws/src/Ecs/Detector.php b/src/Aws/src/Ecs/Detector.php
@@ -17,7 +17,7 @@
  * limitations under the License.
  */
 
-namespace OpenTelemetry\Aws\Ecs;
+namespace OpenTelemetry\Contrib\Aws\Ecs;
 
 use OpenTelemetry\API\Behavior\LogsMessagesTrait;
 use OpenTelemetry\SDK\Common\Attribute\Attributes;
diff --git a/src/Aws/src/Eks/DataProvider.php b/src/Aws/src/Eks/DataProvider.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace OpenTelemetry\Aws\Eks;
+namespace OpenTelemetry\Contrib\Aws\Eks;
 
 class DataProvider
 {
diff --git a/src/Aws/src/Eks/Detector.php b/src/Aws/src/Eks/Detector.php
@@ -17,7 +17,7 @@
  * limitations under the License.
  */
 
-namespace OpenTelemetry\Aws\Eks;
+namespace OpenTelemetry\Contrib\Aws\Eks;
 
 use OpenTelemetry\SDK\Common\Attribute\Attributes;
 use OpenTelemetry\SDK\Resource\ResourceDetectorInterface;
diff --git a/src/Aws/src/Lambda/Detector.php b/src/Aws/src/Lambda/Detector.php
@@ -18,7 +18,7 @@
  * limitations under the License.
  */
 
-namespace OpenTelemetry\Aws\Lambda;
+namespace OpenTelemetry\Contrib\Aws\Lambda;
 
 use OpenTelemetry\SDK\Common\Attribute\Attributes;
 use OpenTelemetry\SDK\Resource\ResourceDetectorInterface;
diff --git a/src/Aws/src/Xray/IdGenerator.php b/src/Aws/src/Xray/IdGenerator.php
@@ -18,7 +18,7 @@
 
 declare(strict_types=1);
 
-namespace OpenTelemetry\Aws\Xray;
+namespace OpenTelemetry\Contrib\Aws\Xray;
 
 use OpenTelemetry\SDK\Trace\IdGeneratorInterface;
 use OpenTelemetry\SDK\Trace\RandomIdGenerator;
diff --git a/src/Aws/src/Xray/Propagator.php b/src/Aws/src/Xray/Propagator.php
@@ -18,7 +18,7 @@
 
 declare(strict_types=1);
 
-namespace OpenTelemetry\Aws\Xray;
+namespace OpenTelemetry\Contrib\Aws\Xray;
 
 use OpenTelemetry\API\Trace as API;
 use OpenTelemetry\API\Trace\SpanContext;
@@ -60,7 +60,7 @@ class Propagator implements TextMapPropagatorInterface
     private const SAMPLED_FLAG_KEY = 'Sampled';
     private const IS_SAMPLED = '1';
     private const NOT_SAMPLED = '0';
-    
+
     /**
      * Returns list of fields used by HTTPTextFormat
      */
diff --git a/src/Aws/tests/Integration/AwsSdkInstrumentationTest.php b/src/Aws/tests/Integration/AwsSdkInstrumentationTest.php
@@ -12,8 +12,8 @@
 use Aws\Sqs\SqsClient;
 use GuzzleHttp\Promise;
 use OpenTelemetry\API\Trace\StatusCode;
-use OpenTelemetry\Aws\AwsSdkInstrumentation;
-use OpenTelemetry\Aws\Xray\Propagator;
+use OpenTelemetry\Contrib\Aws\AwsSdkInstrumentation;
+use OpenTelemetry\Contrib\Aws\Xray\Propagator;
 use OpenTelemetry\SDK\Trace\ReadWriteSpanInterface;
 use OpenTelemetry\SDK\Trace\TracerProvider;
 use PHPUnit\Framework\TestCase;
diff --git a/src/Aws/tests/Unit/AwsSdkInstrumentationTest.php b/src/Aws/tests/Unit/AwsSdkInstrumentationTest.php
@@ -6,8 +6,8 @@
 
 use DG\BypassFinals;
 use OpenTelemetry\API\Trace\TracerInterface;
-use OpenTelemetry\Aws\AwsSdkInstrumentation;
-use OpenTelemetry\Aws\Xray\Propagator;
+use OpenTelemetry\Contrib\Aws\AwsSdkInstrumentation;
+use OpenTelemetry\Contrib\Aws\Xray\Propagator;
 use OpenTelemetry\SDK\Trace\TracerProviderInterface;
 use PHPUnit\Framework\TestCase;
 
diff --git a/src/Aws/tests/Unit/Ec2/DetectorTest.php b/src/Aws/tests/Unit/Ec2/DetectorTest.php
@@ -9,7 +9,7 @@
 use GuzzleHttp\HandlerStack;
 use GuzzleHttp\Psr7\HttpFactory;
 use GuzzleHttp\Psr7\Response;
-use OpenTelemetry\Aws\Ec2\Detector;
+use OpenTelemetry\Contrib\Aws\Ec2\Detector;
 use OpenTelemetry\SDK\Common\Attribute\Attributes;
 use OpenTelemetry\SDK\Resource\ResourceInfoFactory;
 use OpenTelemetry\SemConv\ResourceAttributes;
diff --git a/src/Aws/tests/Unit/Ecs/DetectorTest.php b/src/Aws/tests/Unit/Ecs/DetectorTest.php
@@ -10,8 +10,8 @@
 use GuzzleHttp\Psr7\HttpFactory;
 use GuzzleHttp\Psr7\Response;
 use OpenTelemetry\API\LoggerHolder;
-use OpenTelemetry\Aws\Ecs\DataProvider;
-use OpenTelemetry\Aws\Ecs\Detector;
+use OpenTelemetry\Contrib\Aws\Ecs\DataProvider;
+use OpenTelemetry\Contrib\Aws\Ecs\Detector;
 use OpenTelemetry\SDK\Common\Attribute\Attributes;
 use OpenTelemetry\SDK\Resource\ResourceInfo;
 use OpenTelemetry\SDK\Resource\ResourceInfoFactory;
@@ -36,7 +36,7 @@ class DetectorTest extends TestCase
     private const ECS_ENV_VAR_V3_KEY = 'ECS_CONTAINER_METADATA_URI';
     private const ECS_ENV_VAR_V4_VAL = 'ecs_metadata_v4_uri';
     private const ECS_ENV_VAR_V3_VAL = 'ecs_metadata_v3_uri';
-    
+
     private const CLOUD_PROVIDER     = 'aws';
     private const CLOUD_PLATFORM     = 'aws_ecs';
 
diff --git a/src/Aws/tests/Unit/Eks/DataProviderTest.php b/src/Aws/tests/Unit/Eks/DataProviderTest.php
@@ -4,7 +4,7 @@
 
 namespace OpenTelemetry\Tests\Aws\Unit\Eks;
 
-use OpenTelemetry\Aws\Eks\DataProvider;
+use OpenTelemetry\Contrib\Aws\Eks\DataProvider;
 use org\bovigo\vfs\vfsStream;
 use org\bovigo\vfs\vfsStreamDirectory;
 use PHPUnit\Framework\TestCase;
diff --git a/src/Aws/tests/Unit/Eks/DetectorTest.php b/src/Aws/tests/Unit/Eks/DetectorTest.php
@@ -9,8 +9,8 @@
 use GuzzleHttp\HandlerStack;
 use GuzzleHttp\Psr7\HttpFactory;
 use GuzzleHttp\Psr7\Response;
-use OpenTelemetry\Aws\Eks\DataProvider;
-use OpenTelemetry\Aws\Eks\Detector;
+use OpenTelemetry\Contrib\Aws\Eks\DataProvider;
+use OpenTelemetry\Contrib\Aws\Eks\Detector;
 use OpenTelemetry\SDK\Common\Attribute\Attributes;
 use OpenTelemetry\SDK\Resource\ResourceInfo;
 use OpenTelemetry\SDK\Resource\ResourceInfoFactory;
diff --git a/src/Aws/tests/Unit/Lambda/DetectorTest.php b/src/Aws/tests/Unit/Lambda/DetectorTest.php
@@ -4,7 +4,7 @@
 
 namespace OpenTelemetry\Tests\Aws\Unit\Lambda;
 
-use OpenTelemetry\Aws\Lambda\Detector;
+use OpenTelemetry\Contrib\Aws\Lambda\Detector;
 use OpenTelemetry\SDK\Common\Attribute\Attributes;
 use OpenTelemetry\SDK\Resource\ResourceInfo;
 use OpenTelemetry\SDK\Resource\ResourceInfoFactory;
diff --git a/src/Aws/tests/Unit/Xray/IdGeneratorTest.php b/src/Aws/tests/Unit/Xray/IdGeneratorTest.php
@@ -5,7 +5,7 @@
 namespace OpenTelemetry\Tests\Aws\Unit\Xray;
 
 use OpenTelemetry\API\Trace\SpanContextValidator;
-use OpenTelemetry\Aws\Xray\IdGenerator;
+use OpenTelemetry\Contrib\Aws\Xray\IdGenerator;
 use PHPUnit\Framework\TestCase;
 
 class IdGeneratorTest extends TestCase
diff --git a/src/Aws/tests/Unit/Xray/PropagatorTest.php b/src/Aws/tests/Unit/Xray/PropagatorTest.php
@@ -9,9 +9,9 @@
 use OpenTelemetry\API\Trace\SpanContextValidator;
 use OpenTelemetry\API\Trace\TraceFlags;
 use OpenTelemetry\API\Trace\TraceState;
-use OpenTelemetry\Aws\Xray\Propagator;
 use OpenTelemetry\Context\Context;
 use OpenTelemetry\Context\ContextInterface;
+use OpenTelemetry\Contrib\Aws\Xray\Propagator;
 use OpenTelemetry\SDK\Trace\Span;
 use PHPUnit\Framework\TestCase;
 
diff --git a/src/Instrumentation/Doctrine/src/DoctrineInstrumentation.php b/src/Instrumentation/Doctrine/src/DoctrineInstrumentation.php
@@ -180,7 +180,7 @@ public static function register(): void
 
                 Context::storage()->attach($span->storeInContext($parent));
             },
-            post: static function (\Doctrine\DBAL\Driver\Statement $statement, array $params, ResultInterface $result, ?Throwable $exception) {
+            post: static function (\Doctrine\DBAL\Driver\Statement $statement, array $params, ?ResultInterface $result, ?Throwable $exception) {
                 self::end($exception);
             }
         );
diff --git a/src/Instrumentation/Doctrine/tests/Integration/DoctrineInstrumentationTest.php b/src/Instrumentation/Doctrine/tests/Integration/DoctrineInstrumentationTest.php
@@ -95,7 +95,7 @@ public function test_connection_exception(): void
         ]);
     }
 
-    public function test_statement_execution(): void
+    public function test_connection_execute_statement(): void
     {
         $connection =  self::createConnection();
         $statement = self::fillDB();
@@ -202,4 +202,39 @@ public function test_transaction(): void
         $sth = $connection->prepare('SELECT * FROM `technology`');
         $this->assertSame(2, count($sth->executeQuery()->fetchAllAssociative()));
     }
+
+    public function test_statement_execute(): void
+    {
+        $connection =  self::createConnection();
+        $statement = self::fillDB();
+        $connection->executeStatement($statement);
+        $stmt = $connection->prepare('SELECT * FROM `technology`');
+        $this->storage->exchangeArray([]);
+        $stmt->executeQuery();
+        $this->assertCount(1, $this->storage);
+        $span = $this->storage->offsetGet(0);
+        $this->assertSame('Doctrine::execute', $span->getName());
+        $this->assertSame('execute', $span->getAttributes()->get(TraceAttributes::DB_OPERATION_NAME));
+    }
+
+    public function test_statement_execute_error(): void
+    {
+        $connection =  self::createConnection();
+        $statement = self::fillDB();
+        $connection->executeStatement($statement);
+        $stmt = $connection->prepare('insert into technology(name, date) values (?, ?);');
+        $this->storage->exchangeArray([]);
+        $e = null;
+
+        try {
+            $stmt->executeQuery();
+        } catch (\Throwable $e) {
+            // do nothing
+        }
+        $this->assertNotNull($e);
+        $this->assertCount(1, $this->storage);
+        $span = $this->storage->offsetGet(0);
+        $this->assertSame('Error', $span->getStatus()->getCode());
+        $this->assertStringContainsString('Unable to execute', $span->getStatus()->getDescription());
+    }
 }
diff --git a/src/Instrumentation/Symfony/src/SymfonyInstrumentation.php b/src/Instrumentation/Symfony/src/SymfonyInstrumentation.php
@@ -47,8 +47,14 @@ public static function register(): void
                 $request = ($params[0] instanceof Request) ? $params[0] : null;
                 $type = $params[1] ?? HttpKernelInterface::MAIN_REQUEST;
                 $method = $request?->getMethod() ?? 'unknown';
+                $controller = $request?->attributes?->get('_controller');
+
+                if (!is_callable($controller, true, $controllerName)) {
+                    $controllerName = 'sub-request';
+                }
+
                 $name = ($type === HttpKernelInterface::SUB_REQUEST)
-                    ? sprintf('%s %s', $method, $request?->attributes?->get('_controller') ?? 'sub-request')
+                    ? sprintf('%s %s', $method, $controllerName)
                     : $method;
                 /** @psalm-suppress ArgumentTypeCoercion */
                 $builder = $instrumentation
diff --git a/src/Instrumentation/Symfony/tests/Integration/SymfonyInstrumentationTest.php b/src/Instrumentation/Symfony/tests/Integration/SymfonyInstrumentationTest.php
diff --git a/src/ResourceDetectors/Azure/_register.php b/src/ResourceDetectors/Azure/_register.php
diff --git a/src/ResourceDetectors/Azure/composer.json b/src/ResourceDetectors/Azure/composer.json
diff --git a/src/ResourceDetectors/Azure/src/AppService/Detector.php b/src/ResourceDetectors/Azure/src/AppService/Detector.php
diff --git a/src/ResourceDetectors/Azure/src/ContainerApps/Detector.php b/src/ResourceDetectors/Azure/src/ContainerApps/Detector.php
diff --git a/src/ResourceDetectors/Azure/src/Vm/Detector.php b/src/ResourceDetectors/Azure/src/Vm/Detector.php
diff --git a/src/ResourceDetectors/Azure/tests/Unit/AppService/DetectorTest.php b/src/ResourceDetectors/Azure/tests/Unit/AppService/DetectorTest.php
diff --git a/src/ResourceDetectors/Azure/tests/Unit/ContainerApps/DetectorTest.php b/src/ResourceDetectors/Azure/tests/Unit/ContainerApps/DetectorTest.php
diff --git a/src/ResourceDetectors/Azure/tests/Unit/Vm/DetectorTest.php b/src/ResourceDetectors/Azure/tests/Unit/Vm/DetectorTest.php

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@`
`33`	`33`	`},`
`34`	`34`	`"autoload": {`
`35`	`35`	`"psr-4": {`
`36`		`- "OpenTelemetry\\Aws\\": "src/"`
	`36`	`+ "OpenTelemetry\\Contrib\\Aws\\": "src/"`
`37`	`37`	`}`
`38`	`38`	`},`
`39`	`39`	`"autoload-dev": {`