Skip to content
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

DEVX-390 - PHP sdk set up for Datadog #227

Merged
merged 10 commits into from
Sep 17, 2024
Merged

DEVX-390 - PHP sdk set up for Datadog #227

merged 10 commits into from
Sep 17, 2024

Conversation

barbara79
Copy link
Contributor

  • Changeset added

Features

Fixes

Breaking changes

@barbara79 barbara79 requested a review from a team as a code owner August 28, 2024 06:18
@barbara79 barbara79 changed the title Datadog configured for PHP sdk DEVX-390 - Datadog configured for PHP sdk Aug 28, 2024
Copy link

codecov bot commented Aug 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 12.69%. Comparing base (a86074d) to head (f43b698).
Report is 11 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #227   +/-   ##
=========================================
  Coverage     12.69%   12.69%           
  Complexity   115170   115170           
=========================================
  Files          9311     9311           
  Lines        285681   285681           
=========================================
  Hits          36260    36260           
  Misses       249421   249421           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@barbara79 barbara79 changed the title DEVX-390 - Datadog configured for PHP sdk DEVX-390 - PHP sdk set up for Datadog Aug 28, 2024
@@ -0,0 +1,32 @@
# Dockerfile for New Relic
FROM php:8.2-fpm
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keeping this file because will need it in the next task

@barbara79 barbara79 requested review from a team and jenschude September 11, 2024 13:27
examples/symfony-app/README.md Outdated Show resolved Hide resolved
examples/symfony-app/README.md Outdated Show resolved Hide resolved
examples/symfony-app/README.md Outdated Show resolved Hide resolved
examples/symfony-app/README.md Outdated Show resolved Hide resolved
examples/symfony-app/README.md Show resolved Hide resolved
examples/symfony-app/docker-compose.yml Show resolved Hide resolved
Comment on lines +9 to +15
- DD_SERVICE=symfony-app
- DD_AGENT_HOST=datadog-agent
- DD_TRACE_AGENT_PORT=8126
- DD_API_KEY=${DD_API_KEY}
- DD_TRACE_DEBUG=true
- DD_LOGS_ENABLED=true
- DD_PROCESS_AGENT_ENABLED=true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So the env vars like DD_AGENT_HOST, DD_SERVICE etc. are not taken from .env file?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a good question, I tried to remove them from here but it doesn't work: some variables are taken from the .env file some of them not. The .env file is more for the symfony framework, so for the application

examples/symfony-app/docker-compose.yml Show resolved Hide resolved
@lojzatran
Copy link
Contributor

btw. did you test also for dogstatsd protocol? https://docs.datadoghq.com/developers/dogstatsd/?tab=hostagent. This was explicitly required by a customer in the past, so we had to test it as well in Java: https://commercetools.atlassian.net/browse/DEVX-317

@barbara79
Copy link
Contributor Author

btw. did you test also for dogstatsd protocol? https://docs.datadoghq.com/developers/dogstatsd/?tab=hostagent. This was explicitly required by a customer in the past, so we had to test it as well in Java: https://commercetools.atlassian.net/browse/DEVX-317

I didn't know, I will check it and test it.
Thanks :)

@barbara79
Copy link
Contributor Author

btw. did you test also for dogstatsd protocol? https://docs.datadoghq.com/developers/dogstatsd/?tab=hostagent. This was explicitly required by a customer in the past, so we had to test it as well in Java: https://commercetools.atlassian.net/browse/DEVX-317

I didn't know, I will check it and test it. Thanks :)

I'm gonna create another task for this issue, since this task wasn't included in the main one.

@barbara79 barbara79 merged commit 2a74314 into master Sep 17, 2024
19 checks passed
@barbara79 barbara79 deleted the datadog-sdk branch September 17, 2024 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants