docs: table formatting is hard

docs: fill-out and correct version conversion table

docs: fill-out and correct version conversion table

docs: minor improvements to spec-upgrades

docs: minor improvements to spec-upgrades

docs: add small upgrade instructions

docs: add small upgrade instructions

docs: minor improvements to 0.3.0 upgrade guidance

docs: minor improvements to 0.3.0 upgrade guidance

docs: Edits to v0.3.0 upgrade guidance

docs: Edits to v0.3.0 upgrade guidance

docs: CNI versioning for 0.3.0 upgrade

docs: CNI versioning for 0.3.0 upgrade

libcni: up-convert a Config to a ConfigList when no other configs are found.

Merge pull request #379 from Intel-Corp/master

README: List multus as 3rd party plugin

vendor: Update vishvanana/netlink dependency

pkg/ip: refactored so that builds succeed on non-linux platforms

moved functions that depend on linux packages into a separate file and added
nop methods with a build tag for non-linux platforms in a new file.

pkg/ip: refactored so that builds succeed on non-linux platforms

moved functions that depend on linux packages into a separate file and added
nop methods with a build tag for non-linux platforms in a new file.

pkg/ns, pkg/types: refactored non linux build fix code to

Make GetCurrentNS platform specific instead of getCurrentThreadNetNSPath

pkg/ns, pkg/types: refactored non linux build fix code to

Make GetCurrentNS platform specific instead of getCurrentThreadNetNSPath

Merge pull request #371 from rosenhouse/spec-netconf-routes

spec: Remove `routes` from Network Configuration

Replace Michael Bridgen with Bryan Boreham

README: List multus as 3rd party plugin

refer #343 issue

Merge pull request #376 from zackhsi/fix-grammar

Fix grammar

plugins/main/ptp: set the Sandbox property on the response

Fix grammar

pkg/ns: refactored so that builds succeed on non-linux platforms

moved functions that depend on linux packages (sys/unix) into a separate file
and added nop methods with a build tag for non-linux platforms in a new file.

pkg/ns: refactored so that builds succeed on non-linux platforms

moved functions that depend on linux packages (sys/unix) into a separate file
and added nop methods with a build tag for non-linux platforms in a new file.

Merge pull request #372 from rosenhouse/consolidate-host-local-docs

docs: consolidate host-local documentation

docs: consolidate host-local documentation

- remove the README.md in the plugins/ipam/host-local (no other plugin
  has this)
- merge that info into the main Documentation/host-local.md

docs: consolidate host-local documentation

- remove the README.md in the plugins/ipam/host-local (no other plugin
  has this)
- merge that info into the main Documentation/host-local.md

spec: Remove `routes` from Network Configuration

- this is supported by some IPAM plugins but is not required of all of
  them

Merge pull request #369 from containernetworking/port-mapping

CONVENTIONS.md: Update details on port-mappings

More markups

Update with feedback

CONVENTIONS.md: Update details on port-mappings

After discussion with the CNI maintainers and representatives from Mesos and Kubernetes we've settled on this approach for passing the dynamic port mapping information from runtimes to plugins.

Including it in the plugin config allows "operators" to signal that they expect port mapping information to be passed to a specific plugin (or plugins). This follows the model that config passed in the plugin config should be acted upon by a plugin.

Runtimes can optionally return an error to users if they find no plugin in the chain that supports port mappings.

I'm sure we'd all like to see this merged soon so I'll try to turn any feedback quickly (and I'm happy to get any and all feedback - spellings, unclear working, wrong JSON terminology etc..)

Merge pull request #361 from ofiliz/issue-360

invoke: Enable plugin file names with extensions

Merge pull request #361 from ofiliz/issue-360

invoke: Enable plugin file names with extensions

invoke: Enable plugin file names with extensions

A CNI network configuration file contains the plugin's executable file name.
Some platforms like Windows require a file name extension for executables.
This causes unnecessary burden on admins as they now have to maintain two
versions of each type of netconfig file, which differ only by the ".exe"
extension. A much simpler design is for invoke package to also look for
well-known extensions on platforms that require it. Existing tests are
improved and new tests are added to cover the new behavior.

Fixes #360

invoke: Enable plugin file names with extensions

A CNI network configuration file contains the plugin's executable file name.
Some platforms like Windows require a file name extension for executables.
This causes unnecessary burden on admins as they now have to maintain two
versions of each type of netconfig file, which differ only by the ".exe"
extension. A much simpler design is for invoke package to also look for
well-known extensions on platforms that require it. Existing tests are
improved and new tests are added to cover the new behavior.

Fixes #360

Merge pull request #364 from pires/build_tag

pkg/utils/sysctl/sysctl_linux.go: couple fixes

pkg/utils/sysctl/sysctl_linux.go: fix typo.

pkg/utils/sysctl/sysctl_linux.go: fix typo.

pkg/utils/sysctl/sysctl_linux.go: fix build tag.

pkg/utils/sysctl/sysctl_linux.go: fix build tag.

Merge pull request #359 from ofiliz/issue-355

libcni, pkg\invoke: Use OS-specific list separator when parsing CNI_PATH

spec, libcni, pkg/invoke: Use OS-agnostic separator when parsing CNI_PATH

Hardcoding the list separator character as ":" causes CNI to fail when parsing
CNI_PATH on other operating systems. For example, Windows uses ";" as list
separator because ":" can legally appear in paths such as "C:\path\to\file".
This change replaces use of ":" with OS-agnostic APIs or os.PathListSeparator.

Fixes #358

spec, libcni, pkg/invoke: Use OS-agnostic separator when parsing CNI_PATH

Hardcoding the list separator character as ":" causes CNI to fail when parsing
CNI_PATH on other operating systems. For example, Windows uses ";" as list
separator because ":" can legally appear in paths such as "C:\path\to\file".
This change replaces use of ":" with OS-agnostic APIs or os.PathListSeparator.

Fixes #358

Merge pull request #145 from dcbw/pass-ifname-back

Pass interface name and MAC back to runtime

spec/plugins: return interface details and multiple IP addresses to runtime

Updates the spec and plugins to return an array of interfaces and IP details
to the runtime including:

- interface names and MAC addresses configured by the plugin
- whether the interfaces are sandboxed (container/VM) or host (bridge, veth, etc)
- multiple IP addresses configured by IPAM and which interface they
have been assigned to

Returning interface details is useful for runtimes, as well as allowing
more flexible chaining of CNI plugins themselves.  For example, some
meta plugins may need to know the host-side interface to be able to
apply firewall or traffic shaping rules to the container.

spec/plugins: return interface details and multiple IP addresses to runtime

Updates the spec and plugins to return an array of interfaces and IP details
to the runtime including:

- interface names and MAC addresses configured by the plugin
- whether the interfaces are sandboxed (container/VM) or host (bridge, veth, etc)
- multiple IP addresses configured by IPAM and which interface they
have been assigned to

Returning interface details is useful for runtimes, as well as allowing
more flexible chaining of CNI plugins themselves.  For example, some
meta plugins may need to know the host-side interface to be able to
apply firewall or traffic shaping rules to the container.

pkg/ipam: add testcases

pkg/ipam: add testcases

macvlan/ipvlan: use common RenameLink method

plugins/flannel: organize test JSON alphabetically

Otherwise the test fails, since Go's JSON marshaller prints
dict items alphabetically in its String() call.

types: make Result an interface and move existing Result to separate package

macvlan/ipvlan: use common RenameLink method

testutils: pass netConf in for version operations; pass raw result out for tests

types: make Result an interface and move existing Result to separate package

testutils: pass netConf in for version operations; pass raw result out for tests

Merge pull request #355 from rosenhouse/skel-main-errors

skel: adds PluginMainWithError which returns a *types.Error

skel: adds PluginMainWithError which returns a *types.Error

Enables plugins to do their own error-handling and cleanup logic

skel: adds PluginMainWithError which returns a *types.Error

Enables plugins to do their own error-handling and cleanup logic

Merge pull request #352 from containernetworking/conventions

Documentation: Add conventions doc

Update CONVENTIONS.md

Merge pull request #346 from dcbw/ordered-chaining

api/plugins: implement ordered plugin chaining

api,libcni: add network config list-based plugin chaining

Using a new ".configlist" file format that allows specifying
a list of CNI network configurations to run, add new libcni
helper functions to call each plugin in the list, injecting
the overall name, CNI version, and previous plugin's Result
structure into the configuration of the next plugin.

api,libcni: add network config list-based plugin chaining

Using a new ".configlist" file format that allows specifying
a list of CNI network configurations to run, add new libcni
helper functions to call each plugin in the list, injecting
the overall name, CNI version, and previous plugin's Result
structure into the configuration of the next plugin.

spec: add network configuration list specification

Spec/Conventions: Update to include plugin config

noop: allow specifying debug file in config JSON

Chaining sends different config JSON to each plugin, but the same
environment, and if we want to test multiple noop plugin runs in
the same chain we need a way of telling each run to use a different
debug file.

Documentation: Add conventions doc

Merge pull request #334 from squeed/host-local-dns

ipam/host-local: add ResolvConf argument for DNS configuration

Merge pull request #350 from rosenhouse/spec-version

spec: notice of version

spec: notice of version

ipam/host-local: add ResolvConf argument for DNS configuration

This adds the option `resolvConf` to the host-local IPAM configuration.
If specified, the plugin will try to parse the file as a resolv.conf(5)
type file and return it in the DNS response.

ipam/host-local: add ResolvConf argument for DNS configuration

This adds the option `resolvConf` to the host-local IPAM configuration.
If specified, the plugin will try to parse the file as a resolv.conf(5)
type file and return it in the DNS response.

Merge pull request #115 from aanm/implementing-ipam-as-lib

ipam/host-local: Move allocator and config to backend

ipam/host-local: Move allocator and config to backend

Signed-off-by: André Martins <aanm90@gmail.com>

Merge pull request #325 from squeed/update-maintainer

MAINTAINERS: hi CaseyC!

MAINTAINERS: hi CaseyC!

Merge pull request #345 from rosenhouse/travis-go-versions

travis: roll forward the versions of Go that we test

travis: roll forward the versions of Go that we test

Merge pull request #314 from rosenhouse/noop-helpful-message

plugins/noop: return a helpful message for test authors

Merge pull request #344 from dcbw/interface-checks

all: assert internal objects implement interfaces

Merge branch 'master' into interface-checks

Merge pull request #341 from dcbw/host-local-trim

host-local: trim whitespace from container IDs and disk file contents

all: assert internal objects implement interfaces

all: assert internal objects implement interfaces

Merge pull request #340 from vzcambria/master

Fix README.md host-local IPAM range start/end typo

host-local: trim whitespace from container IDs and disk file contents

It doesn't seem like container IDs should really have whitespace or
newlines in them.  As a complete edge-case, manipulating the host-local
store's IP reservations with 'echo' puts a newline at the end, which
caused matching to fail in ReleaseByID().  Don't ask...

Fix README.md host-local IPAM range start/end typo

plugins/noop: return a helpful message for test authors

Signed-off-by: Gabe Rosenhouse <grosenhouse@pivotal.io>

Merge pull request #335 from JulienBalestra/json_ext

libcni conf: support .json file extension

libcni conf: support .json file extension

Content of the files are json, the configuration load should support .json

Merge pull request #322 from cf-container-networking/flannel-cni-stateDir

Allow flannel CNI plugin stateDir to be configurable

flannel: rename stateDir to dataDir

Rename StateDir to DataDir for flannel CNI plugin

flannel: rename stateDir to dataDir

Rename StateDir to DataDir for flannel CNI plugin

Merge pull request #330 from bboreham/ns-warning

Expand description of goroutine+netns problems

Expand description of goroutine+netns problems