observability naming adjustments (#537)

codegen/smithy-go-codegen/src/main/java/software/amazon/smithy/go/codegen/ServiceGenerator.java

@@ -398,14 +398,14 @@ defer endTimer()
                     decorated := middleware.DecorateHandler(handler, stack)
                     result, metadata, err = decorated.Handle(ctx, params)
                     if err != nil {
-                        span.SetProperty("error.go.type", fmt.Sprintf("%T", err))
-                        span.SetProperty("error.go.error", err.Error())
+                        span.SetProperty("exception.type", fmt.Sprintf("%T", err))
+                        span.SetProperty("exception.message", err.Error())
 
                         var aerr smithy.APIError
                         if $errors.As:T(err, &aerr) {
-                            span.SetProperty("error.api.code", aerr.ErrorCode())
-                            span.SetProperty("error.api.message", aerr.ErrorMessage())
-                            span.SetProperty("error.api.fault", aerr.ErrorFault().String())
+                            span.SetProperty("api.error_code", aerr.ErrorCode())
+                            span.SetProperty("api.error_message", aerr.ErrorMessage())
+                            span.SetProperty("api.error_fault", aerr.ErrorFault().String())
                         }
 
                         err = &$operationError:T{

codegen/smithy-go-codegen/src/main/java/software/amazon/smithy/go/codegen/auth/ResolveAuthSchemeMiddlewareGenerator.java

@@ -83,7 +83,7 @@ private GoWriter.Writable generateBody() {
 
                 ctx = setResolvedAuthScheme(ctx, scheme)
 
-                span.SetProperty("operation.auth.resolved_scheme_id", scheme.Scheme.SchemeID())
+                span.SetProperty("auth.scheme_id", scheme.Scheme.SchemeID())
                 span.End()
                 return next.HandleFinalize(ctx, in)
                 """,

codegen/smithy-go-codegen/src/main/java/software/amazon/smithy/go/codegen/endpoints/EndpointMiddlewareGenerator.java

@@ -141,7 +141,7 @@ private GoWriter.Writable generateResolveEndpoint() {
                     return out, metadata, $fmt.Errorf:T("failed to resolve service endpoint, %w", err)
                 }
 
-                span.SetProperty("operation.resolved_endpoint", endpt.URI.String())
+                span.SetProperty("client.call.resolved_endpoint", endpt.URI.String())
 
                 if endpt.URI.RawPath == "" && req.URL.RawPath != "" {
                     endpt.URI.RawPath = endpt.URI.Path

metrics/smithy-otel-metrics/doc.go → metrics/smithyotelmetrics/doc.go

@@ -11,7 +11,7 @@
 //	import (
 //		"github.com/aws/aws-sdk-go-v2/config"
 //		"github.com/aws/aws-sdk-go-v2/service/s3"
-//		smithyotelmetrics "github.com/aws/smithy-go/metrics/smithy-otel-metrics"
+//		"github.com/aws/smithy-go/metrics/smithyotelmetrics"
 //		"go.opentelemetry.io/otel/sdk/metric"
 //		"go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp"
 //	)

metrics/smithy-otel-metrics/go.mod → metrics/smithyotelmetrics/go.mod

@@ -1,4 +1,4 @@
-module github.com/aws/smithy-go/metrics/smithy-otel-metrics
+module github.com/aws/smithy-go/metrics/smithyotelmetrics
 
 go 1.22
 

tracing/smithy-otel-tracing/doc.go → tracing/smithyoteltracing/doc.go

@@ -11,7 +11,7 @@
 //	import (
 //		"github.com/aws/aws-sdk-go-v2/config"
 //		"github.com/aws/aws-sdk-go-v2/service/s3"
-//		smithyoteltracing "github.com/aws/smithy-go/tracing/smithy-otel-tracing"
+//		"github.com/aws/smithy-go/tracing/smithyoteltracing"
 //		"go.opentelemetry.io/otel/exporters/stdout/stdouttrace"
 //		"go.opentelemetry.io/otel/sdk/trace"
 //	)

tracing/smithy-otel-tracing/go.mod → tracing/smithyoteltracing/go.mod

@@ -1,4 +1,4 @@
-module github.com/aws/smithy-go/tracing/smithy-otel-tracing
+module github.com/aws/smithy-go/tracing/smithyoteltracing
 
 go 1.22
 

transport/http/client.go

@@ -56,14 +56,14 @@ func (c ClientHandler) Handle(ctx context.Context, input interface{}) (
 		return nil, metadata, err
 	}
 
-	span.SetProperty("http.request.method", req.Method)
-	span.SetProperty("http.request.content_length", -1) // at least indicate unknown
+	span.SetProperty("http.method", req.Method)
+	span.SetProperty("http.request_content_length", -1) // at least indicate unknown
 	length, ok, err := req.StreamLength()
 	if err != nil {
 		return nil, metadata, err
 	}
 	if ok {
-		span.SetProperty("http.request.content_length", length)
+		span.SetProperty("http.request_content_length", length)
 	}
 
 	resp, err := c.client.Do(builtRequest)
@@ -93,9 +93,9 @@ func (c ClientHandler) Handle(ctx context.Context, input interface{}) (
 		_ = builtRequest.Body.Close()
 	}
 
-	span.SetProperty("http.proto", fmt.Sprintf("%d.%d", resp.ProtoMajor, resp.ProtoMinor))
-	span.SetProperty("http.response.status_code", resp.StatusCode)
-	span.SetProperty("http.response.content_length", resp.ContentLength)
+	span.SetProperty("net.protocol.version", fmt.Sprintf("%d.%d", resp.ProtoMajor, resp.ProtoMinor))
+	span.SetProperty("http.status_code", resp.StatusCode)
+	span.SetProperty("http.response_content_length", resp.ContentLength)
 
 	return &Response{Response: resp}, metadata, err
 }