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

frequent, random HTTP import failure #24260

Closed
scarf005 opened this issue Jun 19, 2024 · 5 comments · Fixed by #24262
Closed

frequent, random HTTP import failure #24260

scarf005 opened this issue Jun 19, 2024 · 5 comments · Fixed by #24262
Labels
bug Something isn't working correctly high priority

Comments

@scarf005
Copy link
Contributor

Version: deno 1.44.3+f4eead6 (canary, x86_64-unknown-linux-gnu)

Issue

deno cache/download for the first time (in CI) frequently fails, randomly, unexpectedly.

Steps to reproduce

test.ts

import { z } from "https://deno.land/x/[email protected]/mod.ts"
import { Command } from "https://deno.land/x/[email protected]/deps/cliffy.ts"
import { cliOptions } from "https://deno.land/x/[email protected]/utils/cli.ts"
import { timeit } from "https://deno.land/x/[email protected]/utils/timeit.ts"
import {
  applyRecursively,
  schemaTransformer,
} from "https://deno.land/x/[email protected]/utils/transform.ts"
import {
  type CataEntry,
  Entry,
  parseCataJson,
  readRecursively,
} from "https://deno.land/x/[email protected]/utils/parse.ts"
import { fmtJsonRecursively } from "https://deno.land/x/[email protected]/utils/json_fmt.ts"
import { match, P } from "https://deno.land/x/[email protected]/deps/ts_pattern.ts"
import { id } from "https://deno.land/x/[email protected]/utils/id.ts"
deno cache --reload --lock=deno.lock --lock-write test.ts

asciicast


tracing log: error.log

@scarf005 scarf005 changed the title frequent, random cache failure frequent, random HTTP import failure Jun 19, 2024
@dworzycp
Copy link

We've also noticed this in our CI - seems to be related to the latest 1.44.3 release.

Downgrading deno to 1.44.2 fixes the problem, can do this with deno upgrade --version 1.44.2

@lucacasonato
Copy link
Member

lucacasonato commented Jun 19, 2024

We are also noticing this internally. It is likely caused by 22f2779. We will pull 1.44.3 while we are investigating. Sorry for the troubles.

UPDATE: 1.44.3 has been pulled - installing Deno in CI should now install 1.44.2 again.

@scarf005
Copy link
Contributor Author

UPDATE: 1.44.3 has been pulled - installing Deno in CI should now install 1.44.2 again.

image

https://github.com/cataclysmbnteam/Cataclysm-BN/actions/runs/9582298163/job/26422268220

it looks like CI still installs 1.44.3 given version range 1.x, should i trigger new CI using empty commits instead of re-run failed jobs?

image

@lucacasonato
Copy link
Member

lucacasonato commented Jun 19, 2024

This should now be fixed @scarf005 - please try again by just re-running

@lucacasonato lucacasonato added bug Something isn't working correctly high priority labels Jun 19, 2024
@scarf005
Copy link
Contributor Author

This should now be fixed @scarf005 - please try again by just re-running

image

it works now, thanks for the quick action @lucacasonato

bartlomieju added a commit that referenced this issue Jun 19, 2024
…24262)

This reverts commit fb31eaa.

Reverting because users reported spurious errors when downloading
dependencies - #24260.

Closes #24260
bartlomieju added a commit that referenced this issue Jun 19, 2024
…24262)

This reverts commit fb31eaa.

Reverting because users reported spurious errors when downloading
dependencies - #24260.

Closes #24260
sbmsr pushed a commit to sbmsr/deno-1 that referenced this issue Jul 2, 2024
)" (denoland#24262)

This reverts commit fb31eaa.

Reverting because users reported spurious errors when downloading
dependencies - denoland#24260.

Closes denoland#24260
zebreus pushed a commit to zebreus/deno that referenced this issue Jul 8, 2024
)" (denoland#24262)

This reverts commit fb31eaa.

Reverting because users reported spurious errors when downloading
dependencies - denoland#24260.

Closes denoland#24260
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working correctly high priority
Projects
None yet
3 participants