https://pantsbuild.org/ logo
n

narrow-vegetable-37489

05/04/2022, 10:38 PM
What's the best way to go about debugging why Pants is behaving as if both
colors
and
dynamic_ui
are off? It happened to me randomly the other day on 2.11.0rc2 but sorted itself out after a while for unknown reasons, and today it happened to a co-worker on 2.11.0 as well.
h

hundreds-father-404

05/04/2022, 10:44 PM
I think I'd recommend explicitly setting them and see if it still turns off. It might be that our default of seeing if you're a TTY isn't working for some reason sometimes
n

narrow-vegetable-37489

05/04/2022, 10:53 PM
Explicitly setting
colors
and
dynamic_ui
did the trick, and after removing them they stay enabled weirdly enough. He did also kill pantsd though, not sure if pantsd handles stuff like that too. Now the only problem that remains is that pytest in similar sudden fashion refuses to output what was printed to stdout on failing tests..
h

hundreds-father-404

05/04/2022, 10:54 PM
Now the only problem that remains is that pytest in similar sudden fashion refuses to output what was printed to stdout on failing tests..
Hmmmmm you're using modern pants right? I can't remember which version but we did used to have an egregious issue that sometimes output would be missing. Pretty sure that is fixed now
n

narrow-vegetable-37489

05/04/2022, 10:54 PM
2.11.0, yep 🙂
1
b

bitter-ability-32190

05/04/2022, 10:57 PM
Oddly I thought I saw this too
1
My suspicion I never validated was that I ran with them off and the daemon launched, and then it never invalidated when they were back on
👀 1
h

hundreds-father-404

05/04/2022, 10:58 PM
Huh, that would be concerning and high priority bug. Is it flaky or consistent?
b

bitter-ability-32190

05/04/2022, 10:58 PM
I didn't have enough runtime to say. My suspicion should be easy to validate
n

narrow-vegetable-37489

05/04/2022, 11:09 PM
Turns out the pytest stuff was a human error. The TTY issue is still weird though..
👍 1
w

witty-crayon-22786

05/04/2022, 11:26 PM
there are definitely issues around logging levels not triggering
pantsd
restarts… but color and dynamic-ui-ness should definitely be per-client session, and not dependent on the server
bug very appreciated if not
(a quick smoke test here would seem to confirm that both are respected per-session)
b

bitter-ability-32190

05/06/2022, 1:24 AM
I'm running into this on
main
in the Pants repo right now 😮
pants-1651800301867.mp4
(unfortunately powering down computer now. One thing of note was it was a-ok until I started a concurrent run (from `git commit`'s pre-commit check). Once the in-terminal run stopped waiting I lost color and dynamic UI)