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

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

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