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

os: TestGetwdDeepWithPWDSet failing on android trybots #69406

Closed
timothy-king opened this issue Sep 11, 2024 · 3 comments
Closed

os: TestGetwdDeepWithPWDSet failing on android trybots #69406

timothy-king opened this issue Sep 11, 2024 · 3 comments
Assignees
Labels
NeedsFix The path to resolution is known, but the work has not been done. OS-Android
Milestone

Comments

@timothy-king
Copy link
Contributor

The new TestGetwdDeepWithPWDSet is failing on android trybots consistently (https://build.golang.org/).

Example output https://build.golang.org/log/1d3e4c2e0139d5c2e8f5169f4aa95d27517f862a:

--- FAIL: TestGetwdDeep (0.00s)
    getwd_unix_test.go:61: Getwd len: 274
    getwd_unix_test.go:61: Getwd len: 475
    getwd_unix_test.go:61: Getwd len: 676
    getwd_unix_test.go:61: Getwd len: 877
    getwd_unix_test.go:61: Getwd len: 1078
    getwd_unix_test.go:61: Getwd len: 1279
    getwd_unix_test.go:61: Getwd len: 1480
    getwd_unix_test.go:61: Getwd len: 1681
    getwd_unix_test.go:61: Getwd len: 1882
    getwd_unix_test.go:61: Getwd len: 2083
    getwd_unix_test.go:61: Getwd len: 2284
    getwd_unix_test.go:61: Getwd len: 2485
    getwd_unix_test.go:61: Getwd len: 2686
    getwd_unix_test.go:61: Getwd len: 2887
    getwd_unix_test.go:61: Getwd len: 3088
    getwd_unix_test.go:61: Getwd len: 3289
    getwd_unix_test.go:61: Getwd len: 3490
    getwd_unix_test.go:61: Getwd len: 3691
    getwd_unix_test.go:61: Getwd len: 3892
    getwd_unix_test.go:61: Getwd len: 4093
    getwd_unix_test.go:61: Getwd len: 0
    getwd_unix_test.go:69: open ../../../../../../../../../../../../../../../../../../../../../../../../../..: permission denied
--- FAIL: TestGetwdDeepWithPWDSet (0.00s)
    getwd_unix_test.go:57:  $PWD len: 284
    getwd_unix_test.go:61: Getwd len: 284
    getwd_unix_test.go:57:  $PWD len: 485
    getwd_unix_test.go:61: Getwd len: 485
    getwd_unix_test.go:57:  $PWD len: 686
    getwd_unix_test.go:61: Getwd len: 686
    getwd_unix_test.go:57:  $PWD len: 887
    getwd_unix_test.go:61: Getwd len: 887
    getwd_unix_test.go:57:  $PWD len: 1088
    getwd_unix_test.go:61: Getwd len: 1088
    getwd_unix_test.go:57:  $PWD len: 1289
    getwd_unix_test.go:61: Getwd len: 1289
    getwd_unix_test.go:57:  $PWD len: 1490
    getwd_unix_test.go:61: Getwd len: 1490
    getwd_unix_test.go:57:  $PWD len: 1691
    getwd_unix_test.go:61: Getwd len: 1691
    getwd_unix_test.go:57:  $PWD len: 1892
    getwd_unix_test.go:61: Getwd len: 1892
    getwd_unix_test.go:57:  $PWD len: 2093
    getwd_unix_test.go:61: Getwd len: 2093
    getwd_unix_test.go:57:  $PWD len: 2294
    getwd_unix_test.go:61: Getwd len: 2294
    getwd_unix_test.go:57:  $PWD len: 2495
    getwd_unix_test.go:61: Getwd len: 2495
    getwd_unix_test.go:57:  $PWD len: 2696
    getwd_unix_test.go:61: Getwd len: 2696
    getwd_unix_test.go:57:  $PWD len: 2897
    getwd_unix_test.go:61: Getwd len: 2897
    getwd_unix_test.go:57:  $PWD len: 3098
    getwd_unix_test.go:61: Getwd len: 3098
    getwd_unix_test.go:57:  $PWD len: 3299
    getwd_unix_test.go:61: Getwd len: 3299
    getwd_unix_test.go:57:  $PWD len: 3500
    getwd_unix_test.go:61: Getwd len: 3500
    getwd_unix_test.go:57:  $PWD len: 3701
    getwd_unix_test.go:61: Getwd len: 3701
    getwd_unix_test.go:57:  $PWD len: 3902
    getwd_unix_test.go:61: Getwd len: 3902
    getwd_unix_test.go:57:  $PWD len: 4103
    getwd_unix_test.go:61: Getwd len: 0
    getwd_unix_test.go:69: open ../../../../../../../../../../../../../../../../../../../../../../../../..: permission denied

Assigning to @ianlancetaylor as he has worked on this test previously.

@timothy-king timothy-king added OS-Android NeedsFix The path to resolution is known, but the work has not been done. labels Sep 11, 2024
@timothy-king timothy-king added this to the Backlog milestone Sep 11, 2024
@ianlancetaylor
Copy link
Member

The fix is https://go.dev/cl/610756, which I sent out for review last week. Anybody want to review it?

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/610756 mentions this issue: os: check for EACCES, not EPERM, in Getwd test

@dmitshur dmitshur modified the milestones: Backlog, Go1.24 Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NeedsFix The path to resolution is known, but the work has not been done. OS-Android
Projects
None yet
Development

No branches or pull requests

5 participants