mirror of
1
Fork 0
forgejo/release-notes/8.0.0/fix
Rowan Bohde ab66bfff91 fix: allow actions artifacts storage migration to complete succesfully (#31251)
Change the copy to use `ActionsArtifact.StoragePath` instead of the
`ArtifactPath`. Skip artifacts that are expired, and don't error if the
file to copy does not exist.

---

When trying to migrate actions artifact storage from local to MinIO, we
encountered errors that prevented the process from completing
successfully:

* The migration tries to copy the files using the per-run
`ArtifactPath`, instead of the unique `StoragePath`.
* Artifacts that have been marked expired and had their files deleted
would throw an error
* Artifacts that are pending, but don't have a file uploaded yet will
throw an error.

This PR addresses these cases, and allow the process to complete
successfully.

(cherry picked from commit 8de8972baf5d82ff7b58ed77d78e8e1869e64eb5)
(cherry picked from commit 4afbfd3946)
2024-06-09 14:55:01 +00:00
..
3729.md [v7.0/forgejo] docs(release-notes): 7.0.3 (#3884) 2024-05-24 12:40:26 +00:00
3864.md add release-notes 2024-05-24 00:26:20 +00:00
3904.md fix(services): set SendNotificationEmailOnNewUser 2024-05-25 17:32:52 +02:00
3935.md migrations: Map non-existant external users to Ghost 2024-05-28 16:44:23 +00:00
3976.md fix(hook): repo admins are wrongly denied the right to force merge 2024-06-02 22:05:16 +02:00
4026.md test(oauth): coverage for the redirection of a denied grant 2024-06-05 14:19:38 +00:00
4085.md fix: allow actions artifacts storage migration to complete succesfully (#31251) 2024-06-09 14:55:01 +00:00