feat: add retry mechanism for artifact uploads with configurable wait time #56

Merged
Michael.Seele merged 1 commit from feature/upload-artifact-retries into main 2026-07-08 06:12:09 +00:00
No description provided.
feat: add retry mechanism for artifact uploads with configurable wait time
All checks were successful
validate-shared-actions / validate-shared-actions (pull_request) Successful in 26s
Aikido Security PR Check / Aikido Security Scan (pull_request) Successful in 44s
6257033d09
Ruben.Kraft requested changes 2026-07-07 12:13:12 +00:00
Dismissed
@ -10,6 +10,7 @@ Upload files as a Forgejo Actions artifact. Thin wrapper around `forgejo/upload-
| `path` | Yes | — | File or directory path to upload |
| `retention-days` | No | `30` | Number of days to retain the artifact |
| `if-no-files-found` | No | `warn` | Behaviour when no files are found — `warn`, `error`, or `ignore` |
| `retry-wait-seconds` | No | `10` | Seconds to wait before each retry attempt |
Owner

wärs nicht auch gut wenn man anstatt nur die Sekunden zwischen einem Retry angeben zu können auch die Anzahl und ob überhaupt retried werden soll? (Das kann man ja dann zusätzlich über bspw. einen Parameter retry-count machen der default auf 0 steht.)
Damit bleibt die upload-artifact action auch verwendbar wenn kein Retry gemacht werden soll.

wärs nicht auch gut wenn man anstatt nur die Sekunden zwischen einem Retry angeben zu können auch die Anzahl und ob überhaupt retried werden soll? (Das kann man ja dann zusätzlich über bspw. einen Parameter `retry-count` machen der default auf 0 steht.) Damit bleibt die `upload-artifact` action auch verwendbar wenn kein Retry gemacht werden soll.
Author
Owner

Leider ist das sehr schwierig weil man eine composite Action nicht in der loop aufrufen kann. Ich finde es so wie wir es lösen schon hässlich, wenn wir jetzt noch "endlose" Schritte hinzufügen wird es noch hässlicher. Daher wäre für mich das der Kompromis.

Leider ist das sehr schwierig weil man eine composite Action nicht in der loop aufrufen kann. Ich finde es so wie wir es lösen schon hässlich, wenn wir jetzt noch "endlose" Schritte hinzufügen wird es noch hässlicher. Daher wäre für mich das der Kompromis.
Owner

passt für mich.

passt für mich.
Ruben.Kraft marked this conversation as resolved
Michael.Seele deleted branch feature/upload-artifact-retries 2026-07-08 06:12:09 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
schmalz/shared-actions!56
No description provided.