CLI retry failed request and don't cancel entire flow on error #1835

Closed
opened 2026-02-05 04:07:35 +03:00 by OVERLORD · 2 comments
Owner

Originally created by @zackpollard on GitHub (Dec 19, 2023).

Originally assigned to: @etnoy on GitHub.

Currently the CLI will error and cancel the whole process if one request fails. We should probably instead retry a couple of times and then log an error that the asset failed, but continue on to other assets.

Originally created by @zackpollard on GitHub (Dec 19, 2023). Originally assigned to: @etnoy on GitHub. Currently the CLI will error and cancel the whole process if one request fails. We should probably instead retry a couple of times and then log an error that the asset failed, but continue on to other assets.
OVERLORD added the cli label 2026-02-05 04:07:35 +03:00
Author
Owner

@masterT commented on GitHub (Jun 15, 2024):

This is particularly annoying when uploading a lot of assets. Was there any progress toward this issue?

@masterT commented on GitHub (Jun 15, 2024): This is particularly annoying when uploading a lot of assets. Was there any progress toward this issue?
Author
Owner

@masterT commented on GitHub (Jun 18, 2024):

A significant improvement could be using an in-memory queue system (e.g., fastq) and having the ability to retry not just failed asset uploads but also other parts of the CLI.

I'm trying to upload an extensive library, and I have all kinds of network errors (I'm hosted with PikaPod, and my instance sometimes restarts, which makes the whole process crash).

I think I will craft a draft for this and open a PR.

@masterT commented on GitHub (Jun 18, 2024): A significant improvement could be using an in-memory queue system (e.g., [fastq](https://www.npmjs.com/package/fastq)) and having the ability to retry not just failed asset uploads but also other parts of the CLI. I'm trying to upload an extensive library, and I have all kinds of network errors (I'm hosted with PikaPod, and my instance sometimes restarts, which makes the whole process crash). I think I will craft a draft for this and open a PR.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#1835