Summary: | Piglit exits with "invalid literal for int() with base 10: '19225.json" | ||
---|---|---|---|
Product: | piglit | Reporter: | Hi-Angel <Hi-Angel> |
Component: | infrastructure | Assignee: | Dylan Baker <baker.dylan.c> |
Status: | RESOLVED FIXED | QA Contact: | Piglit Mailing List <piglit> |
Severity: | normal | ||
Priority: | medium | CC: | hector.edmundox.ramirez.gomez |
Version: | unspecified | ||
Hardware: | x86-64 (AMD64) | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: |
Description
Hi-Angel
2017-06-19 17:44:24 UTC
That is a very interesting bug. I'm not sure exactly how you hit that. Essentially the python framework is trying to split the '.json' off the end and turn the fist part into a number rather than a string, and failing. What does `ls results/master-d69f82c3c940467/tests/19225.json*` return? $ ls results/master-d69f82c3c940467/tests/19225.json* results/master-d69f82c3c940467/tests/19225.json results/master-d69f82c3c940467/tests/19225.json.tmp Ahhhhhh. Okay. That .tmp file is what's tripping it up. (In reply to Dylan Baker from comment #3) > > Ahhhhhh. Okay. That .tmp file is what's tripping it up. Well, other tmp files are probably fine $ ls results/master-d69f82c3c940467/tests/*.tmp results/master-d69f82c3c940467/tests/19218.json.tmp results/master-d69f82c3c940467/tests/19222.json.tmp results/master-d69f82c3c940467/tests/19224.json.tmp results/master-d69f82c3c940467/tests/19225.json.tmp What are they anyway? Will they may be not appear again if I remove and redownload whole piglit repo dir? Each of those json files is the result of a single test. Because piglit tries to make writes atomic when the test starts a <number>.json file is created that contains the PID of the test and the status "incomplete", when the test completes a new file <number>.json.tmp is created and moved over the original results. What's happening (I think) is that piglit is dying before the .tmp file is moved over the original file. It should be fairly straight forward to patch it, I'll link the patch here. In the mean time if you run `rm results/.../*.tmp`, and then `piglit summary aggregate results/...` (with ... as your full path) you can view the results as is. This series fixes the issue: https://patchwork.freedesktop.org/series/26257/ Thank you, yes, it seems to work! I'm not sure though if I can send my t-b because, for irrelevant reasons (GPU lockup), I can't finish a test run. However the 19225 test passes. I managed to finish a test run (the lockup intermittent), so I sent t-b into ML. *** Bug 101872 has been marked as a duplicate of this bug. *** I'm pretty sure the fix for this landed a long time ago. Feel free to reopen if not. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.