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

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.

README: add some more details on what CNI is

We needed a bit more intro material to scope what CNI does. Here is my
attempt.

The macvlan is initially created in a separate network namespaceand the sysctl must be set in that namespace too.Fixes #208.

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

Fixes #208.

pkg/ns: setns syscall number on arm and evaluate them at build time.

Dummy merge due to previous manual merge.

Merge branch 'anguslees-master'

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

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

Not using NewLinkAttrs() or not initializing TxQLen leaves
the value as 0, which tells the kernel to set a zero-length
tx_queue_len.  That messes up FIFO traffic shapers (like pfifo)
that use the device TX queue length as the default packet
limit.  This leads to a default packet limit of 0, which drops
all packets.

Merge pull request #201 from jonboulle/master

MAINTAINERS: add Tom Denham and Gabe Rosenhouse

Merge pull request #205 from steveeJ/teardown-fix

plugins/{ptp,bridge}: teardown first

plugins/{ptp,bridge}: teardown first

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

MAINTAINERS: add Tom Denham and Gabe Rosenhouse

Merge pull request #198 from steveeJ/org-move

*: appc/cni -> containernetworking/cni

README: fix badge links

README: fix badges

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

Merge pull request #197 from jainvipin/master

Add Contiv to CNI supported networking

README: add contiv to cni supported networking

Merge branch 'pr195' (clean up masq rules)

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.

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.

Merge pull request #189 from tomdee/patch-2

Remove reference to policy on accepting PRs

Remove reference to policy on accepting PRs

There is to policy

Merge pull request #187 from appc/v0.3.0

merge unneeded branch v0.3.0 into master

Merge pull request #186 from steveeJ/project-clarification

README: clarify what CNI is

README: clarify what CNI is

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

Merge pull request #182 from rosenhouse/document-lockosthread

Document use of goroutine and LockOSThread in test helpers

Merge pull request #176 from dcbw/ns-test-fix

ns: fix reading net namespace in multi-threaded processes

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

Document use of goroutine and lockosthread in test helpers

Merge pull request #181 from rosenhouse/testhelpers-extraction

Extract and improve test helpers

Fix issues with MakeNetworkNS test helper

Add basic unit tests of testhelpers

Extract inode inspection functions into testhelpers

Extract testhelpers from loopback test suite

Merge pull request #180 from philips/add-more-intro

README: add some more introductory material

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.

README: add some more details on what CNI is

We needed a bit more intro material to scope what CNI does. Here is my
attempt.

Merge pull request #177 from steveeJ/macvlan-arp-proxy

Macvlan arp proxy

Merge pull request #178 from steveeJ/master

travis: don't go get vet

travis: don't go get vet