[Dependency Cache] Enable Dependency Cache on CI [with GRADLE_RO_DEP_CACHE] #13122
+51
−3
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ParaskP7
added
category: tooling
|
📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.
|
4 tasks
ParaskP7
force-pushed
the
ci/enable-dependency-cache-on-ci
branch
7 times, most recently
from
5036933 to
91c8676
Compare
ParaskP7
changed the title
ParaskP7
force-pushed
the
ci/enable-dependency-cache-on-ci
branch
from
891e39e to
640942f
Compare
Generated by 🚫 Danger |
📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.
|
ParaskP7
force-pushed
the
ci/enable-dependency-cache-on-ci
branch
6 times, most recently
from
c428113 to
d31164c
Compare
FYI: This job will be then used by 'buildkite-ci' and configured as a 'buildkite_pipeline_schedule' with a weekly frequency.
This change points to that remote branch and commit where the 'GRADLE_RO_DEP_CACHE' environment variable is indeed set, and thus is indeed pointing to the directory containing the cache.
This change points to that remote branch and commit where the 'GRADLE_RO_DEP_CACHE' environment variable is not set, and thus not pointing to the directory containing the cache.
This should be now done automatically and via this another standard Buildkite environment variable being the prefix, the 'BUILDKITE_PIPELINE_SLUG' environment variable. For more info see: https://github.com/Automattic/ a8c-ci-toolkit-buildkite-plugin/pull/134/commits/ 4d3f80b9f113076c9a8698f08d137ed591426150
This change effectively reverts this initial 2b9213a commit.
ParaskP7
force-pushed
the
ci/enable-dependency-cache-on-ci
branch
from
eea5e8c to
93bfc61
Compare
ParaskP7
changed the title
4 tasks
…to ci/enable-dependency-cache-on-ci
ParaskP7
force-pushed
the
ci/enable-dependency-cache-on-ci
branch
from
4af8801 to
e6f3975
Compare
…to ci/enable-dependency-cache-on-ci
…to ci/enable-dependency-cache-on-ci
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related: #13170
Description
This is a test only PR that attempts to test the existing dependency cache mechanism, one that does dependent on the read-only cache (GRADLE_RO_DEP_CACHE), but per project.
Testing information
N/ATip
To verify that
GRADLE_RO_DEP_CACHEis enabled make sure you do see thisShared read-only dependency cache is an incubating feature.log when Gradle launches for a CI build (example).The tests that have been performed
RELEASE-NOTES.txtif necessary. Use the "[Internal]" label for non-user-facing changes.Reviewer (or Author, in the case of optional code reviews):
Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement: