Skip to content

fix missing content digest in git#11756

Merged
sipsma merged 2 commits intodagger:mainfrom
sipsma:git-flakes
Jan 28, 2026
Merged

fix missing content digest in git#11756
sipsma merged 2 commits intodagger:mainfrom
sipsma:git-flakes

Conversation

@sipsma
Copy link
Contributor

@sipsma sipsma commented Jan 28, 2026

Missed this during previous effort that split up recipe+content digests. Seems to have been causing flakes in CI where wrong auth was used for git ops. Setting the content digest rather than recipe digest fixes it since it keeps the recipe consistent even when git ops using different auth happen to result in the same directory.

@sipsma sipsma requested a review from a team as a code owner January 28, 2026 21:46
Missed this during previous effort that split up recipe+content digests.
Seems to have been causing flakes in CI where wrong auth was used for
git ops. Setting the content digest rather than recipe digest fixes it
since it keeps the recipe consistent even when git ops using different
auth happen to result in the same directory.

Signed-off-by: Erik Sipsma <erik@sipsma.dev>
Signed-off-by: Erik Sipsma <erik@sipsma.dev>
@sipsma sipsma merged commit acca01c into dagger:main Jan 28, 2026
51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants