Fix build attributes test

What does this MR do?

This is a follow up from https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/9021#note_23782188

Description

Fix build attributes test, see:

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/9021#note_23782188

This is discovered by https://gitlab.com/gitlab-org/gitlab-ce/builds/10815456

I removed gl_project_id and this is failing. I took some look, realizing that:

• trace is duplicated in attributes
• tag_list is not included in build.attributes
• artifacts_expire_at is missing in attributes

So we need to:

• Remove duplicated trace in attributes (40 -> 39)
• Remove tag_list in attributes (39 -> 38)
• Add artifacts_expire_at to attributes (38 -> 39)
• Add gl_project_id to attributes (39 -> 40)

/cc @ayufan

Note that we'll remove gl_project_id again in !9378 (closed)

app/services/ci/retry_build_service.rb

 module Ci
   class RetryBuildService < ::BaseService
-    CLONE_ATTRIBUTES = %i[pipeline ref tag options commands tag_list name
+    CLONE_ATTRIBUTES = %i[pipeline project ref tag options commands name
                           allow_failure stage stage_idx trigger_request
                           yaml_variables when environment coverage_regex]
                             .freeze
     REJECT_ATTRIBUTES = %i[id status user token coverage trace runner
-                           artifacts_file artifacts_metadata artifacts_size
+                           artifacts_expire_at artifacts_file
+                           artifacts_metadata artifacts_size
                            created_at updated_at started_at finished_at
                            queued_at erased_by erased_at].freeze
-    IGNORE_ATTRIBUTES = %i[trace type lock_version project target_url
+    IGNORE_ATTRIBUTES = %i[type lock_version gl_project_id target_url
                            deploy job_id description].freeze
     def execute(build)