libcni: add util function InjectConf

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

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

README: distinguish between runtimes and plugins

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.

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

Merge pull request #199 from dcbw/fix-bridge-tx-queue-length

bridge: leave TX queue length as kernel default, not 0

Merge pull request #212 from squaremo/more-contrib

docs: Mention license headers and doc contributions

Merge pull request #214 from steveeJ/merge-v0.2

Merge v0.2

release script: don't run the tests

For two reasons:

1. They're not functional within rkt
2. They rebuild the binaries dynamically

build/release: link all release binaries statically

scripts: build static releases and create an ACI

* use SHA1 instead of MD5

scripts: add "release with rkt"

This script uses rkt and a fedora image to build release tarballs.

docs: Mention license headers and doc contributions

Closes #174.

Merge pull request #200 from squaremo/simplify-readme

Simplify README preamble

README: Simplify preamble

Merge pull request #175.

plugins/bridge: add support to set hairpin mode

macvlan: sysctl must be set in macvlan interface's namespace

The macvlan is initially created in a separate network namespace
and the sysctl must be set in that namespace too.

pkg/ns: evaluate syscall number at compile-time

Previously this code used a run-time map lookup keyed by
runtime.GOOS/GOARCH.  This version uses conditional compilation to make
this choice at compile time, giving immediate feedback for unsupported
platforms.

pkg/ns: use correct syscall number on arm

MAINTAINERS: add Tom Denham and Gabe Rosenhouse

plugins/{ptp,bridge}: teardown first

This will allow the IPAM allocations to be cleared in case the
interfaces and iptables rules are non-existent.

*: appc/cni -> containernetworking/cni

The project has been moved so internally we simply rename everything.
Consumers are recommended to update their vendored version of cni.

README: fix badge links

README: fix badges

README: add contiv to cni supported networking

plugins/bridge: clean masquerading rules

In the Add command we set up masquerading rules that didn't have a
corresponding clean-up code in Del.

Add the clean-up code.

Remove reference to policy on accepting PRs

There is to policy

macvlan: set proxy_arp in time of creating interface

Resolves CNI part of https://github.com/coreos/rkt/issues/1765
Second part would be adding similar lines into kvm flavored macvlan
support (in time of creating macvtap device).

pkg/utils: add functions to work with sysctl

README: clarify what CNI is

The word "standard" does not represent what CNI is actually trying to
be.

Document use of goroutine and lockosthread in test helpers

ns: fix reading net namespace in multi-threaded processes

/proc/self/ns/net gives the main thread's namespace, not necessarily
the namespace of the thread that's running the testcases.  This causes
sporadic failures of the tests.

For example, with a testcase reading inodes after switching netns:

/proc/27686/task/27689/ns/net 4026532565
/proc/self/ns/net 4026531969
/proc/27686/task/27689/ns/net 4026532565

See also:
https://github.com/vishvananda/netns/commit/008d17ae001344769b031375bdb38a86219154c6

Running Suite: pkg/ns Suite
===========================
Random Seed: 1459953577
Will run 6 of 6 specs

• Failure [0.028 seconds]
Linux namespace operations
/cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:167
  WithNetNS
  /cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:166
    executes the callback within the target network namespace [It]
    /cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:97

    Expected
        <uint64>: 4026531969
    to equal
        <uint64>: 4026532565

    /cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:96
------------------------------
•••••

Summarizing 1 Failure:

[Fail] Linux namespace operations WithNetNS [It] executes the callback within the target network namespace
/cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:96

Ran 6 of 6 Specs in 0.564 seconds
FAIL! -- 5 Passed | 1 Failed | 0 Pending | 0 Skipped --- FAIL: TestNs (0.56s)
FAIL

Fix issues with MakeNetworkNS test helper

Add basic unit tests of testhelpers

Extract inode inspection functions into testhelpers

Extract testhelpers from loopback test suite

README: add some material on what CNI isn't

We get some questions about this stuff from time to time and I want to
acknowledge we have thought about them but aren't currently tackling
them.