Skip to content
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

[7.0.0] Write the action digest to the execution log also for cache misses. #19984

Merged
merged 2 commits into from
Oct 30, 2023

Conversation

bazel-io
Copy link
Member

Specifically, introduce a getter/setter pair to store the digest in the SpawnExecutionContext when looking up the disk or remote cache, and retrieve it later when constructing the SpawnResult.

Fixes #16741.

Commit 40d6780

PiperOrigin-RevId: 577799982
Change-Id: I570db74f4dbeee95e00853883eda68b25a84934c

Specifically, introduce a getter/setter pair to store the digest in the SpawnExecutionContext when looking up the disk or remote cache, and retrieve it later when constructing the SpawnResult.

Fixes bazelbuild#16741.

PiperOrigin-RevId: 577799982
Change-Id: I570db74f4dbeee95e00853883eda68b25a84934c
@bazel-io bazel-io requested a review from a team as a code owner October 30, 2023 15:55
@bazel-io bazel-io added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Oct 30, 2023
@github-actions github-actions bot added team-Local-Exec Issues and PRs for the Execution (Local) team team-Performance Issues for Performance teams labels Oct 30, 2023
@iancha1992 iancha1992 requested a review from tjgq October 30, 2023 17:36
@iancha1992 iancha1992 removed team-Performance Issues for Performance teams team-Local-Exec Issues and PRs for the Execution (Local) team labels Oct 30, 2023
@iancha1992 iancha1992 enabled auto-merge (squash) October 30, 2023 17:47
@iancha1992 iancha1992 merged commit 69feddb into bazelbuild:release-7.0.0 Oct 30, 2023
31 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Oct 30, 2023
@tjgq tjgq changed the title [7.0.0] Execlog parser should output each action's digest [7.0.0] Write the action digest to the execution log also for cache misses. Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Remote-Exec Issues and PRs for the Execution (Remote) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants