update instrumentations to semconv 1.32 (#369)

SemConv 1.32 deprecates a couple of attributes which we use extensively:
- code.function.name is now an FQN (including class name)
- code.namespace is deprecated (included in code.function.name)
- code.linenumber is deprecated and replaced with code.line.number

Updated all packages to conform, updated min semconv to 1.32, and made
a couple of minor changes"

Author: brettmc

composer.json

@@ -13,7 +13,7 @@
     "ext-opentelemetry": "*",
     "ext-reflection": "*",
     "open-telemetry/api": "^1.0",
-    "open-telemetry/sem-conv": "^1.30",
+    "open-telemetry/sem-conv": "^1.32",
     "slim/slim": "^4"
   },
   "require-dev": {

src/SlimInstrumentation.php

@@ -33,7 +33,7 @@ public static function register(): void
         $instrumentation = new CachedInstrumentation(
             'io.opentelemetry.contrib.php.slim',
             null,
-            'https://opentelemetry.io/schemas/1.30.0',
+            'https://opentelemetry.io/schemas/1.32.0',
         );
 
         /**
@@ -53,9 +53,8 @@ public static function register(): void
                 $builder = $instrumentation->tracer()
                     ->spanBuilder(sprintf('%s', $request?->getMethod() ?? 'unknown'))
                     ->setSpanKind(SpanKind::KIND_SERVER)
-                    ->setAttribute(TraceAttributes::CODE_FUNCTION_NAME, $function)
-                    ->setAttribute(TraceAttributes::CODE_NAMESPACE, $class)
-                    ->setAttribute(TraceAttributes::CODE_FILEPATH, $filename)
+                    ->setAttribute(TraceAttributes::CODE_FUNCTION_NAME, sprintf('%s::%s', $class, $function))
+                    ->setAttribute(TraceAttributes::CODE_FILE_PATH, $filename)
                     ->setAttribute(TraceAttributes::CODE_LINE_NUMBER, $lineno);
                 $parent = Context::getCurrent();
                 if ($request) {
@@ -124,6 +123,8 @@ public static function register(): void
          * and type SpanInterface which represents the root span, having been previously set
          * If routing fails (eg 404/not found), then the root span name will not be updated.
          *
+         * @todo this can use LocalRootSpan (available since API 1.1.0)
+         *
          * @psalm-suppress ArgumentTypeCoercion
          * @psalm-suppress UnusedFunctionCall
          */
@@ -161,9 +162,8 @@ public static function register(): void
                 $callable = $params[0];
                 $name = CallableFormatter::format($callable);
                 $builder = $instrumentation->tracer()->spanBuilder($name)
-                    ->setAttribute(TraceAttributes::CODE_FUNCTION_NAME, $function)
-                    ->setAttribute(TraceAttributes::CODE_NAMESPACE, $class)
-                    ->setAttribute(TraceAttributes::CODE_FILEPATH, $filename)
+                    ->setAttribute(TraceAttributes::CODE_FUNCTION_NAME, sprintf('%s::%s', $class, $function))
+                    ->setAttribute(TraceAttributes::CODE_FILE_PATH, $filename)
                     ->setAttribute(TraceAttributes::CODE_LINE_NUMBER, $lineno);
                 $span = $builder->startSpan();
                 Context::storage()->attach($span->storeInContext(Context::getCurrent()));