Fix missing migrate actions artifacts (#30874) (#30886)

Backport #30874 by @lunny

The actions artifacts should be able to be migrate to the new storage
place.

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>

diff --git a/cmd/migrate_storage.go b/cmd/migrate_storage.go
index aa49445a89e81f4f1d766723628632d395cb3ed9..357416fc339fd297de1c2ac9f558425502247f4a 100644 (file)
--- a/cmd/migrate_storage.go
+++ b/cmd/migrate_storage.go
@@ -34,7 +34,7 @@ var CmdMigrateStorage = &cli.Command{
                        Name:    "type",
                        Aliases: []string{"t"},
                        Value:   "",
-                       Usage:   "Type of stored files to copy.  Allowed types: 'attachments', 'lfs', 'avatars', 'repo-avatars', 'repo-archivers', 'packages', 'actions-log'",
+                       Usage:   "Type of stored files to copy.  Allowed types: 'attachments', 'lfs', 'avatars', 'repo-avatars', 'repo-archivers', 'packages', 'actions-log', 'actions-artifacts",
                },
                &cli.StringFlag{
                        Name:    "storage",
@@ -160,6 +160,13 @@ func migrateActionsLog(ctx context.Context, dstStorage storage.ObjectStorage) er
        })
 }
 
+func migrateActionsArtifacts(ctx context.Context, dstStorage storage.ObjectStorage) error {
+       return db.Iterate(ctx, nil, func(ctx context.Context, artifact *actions_model.ActionArtifact) error {
+               _, err := storage.Copy(dstStorage, artifact.ArtifactPath, storage.ActionsArtifacts, artifact.ArtifactPath)
+               return err
+       })
+}
+
 func runMigrateStorage(ctx *cli.Context) error {
        stdCtx, cancel := installSignals()
        defer cancel()
@@ -223,13 +230,14 @@ func runMigrateStorage(ctx *cli.Context) error {
        }
 
        migratedMethods := map[string]func(context.Context, storage.ObjectStorage) error{
-               "attachments":    migrateAttachments,
-               "lfs":            migrateLFS,
-               "avatars":        migrateAvatars,
-               "repo-avatars":   migrateRepoAvatars,
-               "repo-archivers": migrateRepoArchivers,
-               "packages":       migratePackages,
-               "actions-log":    migrateActionsLog,
+               "attachments":       migrateAttachments,
+               "lfs":               migrateLFS,
+               "avatars":           migrateAvatars,
+               "repo-avatars":      migrateRepoAvatars,
+               "repo-archivers":    migrateRepoArchivers,
+               "packages":          migratePackages,
+               "actions-log":       migrateActionsLog,
+               "actions-artifacts": migrateActionsArtifacts,
        }
 
        tp := strings.ToLower(ctx.String("type"))