Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add strace hack for tarfile test #12625

Closed
wants to merge 6 commits into from
Closed

Add strace hack for tarfile test #12625

wants to merge 6 commits into from

Conversation

zooba
Copy link
Member

@zooba zooba commented Mar 30, 2019

This should never be merged. I'm trying to collect more info about a test failure.

@tirkarthi
Copy link
Member

cc : @vstinner @pablogsal since they look after buildbots and pipelines with this being a failure in Ubuntu builds in Azure pipelines.

@zooba
Copy link
Member Author

zooba commented Mar 30, 2019

If you look on the summary page of the Pipelines build, there'll be a build artifact available that (hopefully) will include the strace log:

image

Right now it has an error that I think means I needed "sudo strace", so trying that now.

@zooba
Copy link
Member Author

zooba commented Mar 30, 2019

Here is the output of strace for test_tarfile (including a time.sleep at the start, and a time.sleep/os.kill at the end):

strace: Process 5154 attached
select(0, NULL, NULL, NULL, {1, 828371}) = 0 (Timeout)
umask(022)                              = 022
open("/home/vsts/work/1/s/build/test_python_5154/@test_5154_tmp-tardir/tmp.tar", O_WRONLY|O_CREAT|O_TRUNC|O_CLOEXEC, 0666) = 3
write(3, "BZh91AY&SYY{\270\344\0\0\24P\0\300\0\4\0\0\10 \0000\314\5)\246"..., 46) = 46
close(3)                                = 0
stat("/home/vsts/work/1/s/build/test_python_5154/@test_5154_tmp-tardir/tmp.tar", {st_mode=S_IFREG|0666, st_size=46, ...}) = 0
umask(022)                              = 022
select(0, NULL, NULL, NULL, {5, 0})     = 0 (Timeout)
kill(5154, SIGKILL <unfinished ...>
+++ killed by SIGKILL +++

As near as I can tell, nothing unexpected here? @njsmith any ideas?

@zooba
Copy link
Member Author

zooba commented Mar 30, 2019

This is the output of checking ACLs on the sources directory (which is where temp files get put for these tests):

Default mount options:    user_xattr acl
/dev/sda1 on / type ext4 (rw,relatime,discard,data=ordered)
# file: .
# owner: vsts
# group: docker
user::rwx
user:vsts:rwx
group::rwx
mask::rwx
other::rwx
default:user::rwx
default:user:vsts:rwx
default:group::rwx
default:mask::rwx
default:other::rwx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants