uplink icon indicating copy to clipboard operation
uplink copied to clipboard

Uplink stuck during download

Open ihaid opened this issue 11 months ago • 7 comments

When downloading files Uplink freezing on the beginning of download.

time uplink cp sj://perf/verybig.ISO Downloads/2GB.txt

Downloads/2GB.txt  67.11 MB / 1.69 GB [==>------------------------------------------------------------------------------------] 3.98%
➜  ~ uplink version
Release build
Version:            v1.98.2
Build timestamp:    20 Feb 24 20:26 EET
Git commit:         39ad43b61a965e8b3678e1c7b54e301c6d2a8a65

PATH                      VERSION
storj.io/storj            (devel)
storj.io/common           v0.0.0-20240213084453-c623afb8001a
storj.io/drpc             v0.0.33
storj.io/eventkit         v0.0.0-20240124163201-beae173bc798
storj.io/infectious       v0.0.2
storj.io/monkit-jaeger    v0.0.0-20230707083646-f15e6e8b7e8c
storj.io/picobuf          v0.0.3
storj.io/uplink           v1.12.3-0.20240207220713-2b8836cb2e03

ihaid avatar Feb 29 '24 15:02 ihaid

Does this happen reliably? Can you share a restricted access grant for a test file that this is happening for?

jtolio avatar Feb 29 '24 15:02 jtolio

Yes I can share in DM because it is prod It happens quite often

ihaid avatar Feb 29 '24 15:02 ihaid

hi @jtolio,

I have same issue. When mem cache full, uplink stops downloading.

2024-03-08-224636_962x53_scrot 2024-03-08-224610_961x1064_scrot

ibrahimtan93 avatar Mar 08 '24 20:03 ibrahimtan93

most likely this is timeout problem on the uplink side

ihaid avatar Mar 11 '24 12:03 ihaid

Appears (finally) fixed by https://review.dev.storj.io/c/storj/uplink/+/12743

jtolio avatar Apr 09 '24 13:04 jtolio

@jtolio Let’s try to merge it to include in the nearest release and then I will close it

ihaid avatar Apr 09 '24 13:04 ihaid

And we need to add some tests to it

ihaid avatar Apr 09 '24 14:04 ihaid

I'm having a similar problem on Windows. Hopefully this fix lands soon as I'm consistently getting stuck trying to download ~8gb on this Windows box. I'm not using the uplink executable, just this simple chunk of code with v1.22.2:

src, err := storj.DownloadObject(context.Background(), xglauncher.StorjBucketName, key, nil)
if err != nil {
	return err
}
defer src.Close()
n, err := io.Copy(dst, src)
if err != nil {
	return err
}

I'm not sure if this is the same issue, but my resource monitor is showing 15 idle open TCP connections during the download with large variety of ips. This has worked flawlessly for me on my Linux box (same network).

I'll investigate adding my own retry / download resume code to compensate. That being said, this kind of problem makes me really nervous to use Storj in production.

snichols avatar May 14 '24 21:05 snichols

Something else may be happening, as our fix has been deployed for some time now. If you are using the library, it may be because we haven't tagged a new Uplink library release. If you're doing development, could you see if things work if you do go get storj.io/uplink@main to get the latest commit?

jtolio avatar May 14 '24 21:05 jtolio

Sure, let me grab storj.io/uplink@main to see if that improves things.

snichols avatar May 14 '24 21:05 snichols

Confirmed that we haven't tagged a new release with the fix yet. I'll get that going.

jtolio avatar May 14 '24 21:05 jtolio

Great. Doing a simple go get storj.io/uplink@main isn't playing nice with go modules:

go get go get storj.io/uplink@main
go: downloading storj.io/uplink v1.12.3-0.20240507000622-d7c4c594337f
go: malformed module path "get": missing dot in first path element

Perhaps I'm missing a secret handshake. Once the new tag drops, I'll upgrade and give it a go.

snichols avatar May 14 '24 21:05 snichols

go get go get storj.io/uplink@main

You have an extra go get in the command.

onionjake avatar May 14 '24 21:05 onionjake

Copypasta FTL.

snichols avatar May 14 '24 21:05 snichols

Alrighty, one download succeeded with the new code. I'll do some more tests and let you know if it breaks again. Thanks!

snichols avatar May 14 '24 22:05 snichols

storj.io/[email protected] released. Since the changes in v1.13.0 fixed all the known reproductions of this ticket, I'm going to close it. @snichols feel free to reopen if you find this happening again, and sorry for the tardy Uplink version release!

jtolio avatar May 15 '24 15:05 jtolio