Skip to content

[ci]: continue-on-error for more gh auth status checks#90890

Merged
ztanner merged 1 commit intocanaryfrom
03-04-_ci_continue-on-error_for_more_gh_auth_status_checks
Mar 4, 2026
Merged

[ci]: continue-on-error for more gh auth status checks#90890
ztanner merged 1 commit intocanaryfrom
03-04-_ci_continue-on-error_for_more_gh_auth_status_checks

Conversation

@ztanner
Copy link
Member

@ztanner ztanner commented Mar 4, 2026

This seems to be failing for our deploy test workflow (x-ref).

Updating to continue-on-error similar to #89098 because this check doesn't seem reliable.

If it fixes, will just remove in both spots all together.

Copy link
Member Author

ztanner commented Mar 4, 2026

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

@ztanner ztanner requested review from eps1lon and ijjk March 4, 2026 21:08
@ztanner ztanner marked this pull request as ready for review March 4, 2026 21:08
@ztanner ztanner merged commit 9ff3b7c into canary Mar 4, 2026
144 of 147 checks passed
@ztanner ztanner deleted the 03-04-_ci_continue-on-error_for_more_gh_auth_status_checks branch March 4, 2026 21:17
@nextjs-bot
Copy link
Collaborator

nextjs-bot commented Mar 4, 2026

Failing test suites

Commit: ef8ae85 | About building and testing Next.js

pnpm test-start-turbo test/production/deterministic-build/deployment-id.test.ts (turbopack) (job)

  • deterministic build - changing deployment id > build output API - cacheComponents builder > should produce identical build outputs even when changing deployment id (DD)
Expand output

● deterministic build - changing deployment id › build output API - cacheComponents builder › should produce identical build outputs even when changing deployment id

expect(received).toBe(expected) // Object.is equality

Expected: 0
Received: 1

  123 |   // First build
  124 |   next.env['NEXT_DEPLOYMENT_ID'] = 'foo-dpl-id'
> 125 |   expect((await next.build()).exitCode).toBe(0)
      |                                         ^
  126 |   let run1 = await readFiles(next)
  127 |
  128 |   // Second build

  at toBe (production/deterministic-build/deployment-id.test.ts:125:41)
  at Object.<anonymous> (production/deterministic-build/deployment-id.test.ts:263:30)

pnpm test-dev test/development/app-dir/server-components-hmr-cache/server-components-hmr-cache.test.ts (job)

  • server-components-hmr-cache > should support reading from an infinite streaming fetch (DD)
Expand output

● server-components-hmr-cache › should support reading from an infinite streaming fetch

thrown: "Exceeded timeout of 10000 ms for a test.
Add a timeout value to this test to increase the timeout, if this is a long-running test. See https://jestjs.io/docs/api#testname-fn-timeout."

  224 |   })
  225 |
> 226 |   it('should support reading from an infinite streaming fetch', async () => {
      |   ^
  227 |     const browser = await next.browser('/infinite-stream')
  228 |     const text = await browser.elementByCss('p').text()
  229 |     expect(text).toBe('data: chunk-1')

  at it (development/app-dir/server-components-hmr-cache/server-components-hmr-cache.test.ts:226:3)
  at Object.describe (development/app-dir/server-components-hmr-cache/server-components-hmr-cache.test.ts:6:1)

@nextjs-bot
Copy link
Collaborator

Stats from current PR

✅ No significant changes detected

📊 All Metrics
📖 Metrics Glossary

Dev Server Metrics:

  • Listen = TCP port starts accepting connections
  • First Request = HTTP server returns successful response
  • Cold = Fresh build (no cache)
  • Warm = With cached build artifacts

Build Metrics:

  • Fresh = Clean build (no .next directory)
  • Cached = With existing .next directory

Change Thresholds:

  • Time: Changes < 50ms AND < 10%, OR < 2% are insignificant
  • Size: Changes < 1KB AND < 1% are insignificant
  • All other changes are flagged to catch regressions

⚡ Dev Server

Metric Canary PR Change Trend
Cold (Listen) 455ms 455ms ▁▁▁▇▁
Cold (Ready in log) 439ms 437ms ▂▁▁▆▁
Cold (First Request) 1.267s 1.248s ▃▁▃▆▁
Warm (Listen) 457ms 456ms ▁▁▁▇▁
Warm (Ready in log) 444ms 446ms ▁▁▁█▁
Warm (First Request) 346ms 350ms ▁▂▁▆▁
📦 Dev Server (Webpack) (Legacy)

📦 Dev Server (Webpack)

Metric Canary PR Change Trend
Cold (Listen) 507ms 507ms ▁▁▁▁▁
Cold (Ready in log) 480ms 478ms ▃▂▃▃▃
Cold (First Request) 2.164s 2.161s ▂▁▁▁▂
Warm (Listen) 507ms 508ms ▁▁▁▁▁
Warm (Ready in log) 478ms 478ms ▄▃▃▃▃
Warm (First Request) 2.174s 2.177s ▃▂▁▁▂

⚡ Production Builds

Metric Canary PR Change Trend
Fresh Build 3.822s 3.891s ▁▁▁▆▁
Cached Build 3.884s 3.848s ▁▁▁▆▁
📦 Production Builds (Webpack) (Legacy)

📦 Production Builds (Webpack)

Metric Canary PR Change Trend
Fresh Build 15.745s 15.668s ▂▁▁▁▂
Cached Build 15.743s 15.814s ▁▁▁▁▂
node_modules Size 475 MB 475 MB ▁▁▁▁▁
📦 Bundle Sizes

Bundle Sizes

⚡ Turbopack

Client

Main Bundles: **401 kB** → **401 kB** ✅ -31 B

80 files with content-based hashes (individual files not comparable between builds)

Server

Middleware
Canary PR Change
middleware-b..fest.js gzip 767 B 760 B
Total 767 B 760 B ✅ -7 B
Build Details
Build Manifests
Canary PR Change
_buildManifest.js gzip 451 B 451 B
Total 451 B 451 B

📦 Webpack

Client

Main Bundles
Canary PR Change
5528-HASH.js gzip 5.54 kB N/A -
6280-HASH.js gzip 59.4 kB N/A -
6335.HASH.js gzip 169 B N/A -
912-HASH.js gzip 4.59 kB N/A -
e8aec2e4-HASH.js gzip 62.6 kB N/A -
framework-HASH.js gzip 59.7 kB 59.7 kB
main-app-HASH.js gzip 254 B 254 B
main-HASH.js gzip 39.1 kB 39.1 kB
webpack-HASH.js gzip 1.68 kB 1.68 kB
262-HASH.js gzip N/A 4.59 kB -
2889.HASH.js gzip N/A 169 B -
5602-HASH.js gzip N/A 5.55 kB -
6948ada0-HASH.js gzip N/A 62.6 kB -
9544-HASH.js gzip N/A 60.2 kB -
Total 233 kB 234 kB ⚠️ +734 B
Polyfills
Canary PR Change
polyfills-HASH.js gzip 39.4 kB 39.4 kB
Total 39.4 kB 39.4 kB
Pages
Canary PR Change
_app-HASH.js gzip 194 B 194 B
_error-HASH.js gzip 183 B 180 B 🟢 3 B (-2%)
css-HASH.js gzip 331 B 330 B
dynamic-HASH.js gzip 1.81 kB 1.81 kB
edge-ssr-HASH.js gzip 256 B 256 B
head-HASH.js gzip 351 B 352 B
hooks-HASH.js gzip 384 B 383 B
image-HASH.js gzip 580 B 581 B
index-HASH.js gzip 260 B 260 B
link-HASH.js gzip 2.51 kB 2.51 kB
routerDirect..HASH.js gzip 320 B 319 B
script-HASH.js gzip 386 B 386 B
withRouter-HASH.js gzip 315 B 315 B
1afbb74e6ecf..834.css gzip 106 B 106 B
Total 7.98 kB 7.98 kB ✅ -1 B

Server

Edge SSR
Canary PR Change
edge-ssr.js gzip 125 kB 125 kB
page.js gzip 254 kB 254 kB
Total 379 kB 379 kB ⚠️ +406 B
Middleware
Canary PR Change
middleware-b..fest.js gzip 618 B 618 B
middleware-r..fest.js gzip 156 B 155 B
middleware.js gzip 43.6 kB 43.7 kB
edge-runtime..pack.js gzip 842 B 842 B
Total 45.2 kB 45.3 kB ⚠️ +70 B
Build Details
Build Manifests
Canary PR Change
_buildManifest.js gzip 715 B 718 B
Total 715 B 718 B ⚠️ +3 B
Build Cache
Canary PR Change
0.pack gzip 4.03 MB 4.04 MB 🔴 +8.46 kB (+0%)
index.pack gzip 103 kB 103 kB
index.pack.old gzip 102 kB 102 kB
Total 4.24 MB 4.25 MB ⚠️ +8 kB

🔄 Shared (bundler-independent)

Runtimes
Canary PR Change
app-page-exp...dev.js gzip 321 kB 321 kB
app-page-exp..prod.js gzip 170 kB 170 kB
app-page-tur...dev.js gzip 321 kB 321 kB
app-page-tur..prod.js gzip 170 kB 170 kB
app-page-tur...dev.js gzip 318 kB 318 kB
app-page-tur..prod.js gzip 168 kB 168 kB
app-page.run...dev.js gzip 318 kB 318 kB
app-page.run..prod.js gzip 168 kB 168 kB
app-route-ex...dev.js gzip 70.3 kB 70.3 kB
app-route-ex..prod.js gzip 48.6 kB 48.6 kB
app-route-tu...dev.js gzip 70.4 kB 70.4 kB
app-route-tu..prod.js gzip 48.7 kB 48.7 kB
app-route-tu...dev.js gzip 69.9 kB 69.9 kB
app-route-tu..prod.js gzip 48.4 kB 48.4 kB
app-route.ru...dev.js gzip 69.9 kB 69.9 kB
app-route.ru..prod.js gzip 48.4 kB 48.4 kB
dist_client_...dev.js gzip 324 B 324 B
dist_client_...dev.js gzip 326 B 326 B
dist_client_...dev.js gzip 318 B 318 B
dist_client_...dev.js gzip 317 B 317 B
pages-api-tu...dev.js gzip 43.2 kB 43.2 kB
pages-api-tu..prod.js gzip 32.9 kB 32.9 kB
pages-api.ru...dev.js gzip 43.2 kB 43.2 kB
pages-api.ru..prod.js gzip 32.9 kB 32.9 kB
pages-turbo....dev.js gzip 52.6 kB 52.6 kB
pages-turbo...prod.js gzip 38.5 kB 38.5 kB
pages.runtim...dev.js gzip 52.6 kB 52.6 kB
pages.runtim..prod.js gzip 38.5 kB 38.5 kB
server.runti..prod.js gzip 61.8 kB 61.8 kB
Total 2.83 MB 2.83 MB ⚠️ +3 B
📎 Tarball URL
https://vercel-packages.vercel.app/next/commits/22ee7c967e1ca7e69b864a080c569d5c9bcfa625/next

@eps1lon
Copy link
Member

eps1lon commented Mar 5, 2026

If it fixes, will just remove in both spots all together.

I'd keep it since it's still helpful debugging. We originally added it after we got confused why release was failing only to find out the token was expired. Ideally GH would just fix it because early errors save time.

sokra pushed a commit that referenced this pull request Mar 6, 2026
This seems to be failing for our deploy test workflow
([x-ref](https://github.com/vercel/next.js/actions/runs/22685809647/job/65778800145)).

Updating to `continue-on-error` similar to
#89098 because this check doesn't
seem reliable.

If it fixes, will just remove in both spots all together.
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.

5 participants