Skip to content

Commit 6a0fed0

Browse files
committed
fix remote resource identifier for new resources
1 parent 38d965c commit 6a0fed0

File tree

2 files changed

+15
-15
lines changed

2 files changed

+15
-15
lines changed

src/AWS.Distro.OpenTelemetry.AutoInstrumentation/AwsMetricAttributeGenerator.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -425,12 +425,12 @@ private static void SetRemoteResourceTypeAndIdentifier(Activity span, ActivityTa
425425
else if (IsKeyPresent(span, AttributeAWSSecretsManagerSecretArn))
426426
{
427427
remoteResourceType = NormalizedSecretsManagerServiceName + "::Secret";
428-
remoteResourceIdentifier = EscapeDelimiters((string?)span.GetTagItem(AttributeAWSSecretsManagerSecretArn));
428+
remoteResourceIdentifier = EscapeDelimiters((string?)span.GetTagItem(AttributeAWSSecretsManagerSecretArn)).Split(":").Last();
429429
}
430430
else if (IsKeyPresent(span, AttributeAWSSNSTopicArn))
431431
{
432432
remoteResourceType = NormalizedSNSServiceName + "::Topic";
433-
remoteResourceIdentifier = EscapeDelimiters((string?)span.GetTagItem(AttributeAWSSNSTopicArn));
433+
remoteResourceIdentifier = EscapeDelimiters((string?)span.GetTagItem(AttributeAWSSNSTopicArn)).Split(":").Last();
434434
}
435435
else if (IsKeyPresent(span, AttributeAWSSQSQueueName))
436436
{
@@ -445,12 +445,12 @@ private static void SetRemoteResourceTypeAndIdentifier(Activity span, ActivityTa
445445
else if (IsKeyPresent(span, AttributeAWSStepFunctionsActivityArn))
446446
{
447447
remoteResourceType = NormalizedStepFunctionsName + "::Activity";
448-
remoteResourceIdentifier = EscapeDelimiters((string?)span.GetTagItem(AttributeAWSStepFunctionsActivityArn));
448+
remoteResourceIdentifier = EscapeDelimiters((string?)span.GetTagItem(AttributeAWSStepFunctionsActivityArn)).Split(":").Last();
449449
}
450450
else if (IsKeyPresent(span, AttributeAWSStepFunctionsStateMachineArn))
451451
{
452452
remoteResourceType = NormalizedStepFunctionsName + "::StateMachine";
453-
remoteResourceIdentifier = EscapeDelimiters((string?)span.GetTagItem(AttributeAWSStepFunctionsStateMachineArn));
453+
remoteResourceIdentifier = EscapeDelimiters((string?)span.GetTagItem(AttributeAWSStepFunctionsStateMachineArn)).Split(":").Last();
454454
}
455455
else if (IsKeyPresent(span, AttributeAWSBedrockGuardrailId))
456456
{

test/contract-tests/tests/test/amazon/awssdk/awssdk_test.py

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -322,7 +322,7 @@ def test_secretsmanager_create_secret(self):
322322
remote_service="AWS::SecretsManager",
323323
remote_operation="CreateSecret",
324324
remote_resource_type="AWS::SecretsManager::Secret",
325-
remote_resource_identifier=r"arn:aws:secretsmanager:us-east-1:000000000000:secret:test-secret-[a-zA-Z0-9]{6}$",
325+
remote_resource_identifier=r"test-secret-[a-zA-Z0-9]{6}$",
326326
request_response_specific_attributes={
327327
_AWS_SECRETSMANAGER_SECRET_ARN: r"arn:aws:secretsmanager:us-east-1:000000000000:secret:test-secret-[a-zA-Z0-9]{6}$",
328328
},
@@ -340,7 +340,7 @@ def test_secretsmanager_get_secret_value(self):
340340
remote_service="AWS::SecretsManager",
341341
remote_operation="GetSecretValue",
342342
remote_resource_type="AWS::SecretsManager::Secret",
343-
remote_resource_identifier=r"arn:aws:secretsmanager:us-east-1:000000000000:secret:test-secret-[a-zA-Z0-9]{6}$",
343+
remote_resource_identifier=r"test-secret-[a-zA-Z0-9]{6}$",
344344
request_response_specific_attributes={
345345
_AWS_SECRETSMANAGER_SECRET_ARN: r"arn:aws:secretsmanager:us-east-1:000000000000:secret:test-secret-[a-zA-Z0-9]{6}$",
346346
},
@@ -358,7 +358,7 @@ def test_secretsmanager_error(self):
358358
remote_service="AWS::SecretsManager",
359359
remote_operation="DescribeSecret",
360360
remote_resource_type="AWS::SecretsManager::Secret",
361-
remote_resource_identifier="arn:aws:secretsmanager:us-east-1:000000000000:secret:test-secret-error",
361+
remote_resource_identifier="test-secret-error",
362362
request_response_specific_attributes={
363363
_AWS_SECRETSMANAGER_SECRET_ARN: "arn:aws:secretsmanager:us-east-1:000000000000:secret:test-secret-error",
364364
},
@@ -377,7 +377,7 @@ def test_secretsmanager_error(self):
377377
# remote_service="AWS::SecretsManager",
378378
# remote_operation="CreateSecret",
379379
# remote_resource_type="AWS::SecretsManager::Secret",
380-
# remote_resource_identifier="arn:aws:secretsmanager:us-east-1:000000000000:secret:test-secret-error",
380+
# remote_resource_identifier="test-secret-error",
381381
# request_response_specific_attributes={
382382
# _AWS_SECRETSMANAGER_SECRET_ARN: "arn:aws:secretsmanager:us-east-1:000000000000:secret:test-secret-error",
383383
# },
@@ -394,7 +394,7 @@ def test_sns_publish(self):
394394
remote_service="AWS::SNS",
395395
remote_operation="Publish",
396396
remote_resource_type="AWS::SNS::Topic",
397-
remote_resource_identifier="arn:aws:sns:us-east-1:000000000000:test-topic",
397+
remote_resource_identifier="test-topic",
398398
request_response_specific_attributes={
399399
_AWS_SNS_TOPIC_ARN: "arn:aws:sns:us-east-1:000000000000:test-topic",
400400
},
@@ -411,7 +411,7 @@ def test_sns_error(self):
411411
remote_service="AWS::SNS",
412412
remote_operation="Publish",
413413
remote_resource_type="AWS::SNS::Topic",
414-
remote_resource_identifier="arn:aws:sns:us-east-1:000000000000:test-topic-error",
414+
remote_resource_identifier="test-topic-error",
415415
request_response_specific_attributes={
416416
_AWS_SNS_TOPIC_ARN: "arn:aws:sns:us-east-1:000000000000:test-topic-error",
417417
},
@@ -429,7 +429,7 @@ def test_sns_error(self):
429429
# remote_service="AWS::SNS",
430430
# remote_operation="GetTopicAttributes",
431431
# remote_resource_type="AWS::SNS::Topic",
432-
# remote_resource_identifier="arn:aws:sns:us-east-1:000000000000:invalid-topic",
432+
# remote_resource_identifier="invalid-topic",
433433
# request_response_specific_attributes={
434434
# _AWS_SNS_TOPIC_ARN: "arn:aws:sns:us-east-1:000000000000:invalid-topic",},
435435
# span_name="SNS.GetTopicAttributes"
@@ -446,7 +446,7 @@ def test_stepfunctions_describe_state_machine(self):
446446
remote_service="AWS::StepFunctions",
447447
remote_operation="DescribeStateMachine",
448448
remote_resource_type="AWS::StepFunctions::StateMachine",
449-
remote_resource_identifier="arn:aws:states:us-east-1:000000000000:stateMachine:test-state-machine",
449+
remote_resource_identifier="test-state-machine",
450450
request_response_specific_attributes={
451451
_AWS_STEPFUNCTIONS_STATE_MACHINE_ARN: "arn:aws:states:us-east-1:000000000000:stateMachine:test-state-machine",
452452
},
@@ -464,7 +464,7 @@ def test_stepfunctions_describe_activity(self):
464464
remote_service="AWS::StepFunctions",
465465
remote_operation="DescribeActivity",
466466
remote_resource_type="AWS::StepFunctions::Activity",
467-
remote_resource_identifier="arn:aws:states:us-east-1:000000000000:activity:test-activity",
467+
remote_resource_identifier="test-activity",
468468
request_response_specific_attributes={
469469
_AWS_STEPFUNCTIONS_ACTIVITY_ARN: "arn:aws:states:us-east-1:000000000000:activity:test-activity",
470470
},
@@ -482,7 +482,7 @@ def test_stepfunctions_error(self):
482482
remote_service="AWS::StepFunctions",
483483
remote_operation="DescribeStateMachine",
484484
remote_resource_type="AWS::StepFunctions::StateMachine",
485-
remote_resource_identifier="arn:aws:states:us-east-1:000000000000:stateMachine:error-state-machine",
485+
remote_resource_identifier="error-state-machine",
486486
request_response_specific_attributes={
487487
_AWS_STEPFUNCTIONS_STATE_MACHINE_ARN: "arn:aws:states:us-east-1:000000000000:stateMachine:error-state-machine",
488488
},
@@ -502,7 +502,7 @@ def test_stepfunctions_error(self):
502502
# remote_service="AWS::StepFunctions",
503503
# remote_operation="ListStateMachineVersions",
504504
# remote_resource_type="AWS::StepFunctions::StateMachine",
505-
# remote_resource_identifier="arn:aws:states:us-east-1:000000000000:stateMachine:invalid-state-machine",
505+
# remote_resource_identifier="invalid-state-machine",
506506
# request_response_specific_attributes={
507507
# _AWS_STEPFUNCTIONS_STATE_MACHINE_ARN: "arn:aws:states:us-east-1:000000000000:stateMachine:invalid-state-machine",},
508508
# span_name="SFN.ListStateMachineVersions",

0 commit comments

Comments
 (0)