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

TestSyncManager_upload download_folder_POSIX_perm_with_metadata fail on macOS #8769

Open
nopcoder opened this issue Mar 6, 2025 · 1 comment
Assignees
Labels
infrastructure build, deploy and release processes

Comments

@nopcoder
Copy link
Contributor

nopcoder commented Mar 6, 2025

Running go test ./... on lakeFS project fails locally on my macOS (darwin/arm64) with the following error.

Note that this test creates files under my home folder, which is not expected and not a practice.

--- FAIL: TestSyncManager_upload (0.04s)
    --- FAIL: TestSyncManager_upload/download_folder_POSIX_perm_with_metadata (0.01s)
        sync_test.go:299:
                Error Trace:    /Users/barak/work/lakeFS/pkg/local/sync_test.go:299
                                                        /Users/barak/sdk/go1.23.5/src/net/http/server.go:2220
                                                        /Users/barak/sdk/go1.23.5/src/net/http/server.go:3210
                                                        /Users/barak/sdk/go1.23.5/src/net/http/server.go:2092
                                                        /Users/barak/sdk/go1.23.5/src/runtime/asm_arm64.s:1223
                Error:          Not equal:
                                expected: local.POSIXPermissions{POSIXOwnership:local.POSIXOwnership{UID:501, GID:20}, Mode:0x41f8}
                                actual  : local.POSIXPermissions{POSIXOwnership:local.POSIXOwnership{UID:501, GID:20}, Mode:0x41e8}

                                Diff:
                                --- Expected
                                +++ Actual
                                @@ -5,3 +5,3 @@
                                  },
                                - Mode: (fs.FileMode) 16888
                                + Mode: (fs.FileMode) 16872
                                 }
                Test:           TestSyncManager_upload/download_folder_POSIX_perm_with_metadata
        sync_test.go:332:
                Error Trace:    /Users/barak/work/lakeFS/pkg/local/sync_test.go:332
                Error:          Received unexpected error:
                                upload folder2/ failed: Post "http://127.0.0.1:57153/api/v1/repositories/repo/branches/main/objects?path=folder2%2F": EOF
                Test:           TestSyncManager_upload/download_folder_POSIX_perm_with_metadata
@nopcoder nopcoder added the infrastructure build, deploy and release processes label Mar 6, 2025
@nopcoder
Copy link
Contributor Author

nopcoder commented Mar 6, 2025

From the numbers it expect permissions to be 770, but getting 750

@nopcoder nopcoder self-assigned this Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure build, deploy and release processes
Projects
None yet
Development

No branches or pull requests

1 participant