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

feat(server): clean up interrupted upload files #14265

Merged
merged 7 commits into from
Nov 20, 2024

Conversation

alextran1502
Copy link
Contributor

No description provided.

Copy link
Member

@danieldietzler danieldietzler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be cool if we could write an e2e test for this. Not sure if it's (easily) doable though

@alextran1502
Copy link
Contributor Author

@jrasm91 Done, is this what you are looking for?

Copy link
Contributor

@jrasm91 jrasm91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect, ty!

@alextran1502 alextran1502 merged commit 9a9d40c into main Nov 20, 2024
36 checks passed
@alextran1502 alextran1502 deleted the feat/clean-up-interrupted-upload branch November 20, 2024 23:17
yosit pushed a commit to yosit/immich that referenced this pull request Nov 21, 2024
* feat(server): clean up interrupted upload files

* pr feedback

* remove console.log

* handle all errors

* remove return in callback function

* programming in bed is a bad idea
@akostadinov
Copy link

How can one clean the existing failed uploads?

@bo0tzz
Copy link
Member

bo0tzz commented Dec 9, 2024

Manually

@akostadinov
Copy link

akostadinov commented Dec 9, 2024

@bo0tzz , but how exactly to distinguish stale objects from something that is going on right now?

@bo0tzz
Copy link
Member

bo0tzz commented Dec 9, 2024

If you're using the storage template, anything left behind in upload after running the template job should be safe to remove.

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

Successfully merging this pull request may close these issues.

5 participants