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

Update README.md

No need for full path to ifconfig (and the given path doesn't work...)

Merge pull request #90 from dgonyeo/master

pkg/ip: fix typo in error message, s/iptabes/iptables/

pkg/ip: fix typo in error message, s/iptabes/iptables/

Merge pull request #80 from eyakubovich/add-version

spec: add cniVersion to results and config

spec: add cniVersion to results and config

Version all artifacts using semantic versioning
so that plugins and container runtimes coded against
different versions of the spec can interoperate.

Fixes #44

Merge pull request #78 from steveeJ/improve-errormsg-missing-subnet

host-local: detect missing subnet and error out

host-local: detect missing subnet and error out

Merge pull request #82 from steveeJ/dhcp-linkfix

dhcp: detect if link is down and attempt to set up

dhcp: detect if link is down and attempt to set up

With this we still get the "network is down" error, but after the 2nd
retry the interface is up and the dhcp request works.

Merge pull request #76 from eyakubovich/dns

spec: add "dns" field to the result JSON

spec: add "dns" field to the result JSON

This adds an ability for networks to communicate
about the DNS nameservers they know about to the
container runtime.

Fixes #67

Merge pull request #73 from eyakubovich/flannel-fix

flannel: doc fix and check for missing subnet.env fields

flannel: doc fix and check for missing subnet.env fields

- Document that flannel outputs FLANNEL_NETWORK b/c we use it
- Check and error out if expecited fields are missing in subnetEnv file

Fixes #72

Merge pull request #71 from jonboulle/master

README/CONTRIBUTING: mention cni-dev@ list

README/CONTRIBUTING: mention cni-dev@ list

Merge pull request #69 from eyakubovich/err-msgs

Route del would fail due to wrong scope; better errors

Merge pull request #70 from eyakubovich/copyright

Change copyright from CoreOS to CNI authors

Change copyright from CoreOS to CNI authors

CNI is developed by more than just CoreOS, Inc
and the copyright is retained by all CNI
contributors (for their respective contributed code).

Route del would fail due to wrong scope; better errors

In some cases the route deletion would fail due to wrong
scope. It should be NOWHERE when deleting (per iproute2).
This also adds more verbose error messages.

Merge pull request #65 from steveeJ/fix-veth-setup

Fix veth setup

Fixes #61 #64

link: switch to host netns to set up host veth end

Revert "plugins/ptp: allow host veth to be UP"

This reverts commit 231d2d5a27f3ba54219c3f0b1c8ef2c5dab4faaf.

Merge pull request #63 from steveeJ/no-host-local-ptp

Merge host-local and host-local-ptp

plugins/ptp: allow host veth to be UP

update ptp docs to reflect changes in plugin

remove host-local-ptp plugin

Use single IP on the host for ptp veths

Instead of allocating a /31 for each container,
use the same IP on the host side for all veths.
This is very similar how real point-to-point
devices work (using donor IPs).

revendoring netlink

Merge pull request #60 from eyakubovich/fix-plugin-del

bug fix: exec of DEL cmd caused JSON decode error

bug fix: exec of DEL cmd caused JSON decode error

When plugin is executed with a DEL command, it does not
print result to stdout unless there is an error. Therefore
it stdout bytes should not be passed to json.Unmarshal.