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

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.

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

testing: test invocation of newer plugins with an older libcni

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.

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

vendor: Bump golang.org/x/sys/unix to a newer version

The current vendor of sys/unix is really old, and doesn't work on arm64 and ppc64le
Updating to the latest version might also fix other issues

ref #209

Merge pull request #267 from rosenhouse/version-cmd

Adds VERSION command

Merge pull request #273 from zreigz/reconfigure-bridge-ip

Reconfigure bridge IP address

Merge pull request #274 from lukasredynk/update_netlink

Update of netlink version

Merge pull request #280 from prateekgogia/bug-fix-155

pkg/ip: Return correct error if container name provided exists, and test cases

pkg/ip: Return correct error if container name provided exists, and test cases

If interface name for a container provided by a user is already present,
Veth creation fails with incorrect error.
If os.IsExist error is returned by makeVethPair:
* Check for peer name, if exists generate another random peer name,
* else, IsExist error is due to container interface present, return error.

Fixes #155

libcni: add util function InjectConf

Merge pull request #277 from steveeJ/test-allocator-reserveip

ipam/host-local/allocator tests: cover requested IP

documentation: add description for forceAddress parameter

plugins: reconfigure bridge IP address

Add possibility to reconfigure bridge IP address when there is a new value.
New boolean flag added to net configuration to force IP change if it is need.
Otherwise code behaves as previously and throws error

ipam/host-local/allocator tests: cover requested IP

Further:
* improve error handling

Merge pull request #254 from steveeJ/ip-based-macaddr

pkg/ipam: IP based macaddr

pkg/utils/hwaddr tests: cover v4 in v6 addr

Update of netlink version

This is required for
https://github.com/containernetworking/cni/issues/251 because version of
netlink used doesn't support creating tap devices.

Merge pull request #272 from rosenhouse/types-of-users

README: distinguish between runtimes and plugins

plugins/{bridge,macvlan} test: ensure hardware addr

* bridge: Test the following interface's hardware address for the CNI specific
prefix:
  - bridge with IP address
  - container veth
* plugins/macvlan test: ensure hardware addr

pkg/ip link_test: ensure SetHWAddrByIP has an effect

plugins/ptp: set the host veth hwaddr correctly

pkg/ip: use iface name in SetHWAddrByIP

pkg/ip tests: cover SetupVeth and DelLinkByName*

plugins: set MAC addresses based on IP

This will give deterministic MAC addresses for all interfaces CNI
creates and manages the IP for:
* bridge: container veth and host bridge
* macvlan: container veth
* ptp: container veth and host veth

pkg/utils/hwaddr: migrate code from IPAM pkg

pkg/ipam{,test}: improve error handling and tests

* _suite.go and _test.go file should be in the same package, using the
  _test package for that, which requires some fields and methods to be
  exported
* Introduce error type for cleaner error handling
* test adaptions for error type checking

add tests for generateHardwardAddr

configure mac address based on assigned ip

Merge pull request #270 from rosenhouse/integration-tests

Add integration tests of libcni

plugins: cleanup tests of no-op plugin

README: distinguish between runtimes and plugins

Adds a link to the 3rd party SR-IOV plugin.

docs: fix tuning doc, correct the result json

it does not report the version, it never did

spec: add version command and increment version to 0.2.0

spec: remove CNI_VERSION env var, was never been implemented

No one uses this, and it is causing confusion (see #150)

libcni: add integration test coverage

plugins: adds new no-op plugin that may be used as a test-double

Plugin can be configured to record all inputs and to respond with
arbitrary stdout or error message.  Will support upcoming integration
testing.

Merge pull request #269 from rosenhouse/skel-dependency-injection

Refactor skel with dependency injection

misc: fix up copyright dates

Merge pull request #263 from feiskyer/ptp-test

plugins/main/ptp: add functional tests

pkg/skel: plugins now respond to VERSION command

To support CNI spec versioning, plugins must be able to report version
information to container runtimes.

pkg/skel: improve error message for missing CNI_COMMAND env var

This makes the error message for missing CNI_COMMAND consistent with
that of other required environment variables.

pkg/skel: missing env var log lines appear in stderr

Previously, the log lines appeared in stdout before the JSON encoding of
the error message.  That would break JSON parsing of stdout.  Instead, we use
stderr for these unstructured logs, consistent with the CNI spec.

pkg/skel: refactor to use dependency injection

Extract dependencies on os to enable more complete unit test coverage