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.1] Add tree artifact metadata for the coverage post-processing spawn, Rewrite paths of writable directories that are under the execroot, Always add containing trees of input tree file artifacts to the metadata map #20819

Closed

Conversation

bazel-io
Copy link
Member

@bazel-io bazel-io commented Jan 9, 2024

This makes --experimental_split_coverage_postprocessing work in the wake of fb6658c: before, it was enough to add the metadata of the tree file artifacts to the metadata provider of the post-processing action, but that change made the metadata of the tree artifact necessary, too.

Progress towards #20753.

RELNOTES: None.
Commit b0db044

PiperOrigin-RevId: 596929659
Change-Id: I481ef36328de7f7ab07f2ec7a0ac83d5fd508c36

This makes --experimental_split_coverage_postprocessing work in the wake of bazelbuild@fb6658c: before, it was enough to add the metadata of the tree file artifacts to the metadata provider of the post-processing action, but that change made the metadata of the tree artifact necessary, too.

Progress towards bazelbuild#20753.

RELNOTES: None.
PiperOrigin-RevId: 596929659
Change-Id: I481ef36328de7f7ab07f2ec7a0ac83d5fd508c36
@iancha1992
Copy link
Member

Duplicate

@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Jan 9, 2024
@meteorcloudy
Copy link
Member

Duplicate of?

@iancha1992
Copy link
Member

iancha1992 commented Jan 11, 2024

@meteorcloudy Forgot to mention #20821 has this cherry-picked. Therefore, we don't need to include this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Local-Exec Issues and PRs for the Execution (Local) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants