Skip to content

Add support for Synthetics Network tests#473

Merged
melkouri merged 2 commits intomainfrom
malak.elkouri/SYNTH-24620/support-network-tests
Mar 17, 2026
Merged

Add support for Synthetics Network tests#473
melkouri merged 2 commits intomainfrom
malak.elkouri/SYNTH-24620/support-network-tests

Conversation

@melkouri
Copy link
Contributor

@melkouri melkouri commented Mar 12, 2026

Requirements for Contributing to this repository

  • Fill out the template below. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
  • The pull request must only fix one issue, or add one feature, at the time.
  • The pull request must update the test suite to demonstrate the changed functionality.
  • After you create the pull request, all status checks must be pass before a maintainer reviews your contribution. For more details, please see CONTRIBUTING.

What does this PR do?

This PR adds support for Synthetics Network tests.
Used public API https://docs.datadoghq.com/api/latest/synthetics/#create-a-network-path-test

Description of the Change

Verification Process

Network tests successfuly replicated in R2 (see DDR org in US5) from R1 (US1 datadog2 org)
image

Additional Notes

Release Notes

Review checklist (to be filled by reviewers)

  • Feature or bug fix MUST have appropriate tests (unit, integration, etc...)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have one changelog/ label attached. If applicable it should have the backward-incompatible label attached.
  • PR should not have do-not-merge/ label attached.
  • If Applicable, issue must have kind/ and severity/ labels attached at least.

Comment on lines +87 to +90
# v2 API does not return monitor_id despite the docs listing it
resource.setdefault("monitor_id", None)
# v2 API returns monitor_priority as string but the API expects an integer
monitor_priority = resource.get("options", {}).get("monitor_priority")
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 should be removed once the network test endpoint is fixed
ticket https://datadoghq.atlassian.net/browse/SYNTH-25321

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@melkouri melkouri changed the title add network tests Add support for Synthetics Network tests Mar 16, 2026
@melkouri melkouri marked this pull request as ready for review March 16, 2026 16:20
@melkouri melkouri requested a review from a team as a code owner March 16, 2026 16:20
@melkouri melkouri merged commit 10fe9d2 into main Mar 17, 2026
21 of 22 checks passed
@melkouri melkouri deleted the malak.elkouri/SYNTH-24620/support-network-tests branch March 17, 2026 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants