I think there's a race condition somewhere, when I run the command you gave me multiple times (python_d.exe -m test.bisect_cmd -j1 test_distutils -v --fail-env-changed), I get different results:
changed the title [-]On Windows, test_distutils leaks a temporary file: Microsoft[/-][+]On Windows, test_distutils and test_peg_generator leak a temporary file: Microsoft[/+]on Oct 10, 2022
I've had a chance to look into this, and it looks like what is actually left is an empty directory tree: Microsoft\VSApplicationInsights\vstelAIF-312cbd79-9dbb-4c48-a7da-3cc1a931cb70\ (that last part may not be the same every time).
I've searched around and found no way to disable this, but it may or may not be related to the worker being deployed on Azure. @zooba or @eryksun, if you have any suggestions for making this go away I'm open to them, but I wonder if it wouldn't be better to just ignore a directory called Microsoft in the leaked temporary files check.
Hmm. It turns out that an update to VS (and an attempt to trim down the installed components) seems like it may have resolved this; the latest build (which was a rebuild of the previous failure) passed.
Activity
vstinner commented on Jun 16, 2022
Same issue with test_peg_generator:
Fidget-Spinner commented on Jun 17, 2022
Warning -- test_distutils leaked temporary files (1): Microsoft == Tests result: ENV CHANGED == 1 test altered the execution environment: test_distutils Total duration: 2.8 sec Tests result: ENV CHANGED ran 1 tests/3 exit 3 Tests failed: continuing with this subtest Tests (1): * distutils.tests.test_build_ext.BuildExtTestCase.test_build_ext Bisection completed in 16 iterations and 0:00:38Fidget-Spinner commented on Jun 17, 2022
I think there's a race condition somewhere, when I run the command you gave me multiple times (python_d.exe -m test.bisect_cmd -j1 test_distutils -v --fail-env-changed), I get different results:
vstinner commented on Jun 17, 2022
bisect_cmd picks any test which reproduces the issue. So at least 4 tests leak a temporary file :-)
vstinner commented on Oct 10, 2022
I marked my issue #98168 as a duplicate of this issue.
vstinner commented on Nov 3, 2022
Sadly, I don't have the bandwidth to work on this issue, I just close it.
zware commented on May 29, 2023
I've had a chance to look into this, and it looks like what is actually left is an empty directory tree: Microsoft\VSApplicationInsights\vstelAIF-312cbd79-9dbb-4c48-a7da-3cc1a931cb70\ (that last part may not be the same every time).
I've searched around and found no way to disable this, but it may or may not be related to the worker being deployed on Azure. @zooba or @eryksun, if you have any suggestions for making this go away I'm open to them, but I wonder if it wouldn't be better to just ignore a directory called Microsoft in the leaked temporary files check.
zware commented on May 29, 2023
Hmm. It turns out that an update to VS (and an attempt to trim down the installed components) seems like it may have resolved this; the latest build (which was a rebuild of the previous failure) passed.