UpdateActionResult uploading Action/Command Inputs

[Stu Hood]

Hey folks!

UpdateActionResult indicates that the Action and Command MUST first be uploaded to the CAS. But it doesn't specify whether that is only those root protos, or also all of the transitive input content of the Command.

Intuitively, hitting the ActionCache does not require any input content, but as the text states, having the inputs uploaded would likely be useful for debugging or access control.

How are other clients handling this? And if clients are mostly not eagerly uploading, is it worth clarifying that clients MAY upload some/all of the inputs? A middle ground might be to upload only directory listings but no file content.

Thanks!

--

@stuhood

[Stu Hood]

Based on responses in the BuildTeam slack, neither BuildBarn nor BuildFarm require inputs to be uploaded. Unless there are other servers chiming in with usecases for the content, I'm leaning toward adding a clarification to the spec that the input content is not necessary.

--

@stuhood