Fix issues with MakeNetworkNS test helper

Fix issues with MakeNetworkNS test helper

Add basic unit tests of testhelpers

Add basic unit tests of testhelpers

Extract inode inspection functions into testhelpers

Extract inode inspection functions into testhelpers

Extract testhelpers from loopback test suite

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

travis: don't go get vet

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

pkg/utils: add functions to work with sysctl

Merge pull request #173 from jonboulle/master

*: add missing license headers + check

*: add missing license headers + check

*: add missing license headers + check

Merge pull request #172 from squaremo/readme_more

Readme improvements

docs: add brief section on contributing, to README

docs: cosmetic edits to README.md

Merge pull request #168 from steveeJ/chain-len

attempt to ensure chain name is unique and at most 29 chars long

pkg/utils: correct the test's expected chain names

pkg/utils: correct the test's expected chain names

*: add comment to iptables rules for ipmasq

*: add comment to iptables rules for ipmasq

pkg/utils: remove unneeded condition in tests

pkg/utils: remove unneeded condition in tests

pkg/utils: fix docstring

pkg/utils: fix docstring

pkg/utils: split and unexport constants

pkg/utils: split and unexport constants

pkg/utils: fix docstring

pkg/utils: fix docstring

pkg/utils: use constant for chain prefix

pkg/utils: use constant for chain prefix

pkg/utils: use name+id for hash and extend tests

pkg/utils: use name+id for hash and extend tests

pkg: add a function to generate chain names

Adds a function to generate chain names for use in iptables and
ports all drivers to use that function. Also adds tests for the
said function.

pkg: add a function to generate chain names

Adds a function to generate chain names for use in iptables and
ports all drivers to use that function. Also adds tests for the
said function.

Merge pull request #171 from steveeJ/bump-cni

vendoring: bump coreos/go-iptables to 0.1.0

vendoring: bump coreos/go-iptables to 0.1.0

Merge pull request #170 from jonboulle/master

README: add Calico and Weave as CNI users

README: add Calico and Weave as CNI users

Merge pull request #169 from philips/add-users

README: add a list of CNI users

README: add a list of CNI users

Merge pull request #165 from dcbw/go15-vendor

scripts: fix build with go-1.5 after switch to vendor/

scripts: fix build with go-1.5 after switch to vendor/

go-1.6 enables vendor by default, but go-1.5 needs an environment variable.

Merge pull request #158 from steveeJ/loadargs

Add generic IgnoreUnknown to valid CNI_ARGS

pkg/types: add tests for args

pkg/types: add tests for args

pkg/types: add IgnoreUnknown arg and logic

This commit adds a struct type CommonArgs that is to be embedded in
every plugin's argument struct. It contains a field named
"IgnoreUnknown" which will be parsed as a boolean and can be provided to
ignore unknown arguments passed to the plugin.

Godeps: bump ginkgo for table extensions

pkg/types: add IgnoreUnknown arg and logic

This commit adds a struct type CommonArgs that is to be embedded in
every plugin's argument struct. It contains a field named
"IgnoreUnknown" which will be parsed as a boolean and can be provided to
ignore unknown arguments passed to the plugin.

plugins/host-local: only call LoadArgs() once

Merge pull request #162 from zachgersh/remove-workspace

build script: remove the _workspace from GOPATH

build script: remove the _workspace from GOPATH

Now that we no longer use godeps to manage
dependencies we can safely remove this.
Dependencies in the vendor directory will
automagically be picked up.

Merge pull request #161 from steveeJ/skel-tests

pkg/skel: add rudimentary unit tests

pkg/skel: add rudimentary unit tests

This is an attempt to testing the PluginMain() function of the skel pkg.
We should be able to do better by using a mockable interface for the
plugins, but this is a start.

pkg/skel: add rudimentary unit tests

This is an attempt to testing the PluginMain() function of the skel pkg.
We should be able to do better by using a mockable interface for the
plugins, but this is a start.

Merge pull request #156 from zachgersh/vendor

Moves to go team vendoring solution

Merge pull request #157 from achanda/vet

Fix govet warnings

Let go tip fail, add vendor flag for 1.5

Fix govet warnings

Moves to official vendoring solution

Merge pull request #154 from rosenhouse/withnetns-errors

WithNetNS restores original namespace when callback errors

Update docstring on WithNetNS

Update docstring on WithNetNS

WithNetNS restores original namespace when callback errors

- adds test coverage of WithNetNS in BDD-style

WithNetNS restores original namespace when callback errors

- adds test coverage of WithNetNS in BDD-style

Merge pull request #146 from achanda/trusty

Use Ubuntu Trusty in travis

Merge pull request #153 from steveeJ/maintainer-add-zach

MAINTAINERS: welcome Zach!

MAINTAINERS: welcome Zach!

We are pleased to welcome Zach on board as CNI maintainer.
Happy collaboration!

Merge pull request #148 from steveeJ/coverage-batches

travis & README: add coveralls coverage report

travis & README: add coveralls coverage report

Also, don't use sudo for the entire test script

Merge pull request #144 from squaremo/rename-delegate-execs

Add invoke.Delegate{Add,Del} for use by meta-plugins

Add invoke.Delegate{Add,Del} for use by meta-plugins

The 'flannel' meta plugin delegates to other plugins to do the actual
OS-level work. It used the ipam.Exec{Add,Del} procedures for this
delegation, since those do precisely what's needed.

However this is a bit misleading, since the flannel plugin _isn't_
doing this for IPAM, and the ipam.Exec* procedures aren't doing
something specific to IPAM plugins.

So: anticipating that there may be more meta plugins that want to
delegate in the same way, this commit moves generic delegation
procedures to `pkg/invoke`, and makes the `pkg/ipam` procedures (still
used, accurately, in the non-meta plugins) shims.

Add invoke.Delegate{Add,Del} for use by meta-plugins

The 'flannel' meta plugin delegates to other plugins to do the actual
OS-level work. It used the ipam.Exec{Add,Del} procedures for this
delegation, since those do precisely what's needed.

However this is a bit misleading, since the flannel plugin _isn't_
doing this for IPAM, and the ipam.Exec* procedures aren't doing
something specific to IPAM plugins.

So: anticipating that there may be more meta plugins that want to
delegate in the same way, this commit moves generic delegation
procedures to `pkg/invoke`, and makes the `pkg/ipam` procedures (still
used, accurately, in the non-meta plugins) shims.

Merge pull request #142 from steveeJ/fix-loopback-result

Fix loopback result

Use Ubuntu Trusty in travis

This boots faster

Merge pull request #143 from zachgersh/more-tests

More tests

tests: loopback DEL

Adds travis badge, now we have more tests

Make sure invoke tests are running

tests: loopback stdout must be JSON

plugins/loopback: take lo down on CmdDel

plugins/loopback: return empty result

This is needed to conform to the specification and allow successful
unmarshalling in the invoker.

Merge pull request #120 from zachgersh/find-better-error

Better error messages when plugin is not found

Merge pull request #120 from zachgersh/find-better-error

Better error messages when plugin is not found

Merge pull request #121 from zachgersh/loopback-plugin

Loopback plugin

Handle namespaces with care

- After creating new netns, switch back to main netns
- Lock thread during test and test setup

Create a plugin for up'ing a lo device

- Believe we need sudo to create netns
- Use syscall instead of relying on ip netns
- Add sudo to .travis.yml
- Needs more -E
- Revert Godeps GoVersion to 1.4.2
- in travis, test command is run with all necessary env vars
- Loopback plugin only works on 'lo' interface
- Update README, add loopback plugin config
- note script dependency on jq

Signed-off-by: Gabe Rosenhouse <grosenhouse@pivotal.io>

Adds ginkgo / gomega as a dependency

Merge pull request #135 from achanda/travis

Tweak the travis file

Tweak the travis file

- Build against more versions of Go
- Explicitely specify go versions

Merge pull request #132 from steveeJ/scripts

scripts: add DEBUG option

Merge pull request #117 from kinvolk/alban/net-tuning

new plugin: tuning