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.

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

plugins/main/ptp: add functional tests

Merge pull request #264 from rosenhouse/test-with-vagrant

Add Vagrantfile, document how to run test suite in a vagrant VM

Add Vagrantfile, document how to run test suite in a vagrant VM

Merge pull request #247 from tomdee/spec-args-field

SPEC: introduce "args" field and new error code

SPEC: introduce "args" field and new error code

Based on previous discussions on the CNI maintainers calls, the spec is
unclear on 1) when CNI_ARGS should be used and 2) the fact the dynamic
config can be passed in through the network JSON.

This PR makes it clear that per-container config can be passed in
through the network JSON, adding a top level `args` field into
which orchestrators can add additional metadata without worrying that
plugins might reject the additional data. It also allows for plugins to
reject unknown fields passed in at the top level.

Using JSON is preferable to CNI_ARGS since it allows namespaced and
structured data. CNI_ARGS is a flat list of KV pairs which has reserved
characters with no escaping rules defined.
CNI_ARGS may still be used by orchestrators that want the simplicity of
passing the network config JSON as specified by the user, unchanged
through to the CNI plugin. But for any kind of structured data, it's
recommended that the `args` field in the JSON is used instead.

README: grammatical corrections

pkg/ns: fix misspelling in comment

Merge pull request #240 from steveeJ/unmarshall-string

pkg/types: unmarshal string tests

pkg/types: cover string for unmarshal tests

Merge pull request #238 from tomdee/string-unmarshaller

pkg/types: Add UnmarshallableString type

Merge pull request #243 from containernetworking/maintainers-update-email

MAINTAINERS: Update @tomdee email address

MAINTAINERS: Update @tomdee email address

pkg/types: Add UnmarshallableString type

Allow strings to be unmarshalled for CNI_ARGS

CNI_ARGS uses types.LoadArgs to populate a struct.
The fields in the struct must meet the TextUnmarshaler interface.

This code adds a UnmarshallableString type to assist with this.

Merge pull request #229 from steveeJ/cleanup-travis

travis: cleanup and bump go versions

plugins/ipam: round robin ip allocation for host-local ipam

This changes the ip allocation logic to round robin. Before this, host-local IPAM searched for available IPs from start of subnet. Hence it tends to allocate IPs that had been used recently. This is not ideal since it may cause collisions.

Merge pull request #239 from steveeJ/master

MAINTAINERS: remove @zachgersh, add @dcbw

Merge pull request #218 from steveeJ/roadmap

Add ROADMAP document

ROADMAP: initial version

MAINTAINERS: remove @zachgersh, add @dcbw

Thank you Zach for all the great work done on CNI, farewell!
At the same time we are happy to welcome Dan amongst us who has already
contributed lots of valuable work!

Merge pull request #219 from squaremo/more-contrib

docs: details on PR acceptance policy

docs: details on PR acceptance policy

Merge pull request #228 from jieyu/add_mesos

Added Mesos to CNI users.

Merge pull request #231 from steveeJ/ns-verifiy-errors

pkg/ns: introduce error types to indicate NS verification

pkg/ns: introduce error types indicate NS verification

Merge pull request #230 from steveeJ/netns-optional-on-del

plugins: don't require CNI_NETNS for DEL command

skel/test: add case for empty NETNS

plugins: don't require CNI_NETNS for DEL command

This will allow to free up the IPAM allocations when the caller doesn't
have access to the network namespace anymore, e.g. due to a reboot.

pkg/skel: allow arg requriements specified by CMD

README: add Mesos as CNI users

travis: use go1.5.4

travis: cleanup and bump go versions

* cleanup structure
* explicitly use go-{1.5.3,1.6.2}

Merge pull request #227 from steveeJ/ns-verify

pkg/ns: consider PROCFS during NS verification

pkg/ns: consider PROCFS during NS verification

This is an attempt to bring compatibility with Kernel <3.19, where NSFS
where PROCFS was used for network namespaces.

Merge pull request #222 from steveeJ/ns-check-path

pkg/ns: verify netns when initialized with GetNS

pkg/ns: test IsNSFS()

pkg/ns: test case for rejecting a non-ns nspath

pkg/ns: verify netns when initialized with GetNS

Merge pull request #223 from steveeJ/ns-respect-close

pkg/ns: don't allow operations after Close()

pkg/ns: add tests cases for Close()'d NS

pkg/ns: don't allow operations after Close()

Merge pull request #217 from steveeJ/fix-go-tip

build: don't be verbose / test: fix bash variable escaping in sudo command

build: remove -x bash flag

test: fix bash variable escaping in sudo command

Merge pull request #50 from steveeJ/gwfix

bridge: add isDefaultGateway field

tests/bridge: ensure isDefaultGateway works

bridge: add isDefaultGateway field

When isDefaultGateway is true it automatically sets isGateway to true.
The default route will be added via the (bridge's) gateway IP.
If a default gateway has been configured via IPAM in the same
configuration file, the plugin will error out.

Merge pull request #211 from dcbw/e2e-testing

Fix namespace switch issues and add ipvlan, macvlan, and bridge e2e testing

bridge: add e2e testing

macvlan: add e2e testing

ipvlan: add e2e testing

testutils: add e2e testing helper code

ns: add interface, use it, and fix thread-related namespace switch issues

Add a namespace object interface for somewhat cleaner code when
creating and switching between network namespaces.  All created
namespaces are now mounted in /var/run/netns to ensure they
have persistent inodes and paths that can be passed around
between plugin components without relying on the current namespace
being correct.

Also remove the thread-locking arguments from the ns package
per https://github.com/appc/cni/issues/183 by doing all the namespace
changes in a separate goroutine that locks/unlocks itself, instead of
the caller having to track OS thread locking.