Skip to content

Add Datadog Synthetic tests workflow - #244

Open
onejjm001-cell wants to merge 1 commit into
1inch:masterfrom
onejjm001-cell:master
Open

onejjm001-cell wants to merge 1 commit into
1inch:masterfrom
onejjm001-cell:master

Conversation

@onejjm001-cell

@onejjm001-cell onejjm001-cell commented Sep 25, 2026 •

Copy link
Copy Markdown

This workflow triggers Datadog Synthetic tests on push and pull request events to the master branch, using specified API and application keys.

Change Summary

What does this PR change?

Related Issue/Ticket:

Testing & Verification

How was this tested?

  • Unit tests
  • Integration tests
  • Manual testing (describe steps)
  • Verified on staging

Risk Assessment

Risk Level:

  • Low - Minor changes, no operational impact
  • Medium - Moderate changes, limited impact, standard rollback available
  • High - Significant changes, potential operational impact, complex rollback

Risks & Impact


Note

Low Risk
CI-only addition; merge/push builds can fail if secrets are missing or tagged synthetics fail, and the workflow relies on a third-party Datadog action.

Overview
Adds a new GitHub Actions workflow that runs Datadog Synthetic checks on every push and pull request targeting master.

The job checks out the repo and invokes the pinned DataDog/synthetics-ci-github-action (v1.4.0), using repository secrets DD_API_KEY and DD_APP_KEY, and only runs synthetics matching tag:e2e-tests. This wires e2e synthetics into CI without changing application code.

Reviewed by Cursor Bugbot for commit 0f79437. Bugbot is set up for automated code reviews on this repo. Configure here.

This workflow triggers Datadog Synthetic tests on push and pull request events to the master branch, using specified API and application keys.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit 0f79437. Configure here.

push:
branches: [ "master" ]
pull_request:
branches: [ "master" ]

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fork PRs fail without secrets

Medium Severity

The pull_request job always calls the Datadog action with DD_API_KEY and DD_APP_KEY. GitHub withholds repository secrets from fork pull requests, so those runs get empty keys and the action fails. External contributor PRs then show a failing check.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 0f79437. Configure here.

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.

1 participant