Skip to content
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

test(ci): update datadog-ci #5714

Merged
merged 1 commit into from
Aug 16, 2023
Merged

test(ci): update datadog-ci #5714

merged 1 commit into from
Aug 16, 2023

Conversation

kwonoj
Copy link
Contributor

@kwonoj kwonoj commented Aug 14, 2023

Description

Two updates to fix datadog reporting

  • update datadog-ci dep to specific version to remove dependency workaround
  • remove merging step, let datadog-ci handles concurrent update for the multiple test results report

@vercel
Copy link

vercel bot commented Aug 14, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

11 Ignored Deployments
Name Status Preview Updated (UTC)
examples-basic-web ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm
examples-cra-web ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm
examples-designsystem-docs ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm
examples-gatsby-web ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm
examples-kitchensink-blog ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm
examples-native-web ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm
examples-nonmonorepo ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm
examples-svelte-web ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm
examples-tailwind-web ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm
examples-vite-web ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm
turbo-site ⬜️ Ignored (Inspect) Aug 14, 2023 7:26pm

@github-actions
Copy link
Contributor

✅ This change can build next-swc

@github-actions
Copy link
Contributor

Linux Benchmark for d5f999b

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 5425.37µs ± 23.07µs 5417.32µs ± 38.07µs -0.15%
bench_hmr_to_eval/Turbopack CSR/1000 modules 5080.06µs ± 19.16µs 5306.57µs ± 250.76µs +4.46%
bench_startup/Turbopack CSR/1000 modules 1002.45ms ± 0.68ms 1009.07ms ± 5.22ms +0.66%

kodiakhq bot pushed a commit to vercel/next.js that referenced this pull request Aug 14, 2023
### What

Pairing PR to vercel/turborepo#5714, which I believe last piece to reenable correct datadog test reports for daily next.js integration + turbopack.

1. Creates a unique suite name to avoid collision / merging across different env / grouped test run
2. revert suitename template (which is for _suites_, not _suite_ for top level) as 1 resolves uniqueness
@github-actions
Copy link
Contributor

github-actions bot commented Aug 14, 2023

⚠️ CI failed ⚠️

The following steps have failed in CI:

  • Turbopack Rust tests (mac/win, non-blocking)

See workflow summary for details

@kwonoj kwonoj merged commit d778e3d into main Aug 16, 2023
@kwonoj kwonoj deleted the bump-datadog-ci branch August 16, 2023 16:49
timneutkens pushed a commit to vercel/next.js that referenced this pull request Aug 25, 2023
* vercel/turborepo#5723 <!-- Alex Kirszenberg - Misc
comments in turbo_tasks -->
* vercel/turborepo#5714 <!-- OJ Kwon - test(ci):
update datadog-ci -->
* vercel/turborepo#5705 <!-- Justin Ridgewell -
Transformation code necessary to support Server Actions -->
* vercel/turborepo#5739 <!-- Justin Ridgewell -
Update rust toolchain -->
* vercel/turborepo#5785 <!-- Tobias Koppers - add
evaluatables to chunk list ident -->
* vercel/turborepo#5783 <!-- Tobias Koppers -
Tracing improvements -->
* vercel/turborepo#5738 <!-- Alex Kirszenberg -
Generic types over Vcs -->
* vercel/turborepo#5795 <!-- Leah - fix: snapshot
test path canonicalization -->
* vercel/turborepo#5794 <!-- Tobias Koppers - fix
"any" context condition implementation -->
* vercel/turborepo#5800 <!-- Tobias Koppers - add
middleware entry type -->
* vercel/turborepo#5786 <!-- Tobias Koppers -
perform invalidation directly on writes -->

Closes WEB-1446
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants