Skip to content

Add Dockerfile to run the integration tests#58

Merged
xlamorlette-datadog merged 2 commits intomainfrom
xlamorlette/IDMPL-349-Add-Dockerfile-for-integration-tests
Apr 21, 2026
Merged

Add Dockerfile to run the integration tests#58
xlamorlette-datadog merged 2 commits intomainfrom
xlamorlette/IDMPL-349-Add-Dockerfile-for-integration-tests

Conversation

@xlamorlette-datadog
Copy link
Copy Markdown
Contributor

@xlamorlette-datadog xlamorlette-datadog commented Apr 14, 2026

This introduces a Dockerfile to run the integration tests, and it describes how to use it in test/integration-test/README.md.

By the way, add new mandatory vcr_json_body_normalizers parameter when calling ddapm_test_agent.agent.make_app().

@xlamorlette-datadog xlamorlette-datadog force-pushed the xlamorlette/IDMPL-349-Add-Dockerfile-for-integration-tests branch from 6462c64 to 543564f Compare April 17, 2026 18:26
@xlamorlette-datadog xlamorlette-datadog changed the title Add Dockerfile for integration tests Dockerfile for integration tests Apr 17, 2026
@datadog-prod-us1-3
Copy link
Copy Markdown

datadog-prod-us1-3 Bot commented Apr 17, 2026

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 51.25% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: d869125 | Docs | Datadog PR Page | Give us feedback!

@xlamorlette-datadog xlamorlette-datadog changed the title Dockerfile for integration tests Add Dockerfile to run the integration tests Apr 21, 2026
@xlamorlette-datadog xlamorlette-datadog marked this pull request as ready for review April 21, 2026 07:52
@xlamorlette-datadog xlamorlette-datadog requested a review from a team as a code owner April 21, 2026 07:52
@xlamorlette-datadog xlamorlette-datadog requested review from Anilm3 and removed request for a team April 21, 2026 07:52
@xlamorlette-datadog xlamorlette-datadog force-pushed the xlamorlette/IDMPL-349-Add-Dockerfile-for-integration-tests branch from de62862 to d869125 Compare April 21, 2026 09:16
Copy link
Copy Markdown
Contributor

@dubloom dubloom left a comment

Choose a reason for hiding this comment

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

Looks great to me, will it be used in CI ?

@pawelchcki
Copy link
Copy Markdown
Contributor

How will the httpd be compiled to run in the integration tests? Will it reuse artifacts built via clang?

@xlamorlette-datadog
Copy link
Copy Markdown
Contributor Author

@pawelchcki

How will the httpd be compiled to run in the integration tests? Will it reuse artifacts built via clang?

I'm not sure to understand correctly your question:

  • httpd is compiled within the added test/integration-test/Dockerfile (lines 27-36, via configure and make).
  • There is no dependency on any artifact built via clang.
  • The Datadog module is built via cmake (lines 50-59).

@xlamorlette-datadog xlamorlette-datadog merged commit ffdc0ff into main Apr 21, 2026
10 checks passed
@xlamorlette-datadog xlamorlette-datadog deleted the xlamorlette/IDMPL-349-Add-Dockerfile-for-integration-tests branch April 21, 2026 16:59
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