Skip to content

Only write images to cache from the Build job #1702

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Mar 15, 2024
Merged

Conversation

lucacome
Copy link
Contributor

@lucacome lucacome commented Mar 15, 2024

Proposed changes

Problem: There are multiple workflows and jobs all writing to the same
cache causing it to be invalidated

Solution: Write to cache only in the Build job and read the cache from
all the other jobs.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes

If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

NONE

@github-actions github-actions bot added the chore Pull requests for routine tasks label Mar 15, 2024
@lucacome lucacome force-pushed the chore/build-cache branch 5 times, most recently from 3261b70 to f923b14 Compare March 15, 2024 21:23
Problem: There are multiple workflows and jobs all writing to the same
cache causing it to be invalidated

Solution: Write to cache only in the Build job and read the cache from
all the other jobs.
@lucacome lucacome force-pushed the chore/build-cache branch from f923b14 to 38f9af8 Compare March 15, 2024 21:25
@lucacome lucacome marked this pull request as ready for review March 15, 2024 21:25
@lucacome lucacome requested a review from a team as a code owner March 15, 2024 21:25
@sjberman
Copy link
Collaborator

We'll have to update the required checks at the repo level to the new jobs.

@lucacome lucacome force-pushed the chore/build-cache branch from 2c7b05c to 880b0a6 Compare March 15, 2024 22:07
@lucacome lucacome merged commit fe3bc2a into main Mar 15, 2024
@lucacome lucacome deleted the chore/build-cache branch March 15, 2024 22:51
amimimor pushed a commit to amimimor/nginx-gateway-fabric that referenced this pull request Apr 3, 2024
Problem: There are multiple workflows and jobs all writing to the same
cache causing it to be invalidated

Solution: Write to cache only in the Build job and read the cache from
all the other jobs.
miledxz added a commit to miledxz/nginx-gateway-fabric that referenced this pull request Jan 14, 2025
Problem: There are multiple workflows and jobs all writing to the same
cache causing it to be invalidated

Solution: Write to cache only in the Build job and read the cache from
all the other jobs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Pull requests for routine tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants