Pass dockerOpts by reference to setInfraContainerNetworkConfig

This commit is contained in:
Thomas Liu 2016-04-09 08:52:24 -07:00 committed by tyangliu
parent 660050631e
commit 019efef8f4

View File

@ -656,7 +656,7 @@ func (dm *DockerManager) runContainer(
// Set network configuration for infra-container // Set network configuration for infra-container
if container.Name == PodInfraContainerName { if container.Name == PodInfraContainerName {
setInfraContainerNetworkConfig(pod, netMode, opts, dockerOpts) setInfraContainerNetworkConfig(pod, netMode, opts, &dockerOpts)
} }
setEntrypointAndCommand(container, opts, dockerOpts) setEntrypointAndCommand(container, opts, dockerOpts)
@ -688,7 +688,7 @@ func (dm *DockerManager) runContainer(
// setInfraContainerNetworkConfig sets the network configuration for the infra-container. We only set network configuration for infra-container, all // setInfraContainerNetworkConfig sets the network configuration for the infra-container. We only set network configuration for infra-container, all
// the user containers will share the same network namespace with infra-container. // the user containers will share the same network namespace with infra-container.
func setInfraContainerNetworkConfig(pod *api.Pod, netMode string, opts *kubecontainer.RunContainerOptions, dockerOpts dockertypes.ContainerCreateConfig) { func setInfraContainerNetworkConfig(pod *api.Pod, netMode string, opts *kubecontainer.RunContainerOptions, dockerOpts *dockertypes.ContainerCreateConfig) {
exposedPorts, portBindings := makePortsAndBindings(opts.PortMappings) exposedPorts, portBindings := makePortsAndBindings(opts.PortMappings)
dockerOpts.Config.ExposedPorts = exposedPorts dockerOpts.Config.ExposedPorts = exposedPorts
dockerOpts.HostConfig.PortBindings = dockernat.PortMap(portBindings) dockerOpts.HostConfig.PortBindings = dockernat.PortMap(portBindings)