-
Notifications
You must be signed in to change notification settings - Fork 277
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
Send data to trace agent when no API key is given #1543
Conversation
...ofiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/ProfileUploader.java
Show resolved
Hide resolved
5fd590b
to
54f4c15
Compare
...ofiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/ProfileUploader.java
Show resolved
Hide resolved
...profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/ContainerInfo.java
Outdated
Show resolved
Hide resolved
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.
Could you please update tests for ProfileUpload and consider moving out container stuff into separate PR.
aaa8512
to
28a83c9
Compare
Sorry for the noise, I struggled to have this working end to end in my development environment and I wanted to test both changes (allow missing api key and add container id header). I've removed the container ID changes and will open another PR for that. @AlexJF helped me understand the agent setup in our devenv and we were able to see data coming through successfully for applications with no API key and with this property set:
|
Related PR (container IDs): #1569 |
28a83c9
to
1e88bb8
Compare
1e88bb8
to
1253cce
Compare
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.
LGTM, assuming build passes
dd-java-agent/agent-profiling/src/main/java/com/datadog/profiling/agent/ProfilingAgent.java
Outdated
Show resolved
Hide resolved
...ofiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/ProfileUploader.java
Show resolved
Hide resolved
1253cce
to
8c63d72
Compare
...ofiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/ProfileUploader.java
Show resolved
Hide resolved
edcf348
to
05d1d98
Compare
05d1d98
to
35734f7
Compare
No description provided.