Skip to content

Commit 0d97331

Browse files
committed
Move string literals to AWS Semnatic Conventions class
1 parent 053fc35 commit 0d97331

File tree

5 files changed

+14
-9
lines changed

5 files changed

+14
-9
lines changed

src/OpenTelemetry.Resources.AWS/AWSEBSDetector.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,9 @@ internal static List<KeyValuePair<string, object>> ExtractResourceAttributes(AWS
5656
{
5757
var resourceAttributes = new List<KeyValuePair<string, object>>()
5858
{
59-
new(AWSSemanticConventions.AttributeCloudProvider, "aws"),
60-
new(AWSSemanticConventions.AttributeCloudPlatform, "aws_elastic_beanstalk"),
61-
new(AWSSemanticConventions.AttributeServiceName, "aws_elastic_beanstalk"),
59+
new(AWSSemanticConventions.AttributeCloudProvider, AWSSemanticConventions.CloudProviderValuesAws),
60+
new(AWSSemanticConventions.AttributeCloudPlatform, AWSSemanticConventions.CloudPlatformValuesAwsElasticBeanstalk),
61+
new(AWSSemanticConventions.AttributeServiceName, AWSSemanticConventions.ServiceNameValuesAwsElasticBeanstalk),
6262
};
6363

6464
if (metadata != null)

src/OpenTelemetry.Resources.AWS/AWSEC2Detector.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,8 @@ internal static List<KeyValuePair<string, object>> ExtractResourceAttributes(AWS
4545
{
4646
var resourceAttributes = new List<KeyValuePair<string, object>>()
4747
{
48-
new(AWSSemanticConventions.AttributeCloudProvider, "aws"),
49-
new(AWSSemanticConventions.AttributeCloudPlatform, "aws_ec2"),
48+
new(AWSSemanticConventions.AttributeCloudProvider, AWSSemanticConventions.CloudProviderValuesAws),
49+
new(AWSSemanticConventions.AttributeCloudPlatform, AWSSemanticConventions.CloudPlatformValuesAwsEc2),
5050
new(AWSSemanticConventions.AttributeHostName, hostName),
5151
};
5252

src/OpenTelemetry.Resources.AWS/AWSECSDetector.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,8 @@ public Resource Detect()
2929

3030
var resourceAttributes = new List<KeyValuePair<string, object>>()
3131
{
32-
new(AWSSemanticConventions.AttributeCloudProvider, "aws"),
33-
new(AWSSemanticConventions.AttributeCloudPlatform, "aws_ecs"),
32+
new(AWSSemanticConventions.AttributeCloudProvider, AWSSemanticConventions.CloudProviderValuesAws),
33+
new(AWSSemanticConventions.AttributeCloudPlatform, AWSSemanticConventions.CloudPlatformValuesAwsEcs),
3434
};
3535

3636
try

src/OpenTelemetry.Resources.AWS/AWSEKSDetector.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@ internal static List<KeyValuePair<string, object>> ExtractResourceAttributes(str
3838
{
3939
var resourceAttributes = new List<KeyValuePair<string, object>>()
4040
{
41-
new(AWSSemanticConventions.AttributeCloudProvider, "aws"),
42-
new(AWSSemanticConventions.AttributeCloudPlatform, "aws_eks"),
41+
new(AWSSemanticConventions.AttributeCloudProvider, AWSSemanticConventions.CloudProviderValuesAws),
42+
new(AWSSemanticConventions.AttributeCloudPlatform, AWSSemanticConventions.CloudPlatformValuesAwsEks),
4343
};
4444

4545
if (!string.IsNullOrEmpty(clusterName))

src/OpenTelemetry.Resources.AWS/AWSSemanticConventions.cs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,11 @@ internal static class AWSSemanticConventions
1111
public const string AttributeCloudProvider = "cloud.provider";
1212
public const string AttributeCloudRegion = "cloud.region";
1313
public const string AttributeCloudResourceId = "cloud.resource_id";
14+
public const string CloudPlatformValuesAwsEc2 = "aws_ec2";
15+
public const string CloudPlatformValuesAwsEcs = "aws_ecs";
16+
public const string CloudPlatformValuesAwsEks = "aws_eks";
17+
public const string CloudPlatformValuesAwsElasticBeanstalk = "aws_elastic_beanstalk";
18+
public const string CloudProviderValuesAws = "aws";
1419

1520
public const string AttributeContainerID = "container.id";
1621

0 commit comments

Comments
 (0)