Factor an API out into a module
authorMichael Bridgen <mikeb@squaremobius.net>
Fri, 7 Aug 2015 15:27:52 +0000 (16:27 +0100)
committerMichael Bridgen <mikeb@squaremobius.net>
Wed, 16 Sep 2015 09:14:39 +0000 (10:14 +0100)
This takes some of the machinery from CNI and from the rkt networking
code, and turns it into a library that can be linked into go apps.

Included is an example command-line application that uses the library,
called `cnitool`.

Other headline changes:

 * Plugin exec'ing is factored out

The motivation here is to factor out the protocol for invoking
plugins. To that end, a generalisation of the code from api.go and
pkg/plugin/ipam.go goes into pkg/invoke/exec.go.

 * Move argument-handling and conf-loading into public API

The fact that the arguments get turned into an environment for the
plugin is incidental to the API; so, provide a way of supplying them
as a struct or saying "just use the same arguments as I got" (the
latter is for IPAM plugins).

28 files changed:
README.md
build
cnitool/cni.go [new file with mode: 0644]
libcni/api.go [new file with mode: 0644]
libcni/conf.go [new file with mode: 0644]
pkg/invoke/args.go [new file with mode: 0644]
pkg/invoke/exec.go [new file with mode: 0644]
pkg/invoke/find.go [new file with mode: 0644]
pkg/ip/cidr.go
pkg/ipam/ipam.go [new file with mode: 0644]
pkg/plugin/ipam.go [deleted file]
pkg/skel/skel.go
pkg/types/args.go [moved from pkg/plugin/args.go with 56% similarity]
pkg/types/types.go [moved from pkg/plugin/types.go with 73% similarity]
plugins/ipam/dhcp/daemon.go
plugins/ipam/dhcp/lease.go
plugins/ipam/dhcp/main.go
plugins/ipam/dhcp/options.go
plugins/ipam/dhcp/options_test.go
plugins/ipam/host-local/allocator.go
plugins/ipam/host-local/config.go
plugins/ipam/host-local/main.go
plugins/main/bridge/bridge.go
plugins/main/ipvlan/ipvlan.go
plugins/main/macvlan/macvlan.go
plugins/main/ptp/ptp.go
plugins/meta/flannel/flannel.go
test

index 7b5aad7..5c9b641 100644 (file)
--- a/README.md
+++ b/README.md
@@ -3,7 +3,8 @@
 ## What is CNI?
 
 CNI, the _Container Network Interface_, is a proposed standard for configuring network interfaces for Linux application containers.
-The standard consists of a simple specification for how executable plugins can be used to configure network namespaces.
+The standard consists of a simple specification for how executable plugins can be used to configure network namespaces; this repository also contains a go library implementing that specification.
+
 The specification itself is contained in [SPEC.md](SPEC.md)
 
 ## Why develop CNI?
@@ -47,6 +48,8 @@ $ cat >/etc/cni/net.d/10-mynet.conf <<EOF
 EOF
 ```
 
+The directory `/etc/cni/net.d` is the default location in which the scripts will look for net configurations.
+
 Next, build the plugins:
 
 ```
@@ -78,6 +81,8 @@ lo        Link encap:Local Loopback
           RX bytes:0 (0.0 B)  TX bytes:0 (0.0 B)
 ```
 
+The environment variable `CNI_PATH` tells the scripts and library where to look for plugin executables.
+
 ## Running a Docker container with network namespace set up by CNI plugins
 
 Use instructions in the previous section to define a netconf and build the plugins.
diff --git a/build b/build
index a7173b0..99f42fb 100755 (executable)
--- a/build
+++ b/build
@@ -11,6 +11,12 @@ fi
 export GOBIN=${PWD}/bin
 export GOPATH=${PWD}/gopath:$(pwd)/Godeps/_workspace
 
+echo "Building API"
+go build ${REPO_PATH}/libcni
+
+echo "Building reference CLI"
+go install ${REPO_PATH}/cnitool
+
 echo "Building plugins"
 
 PLUGINS="plugins/meta/* plugins/main/* plugins/ipam/*"
diff --git a/cnitool/cni.go b/cnitool/cni.go
new file mode 100644 (file)
index 0000000..62ac675
--- /dev/null
@@ -0,0 +1,87 @@
+// Copyright 2015 CoreOS, Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package main
+
+import (
+       "fmt"
+       "os"
+       "path/filepath"
+       "strings"
+
+       "github.com/appc/cni/libcni"
+)
+
+const (
+       EnvCNIPath = "CNI_PATH"
+       EnvNetDir  = "NETCONFPATH"
+
+       DefaultNetDir = "/etc/cni/net.d"
+
+       CmdAdd = "add"
+       CmdDel = "del"
+)
+
+func main() {
+       if len(os.Args) < 3 {
+               usage()
+               return
+       }
+
+       netdir := os.Getenv(EnvNetDir)
+       if netdir == "" {
+               netdir = DefaultNetDir
+       }
+       netconf, err := libcni.LoadConf(netdir, os.Args[2])
+       if err != nil {
+               exit(err)
+       }
+
+       netns := os.Args[3]
+
+       cninet := &libcni.CNIConfig{
+               Path: strings.Split(os.Getenv(EnvCNIPath), ":"),
+       }
+
+       rt := &libcni.RuntimeConf{
+               ContainerID: "cni",
+               NetNS:       netns,
+               IfName:      "eth0",
+       }
+
+       switch os.Args[1] {
+       case CmdAdd:
+               _, err := cninet.AddNetwork(netconf, rt)
+               exit(err)
+       case CmdDel:
+               exit(cninet.DelNetwork(netconf, rt))
+       }
+}
+
+func usage() {
+       exe := filepath.Base(os.Args[0])
+
+       fmt.Fprintf(os.Stderr, "%s: Add or remove network interfaces from a network namespace\n", exe)
+       fmt.Fprintf(os.Stderr, "  %s %s <net> <netns>\n", exe, CmdAdd)
+       fmt.Fprintf(os.Stderr, "  %s %s <net> <netns>\n", exe, CmdDel)
+       os.Exit(1)
+}
+
+func exit(err error) {
+       if err != nil {
+               fmt.Fprintf(os.Stderr, "%s\n", err)
+               os.Exit(1)
+       }
+       os.Exit(0)
+}
diff --git a/libcni/api.go b/libcni/api.go
new file mode 100644 (file)
index 0000000..feaed8d
--- /dev/null
@@ -0,0 +1,68 @@
+// Copyright 2015 CoreOS, Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package libcni
+
+import (
+       "strings"
+
+       "github.com/appc/cni/pkg/invoke"
+       "github.com/appc/cni/pkg/types"
+)
+
+type RuntimeConf struct {
+       ContainerID string
+       NetNS       string
+       IfName      string
+       Args        [][2]string
+}
+
+type NetworkConfig struct {
+       Network *types.NetConf
+       Bytes   []byte
+}
+
+type CNI interface {
+       AddNetwork(net *NetworkConfig, rt *RuntimeConf) (*types.Result, error)
+       DelNetwork(net *NetworkConfig, rt *RuntimeConf) error
+}
+
+type CNIConfig struct {
+       Path []string
+}
+
+func (c *CNIConfig) AddNetwork(net *NetworkConfig, rt *RuntimeConf) (*types.Result, error) {
+       return c.execPlugin("ADD", net, rt)
+}
+
+func (c *CNIConfig) DelNetwork(net *NetworkConfig, rt *RuntimeConf) error {
+       _, err := c.execPlugin("DEL", net, rt)
+       return err
+}
+
+// =====
+
+func (c *CNIConfig) execPlugin(action string, conf *NetworkConfig, rt *RuntimeConf) (*types.Result, error) {
+       pluginPath := invoke.FindInPath(conf.Network.Type, c.Path)
+
+       args := &invoke.Args{
+               Command:     action,
+               ContainerID: rt.ContainerID,
+               NetNS:       rt.NetNS,
+               PluginArgs:  rt.Args,
+               IfName:      rt.IfName,
+               Path:        strings.Join(c.Path, ":"),
+       }
+       return invoke.ExecPlugin(pluginPath, conf.Bytes, args)
+}
diff --git a/libcni/conf.go b/libcni/conf.go
new file mode 100644 (file)
index 0000000..47babeb
--- /dev/null
@@ -0,0 +1,85 @@
+// Copyright 2015 CoreOS, Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package libcni
+
+import (
+       "encoding/json"
+       "fmt"
+       "io/ioutil"
+       "os"
+       "path/filepath"
+       "sort"
+)
+
+func ConfFromBytes(bytes []byte) (*NetworkConfig, error) {
+       conf := &NetworkConfig{Bytes: bytes}
+       if err := json.Unmarshal(bytes, &conf.Network); err != nil {
+               return nil, fmt.Errorf("error parsing configuration: %s", err)
+       }
+       return conf, nil
+}
+
+func ConfFromFile(filename string) (*NetworkConfig, error) {
+       bytes, err := ioutil.ReadFile(filename)
+       if err != nil {
+               return nil, fmt.Errorf("error reading %s: %s", filename, err)
+       }
+       return ConfFromBytes(bytes)
+}
+
+func ConfFiles(dir string) ([]string, error) {
+       // In part, adapted from rkt/networking/podenv.go#listFiles
+       files, err := ioutil.ReadDir(dir)
+       switch {
+       case err == nil: // break
+       case os.IsNotExist(err):
+               return nil, nil
+       default:
+               return nil, err
+       }
+
+       confFiles := []string{}
+       for _, f := range files {
+               if f.IsDir() {
+                       continue
+               }
+               if filepath.Ext(f.Name()) == ".conf" {
+                       confFiles = append(confFiles, filepath.Join(dir, f.Name()))
+               }
+       }
+       return confFiles, nil
+}
+
+func LoadConf(dir, name string) (*NetworkConfig, error) {
+       files, err := ConfFiles(dir)
+       switch {
+       case err != nil:
+               return nil, err
+       case len(files) == 0:
+               return nil, fmt.Errorf("no net configurations found")
+       }
+       sort.Strings(files)
+
+       for _, confFile := range files {
+               conf, err := ConfFromFile(confFile)
+               if err != nil {
+                       return nil, err
+               }
+               if conf.Network.Name == name {
+                       return conf, nil
+               }
+       }
+       return nil, fmt.Errorf(`no net configuration with name "%s" in %s`, name, dir)
+}
diff --git a/pkg/invoke/args.go b/pkg/invoke/args.go
new file mode 100644 (file)
index 0000000..6f0a813
--- /dev/null
@@ -0,0 +1,76 @@
+// Copyright 2015 CoreOS, Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package invoke
+
+import (
+       "os"
+       "strings"
+)
+
+type CNIArgs interface {
+       // For use with os/exec; i.e., return nil to inherit the
+       // environment from this process
+       AsEnv() []string
+}
+
+type inherited struct{}
+
+var inheritArgsFromEnv inherited
+
+func (_ *inherited) AsEnv() []string {
+       return nil
+}
+
+func ArgsFromEnv() CNIArgs {
+       return &inheritArgsFromEnv
+}
+
+type Args struct {
+       Command       string
+       ContainerID   string
+       NetNS         string
+       PluginArgs    [][2]string
+       PluginArgsStr string
+       IfName        string
+       Path          string
+}
+
+func (args *Args) AsEnv() []string {
+       env := os.Environ()
+       pluginArgsStr := args.PluginArgsStr
+       if pluginArgsStr == "" {
+               pluginArgsStr = stringify(args.PluginArgs)
+       }
+
+       env = append(env,
+               "CNI_COMMAND="+args.Command,
+               "CNI_CONTAINERID="+args.ContainerID,
+               "CNI_NETNS="+args.NetNS,
+               "CNI_ARGS="+pluginArgsStr,
+               "CNI_IFNAME="+args.IfName,
+               "CNI_PATH="+args.Path)
+       return env
+}
+
+// taken from rkt/networking/net_plugin.go
+func stringify(pluginArgs [][2]string) string {
+       entries := make([]string, len(pluginArgs))
+
+       for i, kv := range pluginArgs {
+               entries[i] = strings.Join(kv[:], "=")
+       }
+
+       return strings.Join(entries, ";")
+}
diff --git a/pkg/invoke/exec.go b/pkg/invoke/exec.go
new file mode 100644 (file)
index 0000000..d7c5b7a
--- /dev/null
@@ -0,0 +1,66 @@
+// Copyright 2015 CoreOS, Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package invoke
+
+import (
+       "bytes"
+       "encoding/json"
+       "fmt"
+       "os"
+       "os/exec"
+       "path/filepath"
+
+       "github.com/appc/cni/pkg/types"
+)
+
+func pluginErr(err error, output []byte) error {
+       if _, ok := err.(*exec.ExitError); ok {
+               emsg := types.Error{}
+               if perr := json.Unmarshal(output, &emsg); perr != nil {
+                       return fmt.Errorf("netplugin failed but error parsing its diagnostic message %q: %v", string(output), perr)
+               }
+               details := ""
+               if emsg.Details != "" {
+                       details = fmt.Sprintf("; %v", emsg.Details)
+               }
+               return fmt.Errorf("%v%v", emsg.Msg, details)
+       }
+
+       return err
+}
+
+func ExecPlugin(pluginPath string, netconf []byte, args CNIArgs) (*types.Result, error) {
+       if pluginPath == "" {
+               return nil, fmt.Errorf("could not find %q plugin", filepath.Base(pluginPath))
+       }
+
+       stdout := &bytes.Buffer{}
+
+       c := exec.Cmd{
+               Env:    args.AsEnv(),
+               Path:   pluginPath,
+               Args:   []string{pluginPath},
+               Stdin:  bytes.NewBuffer(netconf),
+               Stdout: stdout,
+               Stderr: os.Stderr,
+       }
+       if err := c.Run(); err != nil {
+               return nil, pluginErr(err, stdout.Bytes())
+       }
+
+       res := &types.Result{}
+       err := json.Unmarshal(stdout.Bytes(), res)
+       return res, err
+}
diff --git a/pkg/invoke/find.go b/pkg/invoke/find.go
new file mode 100644 (file)
index 0000000..dfad12b
--- /dev/null
@@ -0,0 +1,37 @@
+// Copyright 2015 CoreOS, Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package invoke
+
+import (
+       "os"
+       "path/filepath"
+       "strings"
+)
+
+func FindInPath(plugin string, path []string) string {
+       for _, p := range path {
+               fullname := filepath.Join(p, plugin)
+               if fi, err := os.Stat(fullname); err == nil && fi.Mode().IsRegular() {
+                       return fullname
+               }
+       }
+       return ""
+}
+
+// Find returns the full path of the plugin by searching in CNI_PATH
+func Find(plugin string) string {
+       paths := strings.Split(os.Getenv("CNI_PATH"), ":")
+       return FindInPath(plugin, paths)
+}
index c963398..723a1f7 100644 (file)
 package ip
 
 import (
-       "encoding/json"
        "math/big"
        "net"
 )
 
-// ParseCIDR takes a string like "10.2.3.1/24" and
-// return IPNet with "10.2.3.1" and /24 mask
-func ParseCIDR(s string) (*net.IPNet, error) {
-       ip, ipn, err := net.ParseCIDR(s)
-       if err != nil {
-               return nil, err
-       }
-
-       ipn.IP = ip
-       return ipn, nil
-}
-
 // NextIP returns IP incremented by 1
 func NextIP(ip net.IP) net.IP {
        i := ipToInt(ip)
@@ -62,25 +49,3 @@ func Network(ipn *net.IPNet) *net.IPNet {
                Mask: ipn.Mask,
        }
 }
-
-// like net.IPNet but adds JSON marshalling and unmarshalling
-type IPNet net.IPNet
-
-func (n IPNet) MarshalJSON() ([]byte, error) {
-       return json.Marshal((*net.IPNet)(&n).String())
-}
-
-func (n *IPNet) UnmarshalJSON(data []byte) error {
-       var s string
-       if err := json.Unmarshal(data, &s); err != nil {
-               return err
-       }
-
-       tmp, err := ParseCIDR(s)
-       if err != nil {
-               return err
-       }
-
-       *n = IPNet(*tmp)
-       return nil
-}
diff --git a/pkg/ipam/ipam.go b/pkg/ipam/ipam.go
new file mode 100644 (file)
index 0000000..a76299d
--- /dev/null
@@ -0,0 +1,75 @@
+// Copyright 2015 CoreOS, Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package ipam
+
+import (
+       "fmt"
+       "os"
+
+       "github.com/appc/cni/pkg/invoke"
+       "github.com/appc/cni/pkg/ip"
+       "github.com/appc/cni/pkg/types"
+
+       "github.com/vishvananda/netlink"
+)
+
+func ExecAdd(plugin string, netconf []byte) (*types.Result, error) {
+       if os.Getenv("CNI_COMMAND") != "ADD" {
+               return nil, fmt.Errorf("CNI_COMMAND is not ADD")
+       }
+       return invoke.ExecPlugin(invoke.Find(plugin), netconf, invoke.ArgsFromEnv())
+}
+
+func ExecDel(plugin string, netconf []byte) error {
+       if os.Getenv("CNI_COMMAND") != "DEL" {
+               return fmt.Errorf("CNI_COMMAND is not DEL")
+       }
+       _, err := invoke.ExecPlugin(invoke.Find(plugin), netconf, invoke.ArgsFromEnv())
+       return err
+}
+
+// ConfigureIface takes the result of IPAM plugin and
+// applies to the ifName interface
+func ConfigureIface(ifName string, res *types.Result) error {
+       link, err := netlink.LinkByName(ifName)
+       if err != nil {
+               return fmt.Errorf("failed to lookup %q: %v", ifName, err)
+       }
+
+       if err := netlink.LinkSetUp(link); err != nil {
+               return fmt.Errorf("failed to set %q UP: %v", ifName, err)
+       }
+
+       // TODO(eyakubovich): IPv6
+       addr := &netlink.Addr{IPNet: &res.IP4.IP, Label: ""}
+       if err = netlink.AddrAdd(link, addr); err != nil {
+               return fmt.Errorf("failed to add IP addr to %q: %v", ifName, err)
+       }
+
+       for _, r := range res.IP4.Routes {
+               gw := r.GW
+               if gw == nil {
+                       gw = res.IP4.Gateway
+               }
+               if err = ip.AddRoute(&r.Dst, gw, link); err != nil {
+                       // we skip over duplicate routes as we assume the first one wins
+                       if !os.IsExist(err) {
+                               return fmt.Errorf("failed to add route '%v via %v dev %v': %v", r.Dst, gw, ifName, err)
+                       }
+               }
+       }
+
+       return nil
+}
diff --git a/pkg/plugin/ipam.go b/pkg/plugin/ipam.go
deleted file mode 100644 (file)
index f304301..0000000
+++ /dev/null
@@ -1,151 +0,0 @@
-// Copyright 2015 CoreOS, Inc.
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package plugin
-
-import (
-       "bytes"
-       "encoding/json"
-       "fmt"
-       "os"
-       "os/exec"
-       "path/filepath"
-       "strings"
-
-       "github.com/appc/cni/pkg/ip"
-       "github.com/vishvananda/netlink"
-)
-
-// Find returns the full path of the plugin by searching in CNI_PATH
-func Find(plugin string) string {
-       paths := strings.Split(os.Getenv("CNI_PATH"), ":")
-
-       for _, p := range paths {
-               fullname := filepath.Join(p, plugin)
-               if fi, err := os.Stat(fullname); err == nil && fi.Mode().IsRegular() {
-                       return fullname
-               }
-       }
-
-       return ""
-}
-
-func pluginErr(err error, output []byte) error {
-       if _, ok := err.(*exec.ExitError); ok {
-               emsg := Error{}
-               if perr := json.Unmarshal(output, &emsg); perr != nil {
-                       return fmt.Errorf("netplugin failed but error parsing its diagnostic message %q: %v", string(output), perr)
-               }
-               details := ""
-               if emsg.Details != "" {
-                       details = fmt.Sprintf("; %v", emsg.Details)
-               }
-               return fmt.Errorf("%v%v", emsg.Msg, details)
-       }
-
-       return err
-}
-
-// ExecAdd executes IPAM plugin, assuming CNI_COMMAND == ADD.
-// Parses and returns resulting IPConfig
-func ExecAdd(plugin string, netconf []byte) (*Result, error) {
-       if os.Getenv("CNI_COMMAND") != "ADD" {
-               return nil, fmt.Errorf("CNI_COMMAND is not ADD")
-       }
-       if plugin == "" {
-               return nil, fmt.Errorf(`name of IPAM plugin is missing. Please specify a "type" field in the "ipam" section`)
-       }
-
-       pluginPath := Find(plugin)
-       if pluginPath == "" {
-               return nil, fmt.Errorf("could not find %q IPAM plugin", plugin)
-       }
-
-       stdout := &bytes.Buffer{}
-
-       c := exec.Cmd{
-               Path:   pluginPath,
-               Args:   []string{pluginPath},
-               Stdin:  bytes.NewBuffer(netconf),
-               Stdout: stdout,
-               Stderr: os.Stderr,
-       }
-       if err := c.Run(); err != nil {
-               return nil, pluginErr(err, stdout.Bytes())
-       }
-
-       res := &Result{}
-       err := json.Unmarshal(stdout.Bytes(), res)
-       return res, err
-}
-
-// ExecDel executes IPAM plugin, assuming CNI_COMMAND == DEL.
-func ExecDel(plugin string, netconf []byte) error {
-       if os.Getenv("CNI_COMMAND") != "DEL" {
-               return fmt.Errorf("CNI_COMMAND is not DEL")
-       }
-
-       pluginPath := Find(plugin)
-       if pluginPath == "" {
-               return fmt.Errorf("could not find %q plugin", plugin)
-       }
-
-       stdout := &bytes.Buffer{}
-
-       c := exec.Cmd{
-               Path:   pluginPath,
-               Args:   []string{pluginPath},
-               Stdin:  bytes.NewBuffer(netconf),
-               Stdout: stdout,
-               Stderr: os.Stderr,
-       }
-       if err := c.Run(); err != nil {
-               return pluginErr(err, stdout.Bytes())
-       }
-       return nil
-}
-
-// ConfigureIface takes the result of IPAM plugin and
-// applies to the ifName interface
-func ConfigureIface(ifName string, res *Result) error {
-       link, err := netlink.LinkByName(ifName)
-       if err != nil {
-               return fmt.Errorf("failed to lookup %q: %v", ifName, err)
-       }
-
-       if err := netlink.LinkSetUp(link); err != nil {
-               return fmt.Errorf("failed to set %q UP: %v", ifName, err)
-       }
-
-       // TODO(eyakubovich): IPv6
-       addr := &netlink.Addr{IPNet: &res.IP4.IP, Label: ""}
-       if err = netlink.AddrAdd(link, addr); err != nil {
-               return fmt.Errorf("failed to add IP addr to %q: %v", ifName, err)
-       }
-
-       for _, r := range res.IP4.Routes {
-               gw := r.GW
-               if gw == nil {
-                       gw = res.IP4.Gateway
-               }
-               if err = ip.AddRoute(&r.Dst, gw, link); err != nil {
-                       // we skip over duplicate routes as we assume the first one wins
-                       if !os.IsExist(err) {
-                               return fmt.Errorf("failed to add route '%v via %v dev %v': %v", r.Dst, gw, ifName, err)
-                       }
-               }
-       }
-
-       return nil
-}
index bf79b91..d6204dd 100644 (file)
@@ -22,7 +22,7 @@ import (
        "log"
        "os"
 
-       "github.com/appc/cni/pkg/plugin"
+       "github.com/appc/cni/pkg/types"
 )
 
 // CmdArgs captures all the arguments passed in to the plugin
@@ -93,7 +93,7 @@ func PluginMain(cmdAdd, cmdDel func(_ *CmdArgs) error) {
        }
 
        if err != nil {
-               if e, ok := err.(*plugin.Error); ok {
+               if e, ok := err.(*types.Error); ok {
                        // don't wrap Error in Error
                        dieErr(e)
                }
@@ -102,14 +102,14 @@ func PluginMain(cmdAdd, cmdDel func(_ *CmdArgs) error) {
 }
 
 func dieMsg(f string, args ...interface{}) {
-       e := &plugin.Error{
+       e := &types.Error{
                Code: 100,
                Msg:  fmt.Sprintf(f, args...),
        }
        dieErr(e)
 }
 
-func dieErr(e *plugin.Error) {
+func dieErr(e *types.Error) {
        if err := e.Print(); err != nil {
                log.Print("Error writing error JSON to stdout: ", err)
        }
similarity index 56%
rename from pkg/plugin/args.go
rename to pkg/types/args.go
index 274ec66..6816243 100644 (file)
@@ -1,4 +1,18 @@
-package plugin
+// Copyright 2015 CoreOS, Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package types
 
 import (
        "encoding"
similarity index 73%
rename from pkg/plugin/types.go
rename to pkg/types/types.go
index d5952dd..21ba32d 100644 (file)
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package plugin
+package types
 
 import (
        "encoding/json"
        "net"
        "os"
-
-       "github.com/appc/cni/pkg/ip"
 )
 
+// like net.IPNet but adds JSON marshalling and unmarshalling
+type IPNet net.IPNet
+
+// ParseCIDR takes a string like "10.2.3.1/24" and
+// return IPNet with "10.2.3.1" and /24 mask
+func ParseCIDR(s string) (*net.IPNet, error) {
+       ip, ipn, err := net.ParseCIDR(s)
+       if err != nil {
+               return nil, err
+       }
+
+       ipn.IP = ip
+       return ipn, nil
+}
+
+func (n IPNet) MarshalJSON() ([]byte, error) {
+       return json.Marshal((*net.IPNet)(&n).String())
+}
+
+func (n *IPNet) UnmarshalJSON(data []byte) error {
+       var s string
+       if err := json.Unmarshal(data, &s); err != nil {
+               return err
+       }
+
+       tmp, err := ParseCIDR(s)
+       if err != nil {
+               return err
+       }
+
+       *n = IPNet(*tmp)
+       return nil
+}
+
 // NetConf describes a network.
 type NetConf struct {
        Name string `json:"name,omitempty"`
@@ -68,23 +100,23 @@ func (e *Error) Print() error {
 }
 
 // net.IPNet is not JSON (un)marshallable so this duality is needed
-// for our custom ip.IPNet type
+// for our custom IPNet type
 
 // JSON (un)marshallable types
 type ipConfig struct {
-       IP      ip.IPNet `json:"ip"`
-       Gateway net.IP   `json:"gateway,omitempty"`
-       Routes  []Route  `json:"routes,omitempty"`
+       IP      IPNet   `json:"ip"`
+       Gateway net.IP  `json:"gateway,omitempty"`
+       Routes  []Route `json:"routes,omitempty"`
 }
 
 type route struct {
-       Dst ip.IPNet `json:"dst"`
-       GW  net.IP   `json:"gw,omitempty"`
+       Dst IPNet  `json:"dst"`
+       GW  net.IP `json:"gw,omitempty"`
 }
 
 func (c *IPConfig) MarshalJSON() ([]byte, error) {
        ipc := ipConfig{
-               IP:      ip.IPNet(c.IP),
+               IP:      IPNet(c.IP),
                Gateway: c.Gateway,
                Routes:  c.Routes,
        }
@@ -117,7 +149,7 @@ func (r *Route) UnmarshalJSON(data []byte) error {
 
 func (r *Route) MarshalJSON() ([]byte, error) {
        rt := route{
-               Dst: ip.IPNet(r.Dst),
+               Dst: IPNet(r.Dst),
                GW:  r.GW,
        }
 
index 7b5ecfb..1fc91f2 100644 (file)
@@ -27,8 +27,8 @@ import (
        "runtime"
        "sync"
 
-       "github.com/appc/cni/pkg/plugin"
        "github.com/appc/cni/pkg/skel"
+       "github.com/appc/cni/pkg/types"
        "github.com/coreos/go-systemd/activation"
 )
 
@@ -50,8 +50,8 @@ func newDHCP() *DHCP {
 
 // Allocate acquires an IP from a DHCP server for a specified container.
 // The acquired lease will be maintained until Release() is called.
-func (d *DHCP) Allocate(args *skel.CmdArgs, result *plugin.Result) error {
-       conf := plugin.NetConf{}
+func (d *DHCP) Allocate(args *skel.CmdArgs, result *types.Result) error {
+       conf := types.NetConf{}
        if err := json.Unmarshal(args.StdinData, &conf); err != nil {
                return fmt.Errorf("error parsing netconf: %v", err)
        }
@@ -70,7 +70,7 @@ func (d *DHCP) Allocate(args *skel.CmdArgs, result *plugin.Result) error {
 
        d.setLease(args.ContainerID, conf.Name, l)
 
-       result.IP4 = &plugin.IPConfig{
+       result.IP4 = &types.IPConfig{
                IP:      *ipn,
                Gateway: l.Gateway(),
                Routes:  l.Routes(),
@@ -82,7 +82,7 @@ func (d *DHCP) Allocate(args *skel.CmdArgs, result *plugin.Result) error {
 // Release stops maintenance of the lease acquired in Allocate()
 // and sends a release msg to the DHCP server.
 func (d *DHCP) Release(args *skel.CmdArgs, reply *struct{}) error {
-       conf := plugin.NetConf{}
+       conf := types.NetConf{}
        if err := json.Unmarshal(args.StdinData, &conf); err != nil {
                return fmt.Errorf("error parsing netconf: %v", err)
        }
index 0c8aa7a..42a2d86 100644 (file)
@@ -28,7 +28,7 @@ import (
        "github.com/vishvananda/netlink"
 
        "github.com/appc/cni/pkg/ns"
-       "github.com/appc/cni/pkg/plugin"
+       "github.com/appc/cni/pkg/types"
 )
 
 // RFC 2131 suggests using exponential backoff, starting with 4sec
@@ -285,7 +285,7 @@ func (l *DHCPLease) Gateway() net.IP {
        return parseRouter(l.opts)
 }
 
-func (l *DHCPLease) Routes() []plugin.Route {
+func (l *DHCPLease) Routes() []types.Route {
        routes := parseRoutes(l.opts)
        return append(routes, parseCIDRRoutes(l.opts)...)
 }
index ca80271..c35bdf4 100644 (file)
@@ -20,8 +20,8 @@ import (
        "os"
        "path/filepath"
 
-       "github.com/appc/cni/pkg/plugin"
        "github.com/appc/cni/pkg/skel"
+       "github.com/appc/cni/pkg/types"
 )
 
 const socketPath = "/run/cni/dhcp.sock"
@@ -35,7 +35,7 @@ func main() {
 }
 
 func cmdAdd(args *skel.CmdArgs) error {
-       result := plugin.Result{}
+       result := types.Result{}
        if err := rpcCall("DHCP.Allocate", args, &result); err != nil {
                return err
        }
index 37cfc61..9189e5b 100644 (file)
@@ -20,7 +20,7 @@ import (
        "net"
        "time"
 
-       "github.com/appc/cni/pkg/plugin"
+       "github.com/appc/cni/pkg/types"
        "github.com/d2g/dhcp4"
 )
 
@@ -40,17 +40,17 @@ func classfulSubnet(sn net.IP) net.IPNet {
        }
 }
 
-func parseRoutes(opts dhcp4.Options) []plugin.Route {
+func parseRoutes(opts dhcp4.Options) []types.Route {
        // StaticRoutes format: pairs of:
        // Dest = 4 bytes; Classful IP subnet
        // Router = 4 bytes; IP address of router
 
-       routes := []plugin.Route{}
+       routes := []types.Route{}
        if opt, ok := opts[dhcp4.OptionStaticRoute]; ok {
                for len(opt) >= 8 {
                        sn := opt[0:4]
                        r := opt[4:8]
-                       rt := plugin.Route{
+                       rt := types.Route{
                                Dst: classfulSubnet(sn),
                                GW:  r,
                        }
@@ -62,10 +62,10 @@ func parseRoutes(opts dhcp4.Options) []plugin.Route {
        return routes
 }
 
-func parseCIDRRoutes(opts dhcp4.Options) []plugin.Route {
+func parseCIDRRoutes(opts dhcp4.Options) []types.Route {
        // See RFC4332 for format (http://tools.ietf.org/html/rfc3442)
 
-       routes := []plugin.Route{}
+       routes := []types.Route{}
        if opt, ok := opts[dhcp4.OptionClasslessRouteFormat]; ok {
                for len(opt) >= 5 {
                        width := int(opt[0])
@@ -89,7 +89,7 @@ func parseCIDRRoutes(opts dhcp4.Options) []plugin.Route {
 
                        gw := net.IP(opt[octets+1 : octets+5])
 
-                       rt := plugin.Route{
+                       rt := types.Route{
                                Dst: net.IPNet{
                                        IP:   net.IP(sn),
                                        Mask: net.CIDRMask(width, 32),
index 52b2730..5aa129d 100644 (file)
@@ -18,20 +18,20 @@ import (
        "net"
        "testing"
 
-       "github.com/appc/cni/pkg/plugin"
+       "github.com/appc/cni/pkg/types"
        "github.com/d2g/dhcp4"
 )
 
-func validateRoutes(t *testing.T, routes []plugin.Route) {
-       expected := []plugin.Route{
-               plugin.Route{
+func validateRoutes(t *testing.T, routes []types.Route) {
+       expected := []types.Route{
+               types.Route{
                        Dst: net.IPNet{
                                IP:   net.IPv4(10, 0, 0, 0),
                                Mask: net.CIDRMask(8, 32),
                        },
                        GW: net.IPv4(10, 1, 2, 3),
                },
-               plugin.Route{
+               types.Route{
                        Dst: net.IPNet{
                                IP:   net.IPv4(192, 168, 1, 0),
                                Mask: net.CIDRMask(24, 32),
index 8d20344..56c7947 100644 (file)
@@ -19,7 +19,7 @@ import (
        "net"
 
        "github.com/appc/cni/pkg/ip"
-       "github.com/appc/cni/pkg/plugin"
+       "github.com/appc/cni/pkg/types"
        "github.com/appc/cni/plugins/ipam/host-local/backend"
 )
 
@@ -69,7 +69,7 @@ func validateRangeIP(ip net.IP, ipnet *net.IPNet) error {
 }
 
 // Returns newly allocated IP along with its config
-func (a *IPAllocator) Get(id string) (*plugin.IPConfig, error) {
+func (a *IPAllocator) Get(id string) (*types.IPConfig, error) {
        a.store.Lock()
        defer a.store.Unlock()
 
@@ -103,7 +103,7 @@ func (a *IPAllocator) Get(id string) (*plugin.IPConfig, error) {
                }
 
                if reserved {
-                       return &plugin.IPConfig{
+                       return &types.IPConfig{
                                IP:      net.IPNet{requestedIP, a.conf.Subnet.Mask},
                                Gateway: gw,
                                Routes:  a.conf.Routes,
@@ -123,7 +123,7 @@ func (a *IPAllocator) Get(id string) (*plugin.IPConfig, error) {
                        return nil, err
                }
                if reserved {
-                       return &plugin.IPConfig{
+                       return &types.IPConfig{
                                IP:      net.IPNet{cur, a.conf.Subnet.Mask},
                                Gateway: gw,
                                Routes:  a.conf.Routes,
@@ -135,7 +135,7 @@ func (a *IPAllocator) Get(id string) (*plugin.IPConfig, error) {
 
 // Allocates both an IP and the Gateway IP, i.e. a /31
 // This is used for Point-to-Point links
-func (a *IPAllocator) GetPtP(id string) (*plugin.IPConfig, error) {
+func (a *IPAllocator) GetPtP(id string) (*types.IPConfig, error) {
        a.store.Lock()
        defer a.store.Unlock()
 
@@ -165,7 +165,7 @@ func (a *IPAllocator) GetPtP(id string) (*plugin.IPConfig, error) {
                                _, bits := a.conf.Subnet.Mask.Size()
                                mask := net.CIDRMask(bits-1, bits)
 
-                               return &plugin.IPConfig{
+                               return &types.IPConfig{
                                        IP:      net.IPNet{cur, mask},
                                        Gateway: gw,
                                        Routes:  a.conf.Routes,
index 828b123..8d21955 100644 (file)
@@ -19,20 +19,19 @@ import (
        "fmt"
        "net"
 
-       "github.com/appc/cni/pkg/ip"
-       "github.com/appc/cni/pkg/plugin"
+       "github.com/appc/cni/pkg/types"
 )
 
 // IPAMConfig represents the IP related network configuration.
 type IPAMConfig struct {
        Name       string
-       Type       string         `json:"type"`
-       RangeStart net.IP         `json:"rangeStart"`
-       RangeEnd   net.IP         `json:"rangeEnd"`
-       Subnet     ip.IPNet       `json:"subnet"`
-       Gateway    net.IP         `json:"gateway"`
-       Routes     []plugin.Route `json:"routes"`
-       Args       *IPAMArgs      `json:"-"`
+       Type       string        `json:"type"`
+       RangeStart net.IP        `json:"rangeStart"`
+       RangeEnd   net.IP        `json:"rangeEnd"`
+       Subnet     types.IPNet   `json:"subnet"`
+       Gateway    net.IP        `json:"gateway"`
+       Routes     []types.Route `json:"routes"`
+       Args       *IPAMArgs     `json:"-"`
 }
 
 type IPAMArgs struct {
@@ -53,7 +52,7 @@ func LoadIPAMConfig(bytes []byte, args string) (*IPAMConfig, error) {
 
        if args != "" {
                ipamArgs := IPAMArgs{}
-               err := plugin.LoadArgs(args, &ipamArgs)
+               err := types.LoadArgs(args, &ipamArgs)
                if err != nil {
                        return nil, err
                }
index 76b9e84..18c9756 100644 (file)
@@ -19,8 +19,8 @@ import (
 
        "github.com/appc/cni/plugins/ipam/host-local/backend/disk"
 
-       "github.com/appc/cni/pkg/plugin"
        "github.com/appc/cni/pkg/skel"
+       "github.com/appc/cni/pkg/types"
 )
 
 func main() {
@@ -40,7 +40,7 @@ func cmdAdd(args *skel.CmdArgs) error {
        defer store.Close()
 
        ipamArgs := IPAMArgs{}
-       err = plugin.LoadArgs(args.Args, &ipamArgs)
+       err = types.LoadArgs(args.Args, &ipamArgs)
        if err != nil {
                return err
        }
@@ -51,7 +51,7 @@ func cmdAdd(args *skel.CmdArgs) error {
                return err
        }
 
-       var ipConf *plugin.IPConfig
+       var ipConf *types.IPConfig
 
        switch ipamConf.Type {
        case "host-local":
@@ -66,7 +66,7 @@ func cmdAdd(args *skel.CmdArgs) error {
                return err
        }
 
-       r := &plugin.Result{
+       r := &types.Result{
                IP4: ipConf,
        }
        return r.Print()
index e65e08e..2454977 100644 (file)
@@ -24,16 +24,17 @@ import (
        "syscall"
 
        "github.com/appc/cni/pkg/ip"
+       "github.com/appc/cni/pkg/ipam"
        "github.com/appc/cni/pkg/ns"
-       "github.com/appc/cni/pkg/plugin"
        "github.com/appc/cni/pkg/skel"
+       "github.com/appc/cni/pkg/types"
        "github.com/vishvananda/netlink"
 )
 
 const defaultBrName = "cni0"
 
 type NetConf struct {
-       plugin.NetConf
+       types.NetConf
        BrName string `json:"bridge"`
        IsGW   bool   `json:"isGateway"`
        IPMasq bool   `json:"ipMasq"`
@@ -183,7 +184,7 @@ func cmdAdd(args *skel.CmdArgs) error {
        }
 
        // run the IPAM plugin and get back the config to apply
-       result, err := plugin.ExecAdd(n.IPAM.Type, args.StdinData)
+       result, err := ipam.ExecAdd(n.IPAM.Type, args.StdinData)
        if err != nil {
                return err
        }
@@ -197,7 +198,7 @@ func cmdAdd(args *skel.CmdArgs) error {
        }
 
        err = ns.WithNetNSPath(args.Netns, false, func(hostNS *os.File) error {
-               return plugin.ConfigureIface(args.IfName, result)
+               return ipam.ConfigureIface(args.IfName, result)
        })
        if err != nil {
                return err
@@ -234,7 +235,7 @@ func cmdDel(args *skel.CmdArgs) error {
                return err
        }
 
-       err = plugin.ExecDel(n.IPAM.Type, args.StdinData)
+       err = ipam.ExecDel(n.IPAM.Type, args.StdinData)
        if err != nil {
                return err
        }
index 65e37b4..31a21d5 100644 (file)
@@ -22,14 +22,15 @@ import (
        "runtime"
 
        "github.com/appc/cni/pkg/ip"
+       "github.com/appc/cni/pkg/ipam"
        "github.com/appc/cni/pkg/ns"
-       "github.com/appc/cni/pkg/plugin"
        "github.com/appc/cni/pkg/skel"
+       "github.com/appc/cni/pkg/types"
        "github.com/vishvananda/netlink"
 )
 
 type NetConf struct {
-       plugin.NetConf
+       types.NetConf
        Master string `json:"master"`
        Mode   string `json:"mode"`
        MTU    int    `json:"mtu"`
@@ -122,7 +123,7 @@ func cmdAdd(args *skel.CmdArgs) error {
        }
 
        // run the IPAM plugin and get back the config to apply
-       result, err := plugin.ExecAdd(n.IPAM.Type, args.StdinData)
+       result, err := ipam.ExecAdd(n.IPAM.Type, args.StdinData)
        if err != nil {
                return err
        }
@@ -131,7 +132,7 @@ func cmdAdd(args *skel.CmdArgs) error {
        }
 
        err = ns.WithNetNS(netns, false, func(_ *os.File) error {
-               return plugin.ConfigureIface(args.IfName, result)
+               return ipam.ConfigureIface(args.IfName, result)
        })
        if err != nil {
                return err
@@ -146,7 +147,7 @@ func cmdDel(args *skel.CmdArgs) error {
                return err
        }
 
-       err = plugin.ExecDel(n.IPAM.Type, args.StdinData)
+       err = ipam.ExecDel(n.IPAM.Type, args.StdinData)
        if err != nil {
                return err
        }
index 236693e..c92de10 100644 (file)
@@ -22,14 +22,15 @@ import (
        "runtime"
 
        "github.com/appc/cni/pkg/ip"
+       "github.com/appc/cni/pkg/ipam"
        "github.com/appc/cni/pkg/ns"
-       "github.com/appc/cni/pkg/plugin"
        "github.com/appc/cni/pkg/skel"
+       "github.com/appc/cni/pkg/types"
        "github.com/vishvananda/netlink"
 )
 
 type NetConf struct {
-       plugin.NetConf
+       types.NetConf
        Master string `json:"master"`
        Mode   string `json:"mode"`
        MTU    int    `json:"mtu"`
@@ -126,7 +127,7 @@ func cmdAdd(args *skel.CmdArgs) error {
        }
 
        // run the IPAM plugin and get back the config to apply
-       result, err := plugin.ExecAdd(n.IPAM.Type, args.StdinData)
+       result, err := ipam.ExecAdd(n.IPAM.Type, args.StdinData)
        if err != nil {
                return err
        }
@@ -135,7 +136,7 @@ func cmdAdd(args *skel.CmdArgs) error {
        }
 
        err = ns.WithNetNS(netns, false, func(_ *os.File) error {
-               return plugin.ConfigureIface(args.IfName, result)
+               return ipam.ConfigureIface(args.IfName, result)
        })
        if err != nil {
                return err
@@ -150,7 +151,7 @@ func cmdDel(args *skel.CmdArgs) error {
                return err
        }
 
-       err = plugin.ExecDel(n.IPAM.Type, args.StdinData)
+       err = ipam.ExecDel(n.IPAM.Type, args.StdinData)
        if err != nil {
                return err
        }
index 58a0396..f14b55e 100644 (file)
@@ -26,9 +26,10 @@ import (
        "github.com/vishvananda/netlink"
 
        "github.com/appc/cni/pkg/ip"
+       "github.com/appc/cni/pkg/ipam"
        "github.com/appc/cni/pkg/ns"
-       "github.com/appc/cni/pkg/plugin"
        "github.com/appc/cni/pkg/skel"
+       "github.com/appc/cni/pkg/types"
 )
 
 func init() {
@@ -39,12 +40,12 @@ func init() {
 }
 
 type NetConf struct {
-       plugin.NetConf
+       types.NetConf
        IPMasq bool `json:"ipMasq"`
        MTU    int  `json:"mtu"`
 }
 
-func setupContainerVeth(netns, ifName string, mtu int, pr *plugin.Result) (string, error) {
+func setupContainerVeth(netns, ifName string, mtu int, pr *types.Result) (string, error) {
        var hostVethName string
        err := ns.WithNetNSPath(netns, false, func(hostNS *os.File) error {
                hostVeth, _, err := ip.SetupVeth(ifName, mtu, hostNS)
@@ -52,7 +53,7 @@ func setupContainerVeth(netns, ifName string, mtu int, pr *plugin.Result) (strin
                        return err
                }
 
-               err = plugin.ConfigureIface(ifName, pr)
+               err = ipam.ConfigureIface(ifName, pr)
                if err != nil {
                        return err
                }
@@ -64,7 +65,7 @@ func setupContainerVeth(netns, ifName string, mtu int, pr *plugin.Result) (strin
        return hostVethName, err
 }
 
-func setupHostVeth(vethName string, ipConf *plugin.IPConfig) error {
+func setupHostVeth(vethName string, ipConf *types.IPConfig) error {
        // hostVeth moved namespaces and may have a new ifindex
        veth, err := netlink.LinkByName(vethName)
        if err != nil {
@@ -100,7 +101,7 @@ func cmdAdd(args *skel.CmdArgs) error {
        }
 
        // run the IPAM plugin and get back the config to apply
-       result, err := plugin.ExecAdd(conf.IPAM.Type, args.StdinData)
+       result, err := ipam.ExecAdd(conf.IPAM.Type, args.StdinData)
        if err != nil {
                return err
        }
@@ -152,7 +153,7 @@ func cmdDel(args *skel.CmdArgs) error {
                }
        }
 
-       return plugin.ExecDel(conf.IPAM.Type, args.StdinData)
+       return ipam.ExecDel(conf.IPAM.Type, args.StdinData)
 }
 
 func main() {
index 059b2bb..2dd811d 100644 (file)
@@ -29,8 +29,9 @@ import (
        "strconv"
        "strings"
 
-       "github.com/appc/cni/pkg/plugin"
+       "github.com/appc/cni/pkg/ipam"
        "github.com/appc/cni/pkg/skel"
+       "github.com/appc/cni/pkg/types"
 )
 
 const (
@@ -39,7 +40,7 @@ const (
 )
 
 type NetConf struct {
-       plugin.NetConf
+       types.NetConf
        SubnetFile string                 `json:"subnetFile"`
        Delegate   map[string]interface{} `json:"delegate"`
 }
@@ -130,7 +131,7 @@ func delegateAdd(cid string, netconf map[string]interface{}) error {
                return err
        }
 
-       result, err := plugin.ExecAdd(netconf["type"].(string), netconfBytes)
+       result, err := ipam.ExecAdd(netconf["type"].(string), netconfBytes)
        if err != nil {
                return err
        }
@@ -199,8 +200,8 @@ func cmdAdd(args *skel.CmdArgs) error {
        n.Delegate["ipam"] = map[string]interface{}{
                "type":   "host-local",
                "subnet": fenv.sn.String(),
-               "routes": []plugin.Route{
-                       plugin.Route{
+               "routes": []types.Route{
+                       types.Route{
                                Dst: *fenv.nw,
                        },
                },
@@ -215,12 +216,12 @@ func cmdDel(args *skel.CmdArgs) error {
                return err
        }
 
-       n := &plugin.NetConf{}
+       n := &types.NetConf{}
        if err = json.Unmarshal(netconfBytes, n); err != nil {
                return fmt.Errorf("failed to parse netconf: %v", err)
        }
 
-       return plugin.ExecDel(n.Type, netconfBytes)
+       return ipam.ExecDel(n.Type, netconfBytes)
 }
 
 func main() {
diff --git a/test b/test
index 838880e..0e31514 100755 (executable)
--- a/test
+++ b/test
@@ -11,7 +11,7 @@
 source ./build
 
 TESTABLE="plugins/ipam/dhcp"
-FORMATTABLE="$TESTABLE pkg/ip pkg/ns pkg/plugin pkg/skel plugins/ipam/host-local plugins/main/bridge plugins/meta/flannel"
+FORMATTABLE="$TESTABLE libcni pkg/ip pkg/ns pkg/invoke pkg/types pkg/ipam pkg/skel plugins/ipam/host-local plugins/main/bridge plugins/meta/flannel"
 
 # user has not provided PKG override
 if [ -z "$PKG" ]; then