-
Notifications
You must be signed in to change notification settings - Fork 875
ci: Add DataDog tracing #163
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
Conversation
Codecov Report
@@ Coverage Diff @@
## main #163 +/- ##
==========================================
- Coverage 65.51% 65.41% -0.11%
==========================================
Files 106 106
Lines 5580 5580
Branches 68 68
==========================================
- Hits 3656 3650 -6
- Misses 1568 1574 +6
Partials 356 356
Continue to review full report at Codecov.
|
cf50784
to
a0b2cb5
Compare
.github/workflows/coder.yaml
Outdated
- name: Upload DataDog Trace | ||
if: always() | ||
# See: https://docs.datadoghq.com/continuous_integration/setup_tests/junit_upload/#collecting-environment-configuration-metadata | ||
env: | ||
DATADOG_API_KEY: ${{ secrets.DATADOG_API_KEY }} | ||
DD_GIT_REPOSITORY_URL: ${{ github.repositoryUrl }} | ||
DD_GIT_BRANCH: ${{ github.base_ref }} | ||
DD_GIT_COMMIT_SHA: ${{ github.sha }} | ||
DD_TAGS: ${{ join([format('os.platform:{0}', runner.os), format('os.architecture:{0}', runner.arch)]) }} | ||
# os.platform:${{ runner.os == 'macOS' ? 'darwin' \: runner.os.toLowerCase()}},os.architecture:${{ runner.arch.toLowerCase() }} | ||
run: | | ||
npm install -g @datadog/datadog-ci | ||
datadog-ci junit upload --service coder gotests.xml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neat! Once you merge this in - I'll add the upload for the front-end tests 👍
f281aa2
to
9a2a84f
Compare
No description provided.