=== hugbot is now known as sowrdsmanz === gerald is now known as Guest57402 === sowrdsmanz is now known as swordsmanz [10:14] I fixed the permissions error I was seeing running the tests with: http://paste.ubuntu.com/9832685/ [10:14] That copies the files to a temporary location before doing the permission munging. [10:30] Odd_Bloke, i don't htink i get why that helps any [10:30] Odd_Bloke, unless ... you arn't running as "USER" [10:30] $USER [12:05] apw: My best guess is that something is modifying the contents/permissions of self.autotest_results_root after the chown call happens; my code effectively takes a snapshot which avoids that happening. [12:05] But I'm not sure I would recommend it for inclusion upstream. [12:13] Odd_Bloke, the window between the chown and the cp is like near immediate, i just don't buy that [12:14] Odd_Bloke, can we check we are running as $USER there, as if $USER was something other than the name of the user we are, then we'd not be able to read them, the extra copy may be adding permissing to them or something [12:16] apw: Sure, will do. [12:23] user there ends up as 'ubuntu', as expected. :( [12:27] Odd_Bloke, so what was the actual errror from the copy as "ubuntu" [12:28] apw: "[Errno 13] Permission denied: 'autotest/client/results/default/ubuntu_qrt_kernel_panic.test-kernel-panic.py/debug/crash.amd64_killer.24949/report'" [12:30] result, output = sh(r'sudo find %s -print | xargs chmod u+r' % (self.autotest_results_root)) [12:30] what about adding that after the chmod +x line [12:30] as these files belong to you, and yet you cannot read them, wheresa root would be able to, on copy may well add u+r to them [12:31] That file was still owned by root:root when I examined it. [12:32] k [12:36] I really wish I could get this reproducible with less than a 2 hour test run. :p [13:16] Odd_Bloke, it was still root? that seems, unexpectede [13:16] Yes, indeed. [13:17] I'm re-running the full suite (in lieu of a more concise reproduction) unchanged; will have another look at it then. [14:08] apw: http://paste.ubuntu.com/9835165/ is ls -lah output of the appropriate directory. [14:11] apw: http://paste.ubuntu.com/9835173/ is the stat output. [14:11] So it looks like report is created after core is modified (and copied?; guessing that's what the 'access' is). === swordsmanz is now known as hugbot === adam_g_out is now known as adam_g === pgraner is now known as pgraner-afk