gh-111877: Fixes stat() handling for inaccessible files on Windows#113716
Conversation
|
!buildbot windows |
|
🤖 New build scheduled with the buildbot fleet by @zooba for commit 8b29604 🤖 The command will test the builders whose names match following regular expression: The builders matched are:
|
|
Buildbots look fine, the one failure is unrelated and doesn't repro across the rest, so I'm choosing to ignore it. |
|
Thanks @zooba for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12. |
…ows (pythonGH-113716) (cherry picked from commit ed06648) Co-authored-by: Steve Dower <steve.dower@python.org>
|
GH-113989 is a backport of this pull request to the 3.12 branch. |
|
Not sure this is the best place to add this, but since this change it appears that my Windows 10 buildbot consistently fails the new test during the main run, but passes upon retry. I'm not sure if it indicates an issue with the test or the environment, but it's never passed at any point the first time, which seems odd - maybe some sort of race that only shows up when the tests are being run in parallel? The same thing started happening on the 3.12 branch as soon as it was backported. |
|
@db3l Can you link one of these failures? Or copy in the error text |
|
Sorry, my bad. Here's a recent 3.x run: and the actual failure during the main run (which seems a true failure, as without access still gets a result): The retry at the end of the test run always seems to succeed, but even though the runner says it's retrying in verbose mode I don't get the output from the test, but I expect that could just be an issue in the retry logic. I've also since noticed that Itamar's windows buildbot seems to be getting the same results, so a test issue seems more likely than buildbot - perhaps something with the icacls (actual execution, or latency) during the regular portion of the run that isn't a subprocess failure? The test itself seems fine to me at first glance otherwise. |
|
Yeah, looks like it's failing to reset permissions properly. I wonder if it's because of the Unicode filename, or possibly because it's not using a fully qualified path? I'll try making a full path first, I guess. I'll ping you on a new PR and we can continue there. |
Uh oh!
There was an error while loading. Please reload this page.