This issue tracker has been migrated to GitHub ,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2012年07月12日 20:42 by pitrou, last changed 2022年04月11日 14:57 by admin. This issue is now closed.
| Messages (12) | |||
|---|---|---|---|
| msg165333 - (view) | Author: Antoine Pitrou (pitrou) * (Python committer) | Date: 2012年07月12日 20:42 | |
On the Win64 buildbot, trying to access an UNC path raises PermissionError, which makes a test fail: ====================================================================== ERROR: test_UNC_path (test.test_import.PathsTests) ---------------------------------------------------------------------- Traceback (most recent call last): File "D:\buildbot.python.org3円.x.kloth-win64\build\lib\test\test_import.py", line 469, in test_UNC_path os.listdir(unc) PermissionError: [Error 5] Access is denied: '\\\\hades\\D$\\buildbot.python.org\3円.x.kloth-win64\\build\\build\\test_python_3612\\@test_3612_tmp\\*.*' If this is expected or normal, maybe we should simply skip the test when the listdir() call above fails. |
|||
| msg165334 - (view) | Author: Antoine Pitrou (pitrou) * (Python committer) | Date: 2012年07月12日 20:43 | |
Example URL for the aforementioned failure: http://buildbot.python.org/all/builders/AMD64%20Windows7%20SP1%203.x/builds/282/steps/test/logs/stdio |
|||
| msg165374 - (view) | Author: Tim Golden (tim.golden) * (Python committer) | Date: 2012年07月13日 12:29 | |
Well os.listdir doesn't fail to access a UNC path on Windows x64 in general. So presumably this particular path is not accessible by the buildbot process owner? |
|||
| msg165376 - (view) | Author: Antoine Pitrou (pitrou) * (Python committer) | Date: 2012年07月13日 12:36 | |
Well, this particular path is the build directory itself, so it's certainly accessible through the normal (non-UNC) path. There has to be something else :-) Jeremy told me his buildbot process runs as a service, perhaps that is related? |
|||
| msg165377 - (view) | Author: Tim Golden (tim.golden) * (Python committer) | Date: 2012年07月13日 12:39 | |
It's using an administrative share (\\server\d$) and those are usually restricted with share permissions -- different from NTFS permissions. That the process runs as a service is likely to have an effect since services are conventionally run with minimum-privilege accounts. |
|||
| msg165379 - (view) | Author: Antoine Pitrou (pitrou) * (Python committer) | Date: 2012年07月13日 12:46 | |
> It's using an administrative share (\\server\d$) and those are usually > restricted with share permissions -- different from NTFS permissions. > That the process runs as a service is likely to have an effect since > services are conventionally run with minimum-privilege accounts. Ok, so I guess skipping the test in this case would be appropriate? |
|||
| msg165380 - (view) | Author: Tim Golden (tim.golden) * (Python committer) | Date: 2012年07月13日 13:08 | |
For this particular buildbot setup, maybe yes. But it would be possible in principle to have a buildbot configuration which could allow the test to execute. (eg one running under a user account which can access the path via an admin share). Does the buildbot owner have anything to say on the account and share permissions? |
|||
| msg165383 - (view) | Author: Jeremy Kloth (jeremy.kloth) | Date: 2012年07月13日 14:00 | |
The buildbot service account is a standard user (per the buildbot servce installation directions). When logged on, the user can access the share. Just when logged on as a service is when it cannot. After much searching, I still cannot find any information on how to get that user to access to those shares. |
|||
| msg165388 - (view) | Author: Tim Golden (tim.golden) * (Python committer) | Date: 2012年07月13日 15:22 | |
My guess is that it's to do with Service Hardening. I did a quick dump of my token in an interactive session and as the owner of a service. Quite a few differences. I haven't read up on this area yet so I'm not sure what options there are / how easy to get the necessary privs. |
|||
| msg165390 - (view) | Author: Tim Golden (tim.golden) * (Python committer) | Date: 2012年07月13日 15:32 | |
Simplest solution might be to catch PermissionError and call skipTest from within. This would allow buildbots to run the test which had access through the relevant share. |
|||
| msg165402 - (view) | Author: Roundup Robot (python-dev) (Python triager) | Date: 2012年07月13日 19:01 | |
New changeset 4d480a2a6296 by Antoine Pitrou in branch '3.2': Issue #15338: skip test_UNC_path when the current user doesn't have enough permissions to access the path. http://hg.python.org/cpython/rev/4d480a2a6296 New changeset 7d5e84a44b82 by Antoine Pitrou in branch 'default': Issue #15338: skip test_UNC_path when the current user doesn't have enough permissions to access the path. http://hg.python.org/cpython/rev/7d5e84a44b82 |
|||
| msg165403 - (view) | Author: Roundup Robot (python-dev) (Python triager) | Date: 2012年07月13日 19:08 | |
New changeset 7eac87fa7a06 by Antoine Pitrou in branch '2.7': Issue #15338: skip test_UNC_path when the current user doesn't have enough permissions to access the path. http://hg.python.org/cpython/rev/7eac87fa7a06 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:57:32 | admin | set | github: 59543 |
| 2012年07月13日 22:33:04 | pitrou | set | status: open -> closed resolution: fixed stage: resolved |
| 2012年07月13日 19:08:01 | python-dev | set | messages: + msg165403 |
| 2012年07月13日 19:01:22 | python-dev | set | nosy:
+ python-dev messages: + msg165402 |
| 2012年07月13日 15:32:14 | tim.golden | set | messages: + msg165390 |
| 2012年07月13日 15:22:39 | tim.golden | set | messages: + msg165388 |
| 2012年07月13日 14:00:01 | jeremy.kloth | set | messages: + msg165383 |
| 2012年07月13日 13:11:11 | pitrou | set | nosy:
+ jkloth |
| 2012年07月13日 13:08:33 | tim.golden | set | messages: + msg165380 |
| 2012年07月13日 12:46:01 | pitrou | set | messages: + msg165379 |
| 2012年07月13日 12:40:00 | tim.golden | set | messages: + msg165377 |
| 2012年07月13日 12:36:52 | pitrou | set | messages: + msg165376 |
| 2012年07月13日 12:29:37 | tim.golden | set | messages: + msg165374 |
| 2012年07月12日 20:43:20 | pitrou | set | messages: + msg165334 |
| 2012年07月12日 20:42:41 | pitrou | create | |