pkg/testutils: return errors after restoring stdout

Ensures Ginkgo is able to print detailed failure messages instead
of them being captured by the pipe.

Merge pull request #379 from Intel-Corp/master

README: List multus as 3rd party plugin

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

spec: Remove `routes` from Network Configuration

README: List multus as 3rd party plugin

refer #343 issue

Merge pull request #376 from zackhsi/fix-grammar

Fix grammar

Fix grammar

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

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

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 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

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.

pkg/ipam: add testcases

plugins/flannel: organize test JSON alphabetically

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

macvlan/ipvlan: use common RenameLink method

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

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.

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.

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

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

Merge pull request #330 from bboreham/ns-warning

Expand description of goroutine+netns problems

Expand description of goroutine+netns problems

Merge pull request #327 from dcbw/host-local-tests

host-local: add DataDir IPAM conf option and use it for testcases

flannel: updated flannel test to use pkg/testutils

reworked the flannel test to use testutils CmdAddWithResult and
CmdDelWithResult

Merge pull request #328 from markstgodard/noop-cni-args

noop plugin: support multiple CNI_ARGS

noop plugin: support multiple CNI_ARGS

Updated noop plugin to parse the CNI_ARGS pairs, to allow more than just
the DEBUG arg.

Merge pull request #324 from lstoll/lstoll-ipvlan-l3s

ipvlan: Support L3-Symmetric mode

host-local: add DataDir IPAM conf option and use it for testcases

Add an e2e host-local plugin testcase, which requires being able
to pass the datadir into the plugin so we can erase it later.
We're not always guaranteed to have access to the default data
dir location, plus it should probably be configurable anyway.

flannel: update flannel documentation

add optional 'stateDir' to flannel docs

Merge pull request #307 from squeed/backwards-compatability-tests

testing: test invocation of newer plugins with an older libcni

flannel: add unit tests for config loading

backfill unit tests to add coverage for loadFlannelNetConf and
loadFlannelSubnetEnv

ipvlan: Support ipvlan l3s mode.

This landed in kernel in 4fbae7d

vendor: github.com/vishvananda/netlink@a1f8555

Update to current master, to get IPVLAN L3s def.

flannel: add integration test suite

add new test for flannel plugin that delegates to the noop plugin and
validates that 'storeDir' can be configurable

Allow flannel CNI plugin stateDir to be configurable

- Add optional 'stateDir' to flannel NetConf, if not present default to
/var/lib/cni/flannel

Signed-off-by: Jay Dunkelberger <ldunkelberger@pivotal.io>

testing: test invocation of newer plugins with an older libcni

Merge pull request #312 from tomdee/slack

Add slack badge and link

Add slack badge and link

Merge pull request #311 from rkamudhan/patch-2

Update ipvlan.md

Update ipvlan.md

invalid json format

Merge pull request #308 from rosenhouse/fix-skel-tests-containerid

skel tests: correct name of CNI_CONTAINERID in tests of env vars

skel tests: correct name of CNI_CONTAINERID in tests of env vars

Merge pull request #305 from yuewko/add_infoblox_link

Add link to Infoblox IPAM plugin

Add link to Infoblox IPAM plugin

Added link to Infoblox IPAM plugin in README.

Merge pull request #304 from danp/patch-1

docs/ptp: fix markdown formatting

docs/ptp: fix markdown formatting

Merge pull request #301 from rosenhouse/backwards-compatibility-tests

A test verifies that libcni is backwards compatible with legacy plugins

travis: git clone at great depth

ensures that git commits referenced by backwards compatibility tests are
always available in CI.

Merge pull request #302 from rosenhouse/readme-3rd-party-plugins

3rd party plugins belong in their own repos

docs: on contributing plugins to this repo

testing: adds basic test of backwards compatibility

Merge pull request #296 from rosenhouse/plugins-require-versioned-config

Plugins validate cniVersion of NetConf

Merge pull request #298 from freehan/fix-host-local-30

host-local: fix allocation of last address in a /30

host-local: fix allocation of last address in a /30

Merge pull request #299 from tgraf/cilium-link

List cilium as 3rd party plugin

skel: use named constant for Incompatible CNI Version error code

versioning: ipam config is not versioned

versioning: plugins require version match with config

infer version 0.1.0 when config is missing an explicit "cniVersion" field

versioning: add basic version decode for network config

README: List cilium as 3rd party plugin

Signed-off-by: Thomas Graf <tgraf@suug.ch>