Skip cache restore in update-helm job - #178
Draft
bschwedler wants to merge 1 commit into
Draft
Conversation
update-helm doesn't pass cache-scope, so its dependency-version cache artifact name never matches the build job's writer -- the restore always misses and logs a confusing "Artifact not found" error on every run (posit-dev/images-shared#686). Opt out via the new restore-cache input. Depends on posit-dev/images-shared's setup-bakery gaining the restore-cache input -- this workflow pins @main, so CI here won't pass until that change lands on images-shared's default branch.
Closed
5 tasks
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
update-helm doesn't pass cache-scope, so its dependency-version cache
artifact name never matches the build job's writer -- the restore
always misses and logs a confusing "Artifact not found" error on
every run.
Opt out via the new restore-cache input on setup-bakery.
Blocked on the images-shared PR below merging to main -- this
workflow pins setup-bakery@main, and composite actions reject
unrecognized inputs, so CI here will fail until that lands. Opening
as a draft for that reason.