TraceMachina / nativelink

🔥🔥🔥 Bazel RBE with CAS server implementation in Rust. The free and open source cache and remote execution service, prioritizing stability and speed for the people that need it.

https://docs.nativelink.com

Apache License 2.0

235 stars 46 forks source link

Include ActionState to MatchingEngineActionStateResult #1064

Closed adam-singer closed 4 days ago

adam-singer commented 4 days ago

Description

Refactoring to include the awaited_action watch channel notify_channel: watch::Sender<Arc<ActionState>> to be provided via MatchingEngineActionStateResult. This aids the matching engine's need for access to OperationId.

Fixes https://github.com/TraceMachina/nativelink/issues/1063

Type of change

Please delete options that aren't relevant.

[x] Refactor

Checklist

[x] bazel test //... passes locally
[x] PR is contained in a single commit, using git amend see some docs

This change is