diff options
| author | Rémy Coutable <remy@rymai.me> | 2018-09-06 09:10:04 +0000 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2018-09-06 09:10:04 +0000 |
| commit | f965bc9e669130bd567e575db197a82270affd96 (patch) | |
| tree | c2aa7c9e562aa3307d345abb2f2e20dd95827bb1 /app/models | |
| parent | b92b7d822226724db76b8dfb61dca438ea8dbb1c (diff) | |
| parent | 1792450978ee3766f6e9f8a8fac8117b134ae42c (diff) | |
| download | gitlab-ce-f965bc9e669130bd567e575db197a82270affd96.tar.gz | |
Merge branch 'rails5-explicit-hashed-path-check' into 'master'
Explicit hashed path check for trace, prevents background migration from…
See merge request gitlab-org/gitlab-ce!21533
Diffstat (limited to 'app/models')
| -rw-r--r-- | app/models/ci/job_artifact.rb | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/app/models/ci/job_artifact.rb b/app/models/ci/job_artifact.rb index 4853b23513c..93fc1b145b2 100644 --- a/app/models/ci/job_artifact.rb +++ b/app/models/ci/job_artifact.rb @@ -87,7 +87,9 @@ module Ci end def hashed_path? - super || self.try(:file_location).nil? + return true if trace? # ArchiveLegacyTraces background migration might not have `file_location` column + + super || self.file_location.nil? end def expire_in |