[smithy-rs] remove redundant span attributes and improve log output format (#4075)

## Description
smithy-lang/smithy-rs#4052 aligned our spans
closer to SRA but the output format resulted in `rpc.service` and
`rpc.method` on both the new generated operation span and the internal
orchestrator `invoke` span. This resulted in pretty long and clutter log
output. This PR cleans up the log output a bit to have only the top
level operation span include these attributes. This also fixes a subtle
bug where presigned requests wouldn't contain the same top level
operation span instrumentation because presigned requests make use of
`<Operation>::orchestrate_with_stop_point` and not
`<Operation>::orchestrate`. For IMDS and other "manually constructed"
operations, I've moved the `invoke` span instrumentation into
`Operation::invoke` to ensure we still capture that information in the
logs for those use cases.

Example output format:
```
2025-03-28T14:44:02.471068Z DEBUG S3.ListBuckets{rpc.service="S3" rpc.method="ListBuckets" sdk_invocation_id=4073472 rpc.system="aws-api"}:try_op: aws_smithy_runtime::cli
ent::orchestrator: beginning attempt #1
```

## Checklist
<!--- If a checkbox below is not applicable, then please DELETE it
rather than leaving it unchecked -->
- [x] For changes to the smithy-rs codegen or runtime crates, I have
created a changelog entry Markdown file in the `.changelog` directory,
specifying "client," "server," or both in the `applies_to` key.

----

_By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice._

Author: aajtodd

examples/cross_service/photo_asset_management/Cargo.toml

@@ -35,7 +35,7 @@ aws-sdk-dynamodb= { version = "1.72.0", path = "../../../sdk/dynamodb" }
 aws-sdk-rekognition= { version = "1.68.0", path = "../../../sdk/rekognition" }
 aws-sdk-s3= { version = "1.83.0", path = "../../../sdk/s3" }
 aws-sdk-sns= { version = "1.66.0", path = "../../../sdk/sns" }
-aws-smithy-runtime= { version = "1.8.1", path = "../../../sdk/aws-smithy-runtime" }
+aws-smithy-runtime= { version = "1.8.2", path = "../../../sdk/aws-smithy-runtime" }
 aws-smithy-types-convert= { version = "0.60.9", path = "../../../sdk/aws-smithy-types-convert", features = ["convert-chrono"] }
 
 [dependencies.aws_lambda_events]

examples/cross_service/rest_ses/Cargo.toml

@@ -34,7 +34,7 @@ aws-sdk-cloudwatchlogs= { version = "1.77.0", path = "../../../sdk/cloudwatchlog
 aws-sdk-rdsdata= { version = "1.67.0", path = "../../../sdk/rdsdata" }
 aws-sdk-ses= { version = "1.68.0", path = "../../../sdk/ses" }
 aws-smithy-types= { version = "1.3.0", path = "../../../sdk/aws-smithy-types" }
-aws-smithy-runtime= { version = "1.8.1", path = "../../../sdk/aws-smithy-runtime" }
+aws-smithy-runtime= { version = "1.8.2", path = "../../../sdk/aws-smithy-runtime" }
 
 [dependencies.chrono]
 version = "0.4.22"
@@ -69,7 +69,7 @@ features = ["v4", "serde"]
 once_cell = "1.15.0"
 rand = "0.8.5"
 wiremock = "0.5.15"
-aws-smithy-runtime= { version = "1.8.1", path = "../../../sdk/aws-smithy-runtime" }
+aws-smithy-runtime= { version = "1.8.2", path = "../../../sdk/aws-smithy-runtime" }
 
 [dev-dependencies.fake]
 version = "2.5.0"

examples/examples/custom-root-certificates/Cargo.toml

@@ -11,7 +11,7 @@ publish = false
 rustls = "0.21.9"
 aws-config= { version = "1.6.1", path = "../../../sdk/aws-config", features = ["behavior-version-latest"] }
 aws-sdk-s3= { version = "1.83.0", path = "../../../sdk/s3", default-features = false }
-aws-smithy-runtime= { version = "1.8.1", path = "../../../sdk/aws-smithy-runtime", features = ["tls-rustls"] }
+aws-smithy-runtime= { version = "1.8.2", path = "../../../sdk/aws-smithy-runtime", features = ["tls-rustls"] }
 
 [dependencies.tokio]
 version = "1.21.2"

examples/examples/dynamodb/Cargo.toml

@@ -19,7 +19,7 @@ tracing = "0.1"
 aws-config= { version = "1.6.1", path = "../../../sdk/aws-config", features = ["behavior-version-latest"] }
 aws-http= { version = "0.60.6", path = "../../../sdk/aws-http" }
 aws-sdk-dynamodb= { version = "1.72.0", path = "../../../sdk/dynamodb" }
-aws-smithy-runtime= { version = "1.8.1", path = "../../../sdk/aws-smithy-runtime", features = ["test-util"] }
+aws-smithy-runtime= { version = "1.8.2", path = "../../../sdk/aws-smithy-runtime", features = ["test-util"] }
 aws-smithy-types= { version = "1.3.0", path = "../../../sdk/aws-smithy-types" }
 
 [dependencies.clap]

examples/examples/s3/Cargo.toml

@@ -25,7 +25,7 @@ serde_json = "1"
 chrono = "0.4.38"
 aws-config= { version = "1.6.1", path = "../../../sdk/aws-config", features = ["behavior-version-latest"] }
 aws-sdk-s3= { version = "1.83.0", path = "../../../sdk/s3", features = ["rt-tokio"] }
-aws-smithy-runtime= { version = "1.8.1", path = "../../../sdk/aws-smithy-runtime" }
+aws-smithy-runtime= { version = "1.8.2", path = "../../../sdk/aws-smithy-runtime" }
 aws-smithy-runtime-api= { version = "1.7.4", path = "../../../sdk/aws-smithy-runtime-api", features = ["client"] }
 aws-smithy-types= { version = "1.3.0", path = "../../../sdk/aws-smithy-types", features = ["http-body-0-4-x"] }
 

examples/examples/testing/Cargo.toml

@@ -19,7 +19,7 @@ aws-config= { version = "1.6.1", path = "../../../sdk/aws-config", features = ["
 aws-credential-types= { version = "1.2.2", path = "../../../sdk/aws-credential-types", features = ["hardcoded-credentials"] }
 aws-sdk-s3= { version = "1.83.0", path = "../../../sdk/s3" }
 aws-smithy-types= { version = "1.3.0", path = "../../../sdk/aws-smithy-types" }
-aws-smithy-runtime= { version = "1.8.1", path = "../../../sdk/aws-smithy-runtime", features = ["test-util"] }
+aws-smithy-runtime= { version = "1.8.2", path = "../../../sdk/aws-smithy-runtime", features = ["test-util"] }
 aws-smithy-runtime-api= { version = "1.7.4", path = "../../../sdk/aws-smithy-runtime-api", features = ["test-util"] }
 aws-types= { version = "1.3.6", path = "../../../sdk/aws-types" }
 

examples/examples/tls/Cargo.toml

@@ -17,7 +17,7 @@ path = "src/lib.rs"
 webpki-roots = "0.22.4"
 rustls = "0.20.6"
 aws-config= { version = "1.6.1", path = "../../../sdk/aws-config", features = ["behavior-version-latest"] }
-aws-smithy-runtime= { version = "1.8.1", path = "../../../sdk/aws-smithy-runtime" }
+aws-smithy-runtime= { version = "1.8.2", path = "../../../sdk/aws-smithy-runtime" }
 aws-sdk-kms= { version = "1.66.0", path = "../../../sdk/kms", default-features = false }
 
 [dependencies.tokio]

examples/test-utils/Cargo.toml

@@ -13,7 +13,7 @@ http = "0.2"
 tokio = "1.33.0"
 aws-config= { version = "1.6.1", path = "../../sdk/aws-config" }
 aws-smithy-types= { version = "1.3.0", path = "../../sdk/aws-smithy-types" }
-aws-smithy-runtime= { version = "1.8.1", path = "../../sdk/aws-smithy-runtime", features = ["test-util"] }
+aws-smithy-runtime= { version = "1.8.2", path = "../../sdk/aws-smithy-runtime", features = ["test-util"] }
 aws-types= { version = "1.3.6", path = "../../sdk/aws-types" }
 
 [lib]

sdk/accessanalyzer/Cargo.toml

@@ -37,7 +37,7 @@ version = "0.61.3"
 [dependencies.aws-smithy-runtime]
 path = "../aws-smithy-runtime"
 features = ["client"]
-version = "1.8.1"
+version = "1.8.2"
 
 [dependencies.aws-smithy-runtime-api]
 path = "../aws-smithy-runtime-api"

sdk/accessanalyzer/src/operation/apply_archive_rule.rs

@@ -27,17 +27,7 @@ impl ApplyArchiveRule {
                     .expect("correct error type")
             })
         };
-        use ::tracing::Instrument;
         let context = Self::orchestrate_with_stop_point(runtime_plugins, input, ::aws_smithy_runtime::client::orchestrator::StopPoint::None)
-            // Create a parent span for the entire operation. Includes a random, internal-only,
-            // seven-digit ID for the operation orchestration so that it can be correlated in the logs.
-            .instrument(::tracing::debug_span!(
-                "AccessAnalyzer.ApplyArchiveRule",
-                "rpc.service" = "AccessAnalyzer",
-                "rpc.method" = "ApplyArchiveRule",
-                "sdk_invocation_id" = ::fastrand::u32(1_000_000..10_000_000),
-                "rpc.system" = "aws-api",
-            ))
             .await
             .map_err(map_err)?;
         let output = context.finalize().map_err(map_err)?;
@@ -60,7 +50,17 @@ impl ApplyArchiveRule {
         >,
     > {
         let input = ::aws_smithy_runtime_api::client::interceptors::context::Input::erase(input);
+        use ::tracing::Instrument;
         ::aws_smithy_runtime::client::orchestrator::invoke_with_stop_point("AccessAnalyzer", "ApplyArchiveRule", input, runtime_plugins, stop_point)
+            // Create a parent span for the entire operation. Includes a random, internal-only,
+            // seven-digit ID for the operation orchestration so that it can be correlated in the logs.
+            .instrument(::tracing::debug_span!(
+                "AccessAnalyzer.ApplyArchiveRule",
+                "rpc.service" = "AccessAnalyzer",
+                "rpc.method" = "ApplyArchiveRule",
+                "sdk_invocation_id" = ::fastrand::u32(1_000_000..10_000_000),
+                "rpc.system" = "aws-api",
+            ))
             .await
     }