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

Run build-nextjs for Turbopack test runs on metal machines instead of GitHub hosted runner #75654

Conversation

timneutkens
Copy link
Member

@timneutkens timneutkens commented Feb 4, 2025

What?

All the test runs that happen after this workflow runs are running on our self hosted runners but the initial setup is running on GitHub's runner, this causes an environment mismatch that seems to be causing these errors:

  ⚠ Error: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.39' not found (required by /root/actions-runner/_work/next.js/next.js/packages/next-swc/native/next-swc.linux-x64-gnu.node)
      at mod.require (packages/next/dist/server/require-hook.js:65:28)
      at loadNative (packages/next/dist/build/swc/index.js:853:69)
      at <unknown> (packages/next/dist/build/swc/index.js:264:28)
      at new Promise (<anonymous>)
      at loadBindings (packages/next/dist/build/swc/index.js:232:23) {
    code: 'ERR_DLOPEN_FAILED'
  }

Going to land this to verify it remedies the problem.

@ijjk ijjk added the created-by: Next.js team PRs by the Next.js team. label Feb 4, 2025
Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@timneutkens timneutkens marked this pull request as ready for review February 4, 2025 13:21
@ijjk
Copy link
Member

ijjk commented Feb 4, 2025

Failing test suites

Commit: b243b32

pnpm test test/integration/amp-export-validation/test/index.test.js

  • AMP Validation on Export > production mode > should have shown errors during build
Expand output

● AMP Validation on Export › production mode › should have shown errors during build

expect(received).toMatch(expected)

Expected pattern: /error.*The mandatory attribute 'height' is missing in tag 'amp-video'\./
Received string:  "   Loading config from /root/actions-runner/_work/next.js/next.js/test/integration/amp-export-validation/next.config.js
   Loading config from /root/actions-runner/_work/next.js/next.js/test/integration/amp-export-validation/next.config.js
   ▲ Next.js 15.2.0-canary.39·
   Checking validity of types ...
   Creating an optimized production build ...
   Loading config from /root/actions-runner/_work/next.js/next.js/test/integration/amp-export-validation/next.config.js
   Loading config from /root/actions-runner/_work/next.js/next.js/test/integration/amp-export-validation/next.config.js
   Loading config from /root/actions-runner/_work/next.js/next.js/test/integration/amp-export-validation/next.config.js
 ✓ Compiled successfully
   Collecting page data ...
   Generating static pages (0/8) ...
   Generating static pages (2/8)··
 ⚠ Linting is disabled.
Error occurred prerendering page \"/second\". Read more: https://nextjs.org/docs/messages/prerender-error
AssertionError: Assertion failed: WebAssembly is uninitialized

  at new module$contents$goog$asserts_AssertionError (../evalmachine.<anonymous>:106:1695)
  at module$contents$goog$asserts_doAssertFailure (../evalmachine.<anonymous>:107:354)
  at goog.asserts.assertExists (../evalmachine.<anonymous>:109:142)
  at Object.module$contents$amp$validator_validateString [as validateString] (../evalmachine.<anonymous>:2663:108)
  at Validator.validateString (../packages/next/dist/compiled/amphtml-validator/index.js:1:20650)
  at validateAmp (../packages/next/dist/export/routes/pages.js:87:34)
  at async exportPagesPage (../packages/next/dist/export/routes/pages.js:104:13)
  at async Span.traceAsyncFn (../packages/next/dist/trace/trace.js:156:20)
  at async exportPage (../packages/next/dist/export/worker.js:352:18)
  Export encountered an error on /second, exiting the build.
   ⨯ Static worker exited with code: 1 and signal: null
  "
  at Object.toMatch (integration/amp-export-validation/test/index.test.js:28:29)

Read more about building and testing Next.js in contributing.md.

@timneutkens timneutkens merged commit c70bf42 into canary Feb 4, 2025
121 of 124 checks passed
@timneutkens timneutkens deleted the 02-04-run_build-nextjs_for_turbopack_test_runs_on_metal_machines_instead_of_github_hosted_runner branch February 4, 2025 13:23
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
created-by: Next.js team PRs by the Next.js team. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants