Skip to content

Add strace hack for tarfile test #12625

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.

Sign up for GitHub

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

Jump to bottom
Closed
zooba wants to merge 6 commits into from
Closed

Add strace hack for tarfile test #12625

zooba 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

tirkarthi commented Mar 30, 2019

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 zooba mentioned this pull request Mar 30, 2019
Merged
@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

@tirkarthi tirkarthi mentioned this pull request Mar 31, 2019
Merged
@zooba zooba mentioned this pull request Apr 1, 2019
Merged
@zooba zooba closed this Apr 1, 2019
@zooba zooba deleted the strace branch April 1, 2019 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

awaiting core review DO-NOT-MERGE skip issue skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@zooba @tirkarthi @the-knights-who-say-ni @bedevere-bot