toolshed/docker-cli
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

integration/internal/container/ops: rm unused code

Browse Source 
Since container.Create() already initializes HostConfig
to be non-nil, there is no need for this code. Remove it.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
(cherry picked from commit 17022b3ad2d95863e5acd17ebaf943b7761623cc)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Upstream-commit: ed48a92a580b7ccc5c5837c7d802f787000146eb
Component: engine
This commit is contained in:
Kir Kolyshkin
2019-04-17 23:09:45 +02:00
committed by Sebastiaan van Stijn
parent 00e3442134
commit 9659d7a8ac
1 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
components/engine/integration/internal/container/ops.go
-6
View File
@@ -120,18 +120,12 @@ func WithIPv6(network, ip string) func(*TestContainerConfig) {
// WithLogDriver sets the log driver to use for the container
func WithLogDriver(driver string) func(*TestContainerConfig) {
return func(c *TestContainerConfig) {
if c.HostConfig == nil {
c.HostConfig = &containertypes.HostConfig{}
}
c.HostConfig.LogConfig.Type = driver
}
}
// WithAutoRemove sets the container to be removed on exit
func WithAutoRemove(c *TestContainerConfig) {
if c.HostConfig == nil {
c.HostConfig = &containertypes.HostConfig{}
}
c.HostConfig.AutoRemove = true
}