-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add EKS integration testing for APM Traces #327
Changes from all commits
53295cc
88a63c1
ff6e726
099d960
1af68f8
e942c2c
ddfaa4f
11e2a50
df63334
31f005d
0aabd78
f92f866
ee1a465
e309dab
da2f6e7
f0b6d7c
9f55f2e
74a90ae
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -162,7 +162,7 @@ | |
{ | ||
"key": "aws.span.kind", | ||
"value": { | ||
"stringValue": "CONSUMER" | ||
"stringValue": "SERVER" | ||
} | ||
}, | ||
{ | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
package main | ||
|
||
import ( | ||
"crypto/rand" | ||
"encoding/binary" | ||
"encoding/hex" | ||
"fmt" | ||
"time" | ||
) | ||
|
||
func main() { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I understand that this is probably to avoid dependencies, but https://github.com/open-telemetry/opentelemetry-go-contrib/tree/main/propagators/aws/xray provides an ID generator, which is what customers would be using as well. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. oh nice! I can look into this |
||
var r [16]byte | ||
epochNow := time.Now().Unix() | ||
binary.BigEndian.PutUint32(r[0:4], uint32(epochNow)) | ||
rand.Read(r[4:]) | ||
fmt.Printf("%s", hex.EncodeToString(r[:])) | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,90 @@ | ||
{ | ||
"resourceSpans": [ | ||
{ | ||
"resource": { | ||
"attributes": [ | ||
{ | ||
"key": "k8s.namespace.name", | ||
"value": { | ||
"stringValue": "default" | ||
} | ||
}, | ||
{ | ||
"key": "k8s.pod.name", | ||
"value": { | ||
"stringValue": "pod-name" | ||
} | ||
}, | ||
{ | ||
"key": "aws.deployment.name", | ||
"value": { | ||
"stringValue": "deployment-name" | ||
} | ||
}, | ||
{ | ||
"key": "host.id", | ||
"value": { | ||
"stringValue": "i-00000000000000000" | ||
} | ||
} | ||
] | ||
}, | ||
"scopeSpans": [ | ||
{ | ||
"scope": { | ||
"name": "apm-integration-test" | ||
}, | ||
"spans": [ | ||
{ | ||
"traceId": "TRACE_ID", | ||
"spanId": "EEE19B7EC3C1B174", | ||
"parentSpanId": "EEE19B7EC3C1B173", | ||
"name": "apm-integration-test-traces", | ||
"startTimeUnixNano": START_TIME, | ||
"endTimeUnixNano": START_TIME, | ||
"kind": 2, | ||
"attributes": [ | ||
{ | ||
"key": "aws.span.kind", | ||
"value": { | ||
"stringValue": "CLIENT" | ||
} | ||
}, | ||
{ | ||
"key": "aws.local.operation", | ||
"value": { | ||
"stringValue": "operation" | ||
} | ||
}, | ||
{ | ||
"key": "aws.local.service", | ||
"value": { | ||
"stringValue": "service-name" | ||
} | ||
}, | ||
{ | ||
"key": "aws.remote.operation", | ||
"value": { | ||
"stringValue": "remote-operation" | ||
} | ||
}, | ||
{ | ||
"key": "aws.remote.service", | ||
"value": { | ||
"stringValue": "service-name-remote" | ||
} | ||
}, | ||
{ | ||
"key": "aws.remote.target", | ||
"value": { | ||
"stringValue": "remote-target" | ||
} | ||
} | ||
] | ||
} | ||
] | ||
} | ||
] | ||
} | ||
] | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: Use local_file instead of template_file if it isn't templating anything.