Skip to content

Fix Datadog Synthetics CI workflow failing on missing secrets#6

Open
Copilot wants to merge 2 commits into
copilot/update-project-info-repofrom
copilot/fix-commit-checks-issue
Open

Fix Datadog Synthetics CI workflow failing on missing secrets#6
Copilot wants to merge 2 commits into
copilot/update-project-info-repofrom
copilot/fix-commit-checks-issue

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 7, 2026

The Run Datadog Synthetic tests workflow hard-fails on every PR because DD_API_KEY / DD_APP_KEY secrets are not configured, blocking CI with an unrecoverable action error.

Changes

  • Guard step added: checks both secrets via env vars before running Datadog tests
  • Conditional execution: test step gated on has_secrets == 'true'; emits a ::warning:: and skips cleanly when secrets are absent
  • No behavior change when secrets are properly configured
- name: Check for required Datadog secrets
  id: check-secrets
  env:
    DD_API_KEY: ${{ secrets.DD_API_KEY }}
    DD_APP_KEY: ${{ secrets.DD_APP_KEY }}
  run: |
    if [ -n "$DD_API_KEY" ] && [ -n "$DD_APP_KEY" ]; then
      echo "has_secrets=true" >> $GITHUB_OUTPUT
    else
      echo "has_secrets=false" >> $GITHUB_OUTPUT
      echo "::warning::DD_API_KEY and/or DD_APP_KEY secrets are not configured. Skipping Datadog Synthetic tests."
    fi

- name: Run Datadog Synthetic tests
  if: steps.check-secrets.outputs.has_secrets == 'true'
  uses: DataDog/synthetics-ci-github-action@87b505388a22005bb8013481e3f73a367b9a53eb
  ...

Checklist

  • Use the owner account that submitted registrar_reserve to sign the project information, and fill in the signature.
  • After each update of the project information, please sign the latest data and submit the signature.
  • Add a description for your pr, the description includes the original data.
  • Fill up the signature of the original data, signature account to Signature Auto check.

Signature Auto check

Para ID Owner & Signature Account:
<REPLACE THE OWNER>

Data that needs to be signed:
file

Signature Hash: <REPLACE THE SIGNATURE>

Copilot AI changed the title [WIP] Fix issue with commit checks in project Fix Datadog Synthetics CI workflow failing on missing secrets Apr 7, 2026
Copilot AI requested a review from TuneMe81 April 7, 2026 02:11
@TuneMe81 TuneMe81 marked this pull request as ready for review April 8, 2026 20:37
Copy link
Copy Markdown
Owner

@TuneMe81 TuneMe81 left a comment

Choose a reason for hiding this comment

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

Looks good

@TuneMe81 TuneMe81 self-requested a review April 8, 2026 22:29
with:
api_key: ${{secrets.DD_API_KEY}}
app_key: ${{secrets.DD_APP_KEY}}
test_search_query: 'tag:e2e-tests' #Modify this tag to suit your tagging strategy
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Polkadot wallet is where all my info

Suggested change
test_search_query: 'tag:e2e-tests' #Modify this tag to suit your tagging strategy**_[]()![sign_checker.yaml.txt](https://github.com/user-attachments/files/26581733/sign_checker.yaml.txt)
_**

@TuneMe81 TuneMe81 added the documentation Improvements or additions to documentation label Apr 21, 2026
@TuneMe81
Copy link
Copy Markdown
Owner

TuneMe81 commented May 5, 2026

get rid the datadog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants