GitList

Browse code

builder: remove unused netnsRoot field in builder-next

Signed-off-by: Tibor Vass <tibor@docker.com>

Tibor Vass authored on 2018/08/31 06:35:35
Showing 5 changed files

builder/builder-next/builder.go index 6d3098a..97cdfc6 100644
builder/builder-next/controller.go index 48e74cb..9794f5f 100644
builder/builder-next/executor_unix.go index f72a27f..808889a 100644
builder/builder-next/executor_windows.go index f19bf18..e3b5aba 100644
cmd/dockerd/daemon.go index 7e8049c..a888bc8 100644

@@ -37,7 +37,6 @@ func init() {
                      type Opt struct {
                      	SessionManager    *session.Manager
                      	Root              string
                     -	NetnsRoot         string
                      	Dist              images.DistributionServices
                      	NetworkController libnetwork.NetworkController
+                     }

builder/builder-next/controller.go

History View file @ 8ab9e78

@@ -90,7 +90,7 @@ func newController(rt http.RoundTripper, opt Opt) (*control.Controller, error) {
 		return nil, err
 	}
 
-	exec, err := newExecutor(root, opt.NetnsRoot, opt.NetworkController)
+	exec, err := newExecutor(root, opt.NetworkController)
 	if err != nil {
 		return nil, err
 	}

builder/builder-next/executor_unix.go

History View file @ 8ab9e78

@@ -19,9 +19,9 @@ import (
                      const networkName = "bridge"
                     -func newExecutor(root, netnsRoot string, net libnetwork.NetworkController) (executor.Executor, error) {
                     +func newExecutor(root string, net libnetwork.NetworkController) (executor.Executor, error) {
                      	networkProviders := map[pb.NetMode]network.Provider{
                     -		pb.NetMode_UNSET: &bridgeProvider{NetworkController: net, netnsRoot: netnsRoot},
                     +		pb.NetMode_UNSET: &bridgeProvider{NetworkController: net},
                      		pb.NetMode_HOST:  network.NewHostProvider(),
                      		pb.NetMode_NONE:  network.NewNoneProvider(),
+                     	}
@@ -33,7 +33,6 @@ func newExecutor(root, netnsRoot string, net libnetwork.NetworkController) (exec
                      type bridgeProvider struct {
                      	libnetwork.NetworkController
                     -	netnsRoot string
+                     }
                      func (p *bridgeProvider) New() (network.Namespace, error) {

builder/builder-next/executor_windows.go

History View file @ 8ab9e78

@@ -10,7 +10,7 @@ import (
 	"github.com/moby/buildkit/executor"
 )
 
-func newExecutor(_, _ string, _ libnetwork.NetworkController) (executor.Executor, error) {
+func newExecutor(_ string, _ libnetwork.NetworkController) (executor.Executor, error) {
 	return &winExecutor{}, nil
 }
 

cmd/dockerd/daemon.go

History View file @ 8ab9e78

@@ -288,7 +288,6 @@ func newRouterOptions(config *config.Config, daemon *daemon.Daemon) (routerOptio
                      	bk, err := buildkit.New(buildkit.Opt{
                      		SessionManager:    sm,
                      		Root:              filepath.Join(config.Root, "buildkit"),
                     -		NetnsRoot:         filepath.Join(config.ExecRoot, "netns"),
                      		Dist:              daemon.DistributionServices(),
                      		NetworkController: daemon.NetworkController(),
                      	})

...	...	@@ -90,7 +90,7 @@ func newController(rt http.RoundTripper, opt Opt) (*control.Controller, error) {
90	90	return nil, err
91	91	}
92	92
93		- exec, err := newExecutor(root, opt.NetnsRoot, opt.NetworkController)
	93	+ exec, err := newExecutor(root, opt.NetworkController)
94	94	if err != nil {
95	95	return nil, err
96	96	}

...	...	@@ -10,7 +10,7 @@ import (
10	10	"github.com/moby/buildkit/executor"
11	11	)
12	12
13		-func newExecutor(_, _ string, _ libnetwork.NetworkController) (executor.Executor, error) {
	13	+func newExecutor(_ string, _ libnetwork.NetworkController) (executor.Executor, error) {
14	14	return &winExecutor{}, nil
15	15	}
16	16