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 #178 from steveeJ/master

travis: don't go get vet

travis: don't go get vet

Merge pull request #173 from jonboulle/master

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

*: add comment to iptables rules for ipmasq

pkg/utils: remove unneeded condition in tests

pkg/utils: fix docstring

pkg/utils: split and unexport constants

pkg/utils: fix docstring

pkg/utils: use constant for chain prefix

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.

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

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.

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

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.

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 #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

scripts: add DEBUG option

Merge pull request #107 from aanm/pretty-result

added the String method to Result type

added the String method to Result type

Signed-off-by: André Martins <aanm90@gmail.com>

new plugin: tuning

Allow users to tune net network parameters such as somaxconn.

With this patch, users can add a new network configuration:

> {
>   "name": "mytuning",
>   "type": "tuning",
>   "sysctl": {
>           "net.core.somaxconn": "500"
>   }
> }

The value /proc/sys/net/core/somaxconn will be set to 500 in the network
namespace but will remain unchanged on the host.

Only sysctl parameters that belong to the network subsystem can be
modified.

Related to: https://github.com/coreos/rkt/pull/2140

Merge pull request #108 from aanm/fix-ip-json-tag

fixed ipam host-local IP json tag

Better error message when plugin cannot be found

Merge pull request #119 from jonboulle/master

MAINTAINERS: remove Eugene from list

MAINTAINERS: remove Eugene from list

Eugene is no longer with CoreOS or actively involved with CNI, so remove
him from the current list of maintainers. He'll be gladly welcomed back
if he decides to rejoin the project.

fixed ipam host-local IP json tag

Signed-off-by: André Martins <aanm90@gmail.com>

Merge pull request #105 from steveeJ/scripts

script/priv-net-run: improve shebang compatibility

script/priv-net-run: improve shebang compatibility

Merge pull request #103 from steveeJ/dns

*: DNS information as dictionary, adding domain, search domains and options

*: reflect SPEC's DNS changes in implementation

* DNS is now a type which will result in a JSON dictionary in
  configurations and results
* Minor refactoring, making use of type embedding

docs/ptp: update example and DNS description

SPEC: DNS information as dictionary, adding domain, search domains, options

Merge pull request #104 from dcbw/revendor-go-iptables

Revendor go-iptables to get --wait behavior

Revendor go-iptables to get --wait behavior

Merge pull request #102 from steveeJ/scripts

scripts/exec-plugins: print plugin results if DEBUG>0

scripts/priv-net-run: print plugin results if DEBUG>0

Merge pull request #101 from steveeJ/shebangs

scripts: improve shebang compatibility

scripts: improve shebang compatibility

Merge pull request #99 from kinvolk/alban/dns-config

*: add "dns" field to the configuration

*: add "dns" field to the configuration

appc/cni#76 added a "dns" field in the result JSON. But before this
patch, the plugins had no way of knowing which name server to return.

There could be two ways of knowing which name server to return:
1. add it as an extra argument ("CNI_ARGS")
2. add it in the network configuration as a convenience (received via
   stdin)

I chose the second way because it is easier. In the case of rkt, it
means the user could just add the DNS name servers in
/etc/rkt/net.d/mynetwork.conf.

Merge pull request #98 from apcera/remove-addif

spec: Removed reference to unsupported addIf attribute

spec: Removed reference to unsupported addIf attribute

Removes the "addIf" attribute from the example of a bridge network
definition. This doesn't appear to exist in the implementation.

Merge pull request #92 from ChengTiesheng/patch-1

Update README.md

Update README.md

Highlight bash/shell code in markdown for readability.

Signed-off-by: chengtiesheng <chengtiesheng@huawei.com>

Merge pull request #91 from tomdee/patch-1

Fix docker example in README.md