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>

skel: Plugins require a cniVersion in the NetConf

Merge pull request #287 from rosenhouse/multi-version

Plugins report a list of supported versions

Merge pull request #295 from rosenhouse/update-readme-cloudfoundry-link

readme: update link to CloudFoundry usage of CNI

readme: update link to CloudFoundry usage of CNI

versioning: revert spec version to 0.2.0

versioning: misc cleanups

highlights:
 - NetConf struct finally includes cniVersion field
 - improve test coverage of current version report behavior
 - godoc a few key functions
 - allow tests to control version list reported by no-op plugin

Merge pull request #294 from dcbw/host-local-ranges

Fix handling of host-local IP address ranges

host-local: don't allocate the broadcast address or allow invalid networks

There aren't any IPs to allocate in /32 or /31 networks, so don't allow them.

host-local: ensure requested IPs are within the given range

And also make sure that RangeStart and RangeEnd are sane.

host-local: don't allocate past RangeEnd

When RangeEnd is given, a.end = RangeEnd+1.

If when getSearchRange() is called and lastReservedIP equals
RangeEnd, a.nextIP() only compares lastReservedIP (which in this
example is RangeEnd) against a.end (which in this example is
RangeEnd+1) and they clearly don't match, so a.nextIP() returns
start=RangeEnd+1 and end=RangeEnd.

Get() happily allocates RangeEnd+1 because it only compares 'cur'
to the end returned by getSearchRange(), not to a.end, and thus
allocates past RangeEnd.

Since a.end is inclusive (eg, host-local will allocate a.end) the
fix is to simply set a.end equal to RangeEnd.

invoke: better name and unit test coverage for GetVersionInfo

invoke: correctly infer version for 0.1.0-vintage plugins

Older plugins return a known error when issued the VERSION command.
Capture this error and report it as a 0.1.0 version plugin.

versioning: adds tooling to compile a program against a given old CNI version

Allows us to write tests that cover interactions between components of
differing versions

pkg/invoke: refactor plugin exec and backfill unit tests

versioning: document meaning of 'Legacy' version support

versioning: plugins report a list of supported versions

Further progress on versioning support (Issue #266).
Bump CNI spec version to 0.3.0

Merge pull request #293 from rosenhouse/backfill-invoke-exec-tests

Backfill tests for plugin execution

Merge pull request #290 from asridharan/dcos

Fixed the check for network namespace path.

pkg/ns: fixed the check for network namespace path.

The expectation on older kernels (< 3.19) was to have the network
namespace always be a directory. This is not true if the network
namespace is bind mounted to a file, and will make the plugin fail
erroneously in such cases. The fix is to remove this assumption
completely and just do a basic check on the file system types being
returned.

Fixes #288

Merge pull request #291 from tomdee/SetupVeth

pkg/ip: Ensure that SetupVeth returns correct hostVeth

pkg/ip: Ensure that SetupVeth returns correct hostVeth

The veth is moved from the container NS to the host NS.
This is handled by the code that sets the link to UP but the wrong
hostVeth is returned to the calling code.

invoke: backfill tests for plugin execution

Merge pull request #241 from luxas/add_platforms

Add arm64 and ppc64le as new platforms

release: Release binaries for arm, arm64 and ppc64le too

Modify the releasing script to cross-compile for the new architectures, but also keep backwards-compability

ref #209

travis: Cross-compile for arm, arm64 and ppc64le on every build

Cross-compile cni for arm, arm64 and ppc64le with go1.6 only
Allow go tip to fail
Set fast_finish to true, which means travis will instantly return build failure when any of the required builds fail

ref #209

build: Update the build script to make it possible to build for other architectures

This makes it possible to cross-compile cni like so:
$ GOARCH=arm ./build
$ GOARCH=arm64 ./build
$ GOARCH=ppc64le ./build

ref #209