Stu Hood

Results 276 comments of Stu Hood

> 2. There is a [`setup_sandbox`](https://github.com/pantsbuild/pants/blob/221565246db4f31822ae63f6cc8cffb83dd6740a/src/rust/engine/process_execution/src/local.rs#L703-L712) workunit (which is disconnected from any relevant parent workunit: likely a bug), which could only really be running for so long because it is...

> * I assume that you are running with `cache_content_behavior="fetch"` (the default)? If so, you might try capturing another trace with `cache_content_behavior="validate"` to see how much of a difference that...

> Without changing the semantics of `CacheContentBehavior` (because we _do_ still want to guarantee that all content exists locally when "fetch" is set), this suggests that we should optimize the...

Drafted here: https://github.com/pantsbuild/pants/pull/16928 ... will land if it doesn't hit some other timeout in the same file instead.

It doesn't look like that was it: ``` 17:28:54.85 [ERROR] Completed: Run Pytest - src/python/pants/backend/python/util_rules/pex_test.py:tests failed (exit code -15). ============================= test session starts ============================== collecting ... collected 27 items src/python/pants/backend/python/util_rules/pex_test.py::test_pex_execution[True-Pex]...

> If you haven't tried yet, running with `--debug` locally may be instructive Doesn't repro locally... finishes in a minute or two. But I haven't yet tried to repro without...

> I repro this locally, and in #16910 > > https://pantsbuild.slack.com/archives/C0D7TNJHL/p1663758370882199?thread_ts=1662998340.585589&channel=C0D7TNJHL&message_ts=1663758370.882199 Ooh: if you repro locally, I'm very curious what you're seeing in stack traces via `py-spy`, both in the...

Fixed by #16939: thanks @tdyas!

> Generally, I also believe it is unsafe to default to the possibly dangerous thing, marking something as platform-agnostic. Instead, we should default to the safe semantics, and let others...

In the context of #17354, this becomes very relevant again. The key takeaway from that issue is that processes themselves can't really be truly platform independent (they need to run...