Skip to content

Conversation

@sluongng
Copy link
Contributor

@sluongng sluongng commented Dec 8, 2025

In open source remote execution setup, request metdata let the server group
cache requests using a build (invocation) and build targets.

In open source remote execution setup, request metdata let the server group
cache requests using a build (invocation) and build targets.
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 8, 2025
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Dec 8, 2025

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this in D88638126. (Because this pull request was imported automatically, there will not be any future comments.)

@sluongng
Copy link
Contributor Author

sluongng commented Dec 9, 2025

worth noting that there are still a few requests that don't come with metadata / identity attached.

sluongng added a commit to sluongng/buck2 that referenced this pull request Dec 9, 2025
Instead of making 'identity' optional in some of the request paths, make the fields inside
ReActionIdentity optional instead.

This enables us to mandate 'identity' for all call RBE client consumer.
We simplify the creation of the identity by introducing a minimal constructor to use in
different places with insufficient data to build a full one.

This change is based on facebook#1169
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant