Merge branch '57431-sprockets-caches-files-independent-of-content' into 'master'

Use correct integrity hash for sprockets caching

Closes #57431

See merge request gitlab-org/gitlab-ce!25033
This commit is contained in:
Lin Jen-Shin 2019-02-11 09:29:23 +00:00
commit 8886924ca9
2 changed files with 3 additions and 5 deletions

View File

@ -388,13 +388,11 @@ flaky-examples-check:
.assets-compile-cache: &assets-compile-cache
cache:
key: "assets-compile:vendor_ruby:.yarn-cache:tmp_cache_assets_sprockets:v4"
key: "assets-compile:vendor_ruby:.yarn-cache:tmp_cache_assets_sprockets:v5"
paths:
- vendor/ruby/
- .yarn-cache/
# We have disabled caching of sprockets for now, as it fails to pick up changes in SCSS:
# https://gitlab.com/gitlab-org/gitlab-ce/issues/57431
# - tmp/cache/assets/sprockets
- tmp/cache/assets/sprockets
compile-assets:
<<: *dedicated-runner

View File

@ -9,7 +9,7 @@ module Gitlab
def file_digest(path)
if stat = self.stat(path)
digest = self.stat_digest(path, stat)
integrity_uri = self.hexdigest_integrity_uri(digest)
integrity_uri = self.integrity_uri(digest)
key = Sprockets::UnloadedAsset.new(path, self).file_digest_key(integrity_uri)
cache.fetch(key) do