FYI It hurts to pay for 43 minutes of Python test ...
# developmentb
bitter-ability-32190
01/12/2022, 5:07 PMFYI It hurts to pay for 43 minutes of Python test time when making PRs to Pants š¬
https://github.com/pantsbuild/pants/runs/4791148287?check_suite_focus=true#step:14:476
Is remote caching supposed to be enabled?
I see
15:13:33.51 [WARN] [auth-plugin] Failed to load Toolchain token from env var 'TOOLCHAIN_AUTH_TOKEN'. Please make sure the env var is set in your environment.ā 1
w
witty-crayon-22786
01/12/2022, 5:25 PMso: yes, it is enabled. but it uses delegated auth to confirm that the PR submitter is authorized to use the Toolchain cache.
witty-crayon-22786
01/12/2022, 5:26 PMand: we began a process last week to open that up to all Contributors
witty-crayon-22786
01/12/2022, 5:27 PMthe caching helps quite a bit: https://github.com/pantsbuild/pants/runs/4783397469?check_suite_focus=true ā¦ sorry that you havenāt had access to it so far
b
bitter-ability-32190
01/12/2022, 5:31 PMWhy does the PR submitter matter in this case?
w
witty-crayon-22786
01/12/2022, 5:32 PMa malicious PR could write whatever it wanted to into the cache, and corrupt it
š¤ 1
witty-crayon-22786
01/12/2022, 5:33 PMyou canāt corrupt the CAS, since itās verified by the server as content addressable.
witty-crayon-22786
01/12/2022, 5:33 PMbut the cache itself cannot be checked by the serverā¦ process in, āsome process outputā out.
witty-crayon-22786
01/12/2022, 7:26 PMbut yea. stay tuned here: hopefully we can have it enabled in the next few weeks
š 2