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

[20.10 backport] Update some tests for supplementary group permissions #44120

Merged
merged 1 commit into from Sep 8, 2022

Commits on Sep 8, 2022

  1. Update some tests for supplementary group permissions

    Update tests checking for groups to adjust for new policy updated in
    de7af81, which caused those tests
    to fail:
    
        === FAIL: amd64.integration-cli TestDockerSwarmSuite/TestSwarmServiceWithGroup (1.94s)
        docker_cli_swarm_test.go:311: assertion failed: uid=0(root) gid=0(root) groups=0(root),10(wheel),29(audio),50(staff),777 (string) != uid=0(root) gid=0(root) groups=10(wheel),29(audio),50(staff),777
    (string)
        --- FAIL: TestDockerSwarmSuite/TestSwarmServiceWithGroup (1.94s)
    
        === FAIL: amd64.integration-cli TestDockerCLIRunSuite/TestRunGroupAdd (0.41s)
        docker_cli_run_test.go:1091: expected output uid=0(root) gid=0(root) groups=10(wheel),29(audio),50(staff),777 received uid=0(root) gid=0(root) groups=0(root),10(wheel),29(audio),50(staff),777
        --- FAIL: TestDockerCLIRunSuite/TestRunGroupAdd (0.41s)
    
        === FAIL: amd64.integration-cli TestDockerCLIRunSuite/TestRunUserByIDZero (0.41s)
        docker_cli_run_test.go:790: expected daemon user got uid=0(root) gid=0(root) groups=0(root),10(wheel)
        --- FAIL: TestDockerCLIRunSuite/TestRunUserByIDZero (0.41s)
    
        === FAIL: amd64.integration-cli TestDockerCLIRunSuite (195.70s)
    
    Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
    (cherry picked from commit c7e77db)
    Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
    thaJeztah committed Sep 8, 2022
    Copy the full SHA
    fcd4df9 View commit details
    Browse the repository at this point in the history