Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.

[Dependency Cache] Remove Save Cache from CI #39

Merged
merged 1 commit into from
Jan 13, 2025
Merged

Conversation

ParaskP7
Copy link
Contributor

@ParaskP7 ParaskP7 commented Jan 10, 2025

Project Thread: paaHJt-7CE-p2
(Softly) Required By: a8c-ci-toolkit#135

Description

This PR removed the CI job that was responsible to trigger save_gradle_dependency_cache.

With this job present, every tag related build tries to save the dependencies in a cache, but, this functionality has been broken for a while, the example:build task is constantly failing on CI (see below).

Example Build: 1.23.1

FYI: No matter, we have decided to not use this project for this purpose, as as such to have a single dependency cache for all projects, and then restore this bigger than needed cache per project, only to use a specific set of dependencies per project. Instead, we decided it is better for each project to have a scheduled job, either weekly or daily, which will trigger this 'save_gradle_dependency_cache' script and only save the dependencies that it really needs.


Testing information

Just verify that all the CI checks are successful.

With this job present, every tag related build tries to save the
dependencies in a cache, but, this functionality has been broken for a
while, the 'example:build' task is constantly failing on CI (see below).

Example Build (1.23.1): https://buildkite.com/automattic/
android-dependency-catalog/builds/189
#01922323-d7e6-4526-82b5-f5dca9e6ed6a

FYI: No matter, we have decided to not use this project for this
purpose, as as such to have a single dependency cache for all projects,
and then restore this bigger than needed cache per project, only to use
a specific set of dependencies per project. Instead, we decided it is
better for each project to have a scheduled job, either weekly or daily,
which will trigger this 'save_gradle_dependency_cache' script and only
save the dependencies that it really needs.
@ParaskP7 ParaskP7 merged commit 37cd96f into trunk Jan 13, 2025
7 checks passed
@ParaskP7 ParaskP7 deleted the ci/remove-save-cache branch January 13, 2025 10:05
@ParaskP7 ParaskP7 changed the title [Dependency Cache] Remove Restore Cache from CI [Dependency Cache] Remove Save Cache from CI Jan 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants