Skip to content

[19.03 backport] plugin: fix a double RLock bug#41274

Merged
tiborvass merged 1 commit intomoby:19.03from
thaJeztah:19.03_backport_Double_RLock
Jul 28, 2020
Merged

[19.03 backport] plugin: fix a double RLock bug#41274
tiborvass merged 1 commit intomoby:19.03from
thaJeztah:19.03_backport_Double_RLock

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

backport of #40588

Signed-off-by: Ziheng Liu <lzhfromustc@gmail.com>
(cherry picked from commit 34837fe)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Copy Markdown
Member Author

@AkihiroSuda @cpuguy83 @tiborvass ptal

@thaJeztah
Copy link
Copy Markdown
Member Author

Failure should now be fixed by #41271

== RUN   TestDockerSuite/TestRunNonRootUserResolvName
    --- FAIL: TestDockerSuite/TestRunNonRootUserResolvName (0.59s)
        cli.go:35: assertion failed: 
            Command:  /usr/local/cli/docker run --name=testperm --user=nobody busybox nslookup apt.dockerproject.org
            ExitCode: 1
            Error:    exit status 1
            Stdout:   Server:    10.100.0.2
            Address 1: 10.100.0.2 ip-10-100-0-2.ec2.internal
            
            
            Stderr:   nslookup: can't resolve 'apt.dockerproject.org'
            
            
            Failures:
            ExitCode was 1 expected 0
            Expected no error

@tiborvass tiborvass merged commit 22c458b into moby:19.03 Jul 28, 2020
@thaJeztah thaJeztah deleted the 19.03_backport_Double_RLock branch July 28, 2020 12:29
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.

4 participants