Merge pull request #84 from rosenhouse/add-windows-ci

Enable Windows CI (Appveyor)

Enable Windows CI (Appveyor)

- start list of linux_only plugins; ignore them when testing on Windows
- Isolate linux-only code by filename suffix
- Remove stub (NotImplemented) functions
- other misc. fixes for Windows compatibility

Merge pull request #89 from dcbw/del-link-by-name-addr-fix

pkg/ip: don't return error from DelLinkByNameAddr() if no addresses exist

pkg/ip: don't return error from DelLinkByNameAddr() if no addresses exist

For some reason no addresses on the interface returned an error, despite
having a testcase that explicitly tested for success.

Merge pull request #65 from rosenhouse/golang-to-1.9

Golang versions: add 1.9, drop 1.7

Vagrantfile: update to golang 1.9.1

Merge pull request #78 from rmohr/dhcp

Don't let DHCP IPAM plugin fail on missing lease

Don't let DHCP delete fail on missing lease

There are at least two reasons why a lease is not present:

 * The dhcp ipam daemon was restarted
 * On add the IPAM plugin failed

Don't fail the IPAM invocation when the lease is not present, to allow
proper device cleanup on CNI delete invocations.

Merge pull request #74 from rosenhouse/host-device-fixes

Host-device fixes

host-device plugin: result is valid JSON

test:
- feed valid config JSON to plugin
- execute plugin inside the namespace with the test device

host-device: do not swallow netlink errors

Fix `go get github.com/containernetworking/plugins`.

Signed-off-by: Lantao Liu <lantaol@google.com>

Merge pull request #3 from trusch/master

added host-device plugin which adds a specified link to container

Merge pull request #72 from rosenhouse/move-echosvr

testing: move echosvr into testutils

testing: move echosvr into testutils

Merge pull request #63 from squeed/v6-fixes

Fix ipmasq teardown on v6-only interfaces

Merge pull request #71 from rosenhouse/fix-portmap-integ-test

portmap integration test: echo server runs in separate process

pkg/ip: Fix ipmasq teardown on v6-only interfaces

portmap integration test: echo server runs in separate process

this way we're not mixing goroutines and namespaces

Merge pull request #64 from rosenhouse/travis-ginkgo

travis: run with ginkgo -p instead of go test

travis: run with ginkgo -p instead of go test

may help reduce test-pollution due to namespace-affinity

see http://onsi.github.io/ginkgo/#parallel-specs

Merge pull request #67 from rosenhouse/test-all-packages

Test all packages

Test all non-vendored packages

Merge pull request #66 from rosenhouse/filelock-vet

host-local FileLock is used by value instead of by reference

test.sh: cover host-local disk backend for go test and go vet

host-local disk backend store uses FileLock by reference

- this change fixes go vet warnings for the package

Merge pull request #62 from squeed/bridge-fixes

bridge: various fixes

Golang versions: add 1.9, drop 1.7

bridge: various fixes

* Don't set the MAC, send gratuitous arp instead
* Set the bridge's MAC to itself
* Only disable DAD when necessary

Merge pull request #54 from squeed/clean-build

Release: clean the builddir when building

Release: clean the builddir when building

Merge pull request #50 from squeed/host-local-multi

ipam/host-local: support sets of disjoint ranges

ipam/host-local: support sets of disjoint ranges

In real-world address allocations, disjoint address ranges are common.
Therefore, the host-local allocator should support them.

This change still allows for multiple IPs in a single configuration, but
also allows for a "set of subnets."

Fixes: #45

Merge pull request #47 from dcbw/tuning-prev-result

tuning: pass prevResult through

tuning: pass prevResult through

Merge pull request #44 from verizonlabs/build-on-osx

If we're on OS X then have Go cross compile for us

If we're on OS X then have Go cross compile for us

Merge pull request #35 from dnardo/bridge-plugin

bridge: add support for promiscuous mode

Merge pull request #40 from squeed/portmap-timeout

plugins/portmap: fix test flake

Merge pull request #38 from tomdee/release-process

Add release process

Add release process

Copied from containernetworking/cni

The only difference is that the release archives also include the the
word "plugin"

plugins/portmap: fix test flake

The source address selection was random, and sometimes we picked a
source address that the container didn't have a route to. Adding a
default route fixes that!

Merge pull request #37 from containernetworking/travis-badge

README.md: Add travis badge

Merge pull request #39 from dcbw/ptp-fix-test

ptp: fix testcase for Interface int-pointer changes

Merge pull request #36 from tomdee/vendor-libcni-v0.6.0-rc1

Update vendor to v0.6.0-rc1

ptp: fix testcase for Interface int-pointer changes

README.md: Add travis badge

Update vendor to v0.6.0-rc1

bridge: add support for promiscuous mode

Merge pull request #25 from squeed/ptp-v6

ptp: add ipv6 support

Merge pull request #34 from dnardo/netlink-update

vendor: Update vishvanana/netlink dependency.

Merge pull request #28 from dcbw/update-for-interface-change

pkg,plugins: update for Result struct Interface index changes

vendor: Update vishvanana/netlink dependency.

This updates the netlink dependency to include recent updates, including a fix when setting prosmic mode on a bridge and additions for creating qdisc/classes/filters. This is necessary for some upcoming additions to CNI

ptp: add ipv6 support

* Wait for addresses to leave tentative state before setting routes
* Enable forwarding correctly
* Set up masquerading according to the active protocol

Merge pull request #33 from squeed/bump-iptables

vendor: bump go-iptables to 0.2.0

pkg,plugins: update for Result struct Interface index changes

It's a pointer now, so we need to use the helper function to set
the field and also test for nil before accessing it.

vendor: bump go-iptables to 0.2.0

host-device: cleanup + completed tests;

vendor: bump cni to 0.5.2-git49d814c

Merge pull request #24 from squeed/flannel-return

flannel: set the cniVersion on the delegated config

Merge pull request #10 from leblancd/v6_bridge_plugin

bridge: Add support for IPv6 to bridge plugin

bridge: Add support for IPv6 to bridge plugin

This change adds support for IPv6 container/pod addresses to the CNI
bridge plugin, both for dual-stack (IPv4 + IPv6) and for IPv6-only
network configurations.

The proposed changes support multiple IPv6 addresses on a container
interface. If isGW is configured, the bridge will also be configured with
gateway addresses for each IPv6 subnet.

Please note that both the dual-stack functionality and support for multiple
IPv6 container/gateway addresses depends upon containernetworking/cni
PR 451 "ipam/host-local: support multiple IP ranges".
This change could potentially be committed independently from this host-local
plugin change, however the dual-stack and multiple IPv6 address
functionality that is enabled by this change can't be exercised/tested
until the host-local plugin change is committed.

There are some IPv6 unit test cases that are currently commented out
in the proposed changes because these test cases will fail without the
prior commits of the multiple IP range host-local change.

This pull request includes a temporary workaround for Kubernetes
Issue #32291 (Container IPv6 address is marked as duplicate, or dadfailed).
The problem is that kubelet enables hairpin mode on bridge veth
interfaces. Hairpin mode causes the container/pod to see echos of its
IPv6 neighbor solicitation packets, so that it declares duplicate address
detection (DAD) failure. The long-term fix is to use enhanced-DAD
when that feature is readily available in kernels. The short-term fix is
to disable IPv6 DAD in the container. Unfortunately, this has to be done
unconditionally (i.e. without a check for whether hairpin mode is enabled)
because hairpin mode is turned on by kubelet after the CNI bridge plugin
has completed cmdAdd processing. Disabling DAD should be okay if
IPv6 addresses are guaranteed to be unique (which is the case for
host-local IPAM plugin).

flannel: set the cniVersion on the delegated config

Fixes: #23

Merge pull request #16 from squeed/gratuitous-arp

 ptp, macvlan: don't change mac address; send gratuitous arp

Merge pull request #22 from squeed/portmap-plugin

portmap: Don't accept a port number of 0

Merge pull request #11 from dcbw/dhcp-pidfile

dhcp: add --pidfile option to DHCP client daemon

portmap: Don't accept a port number of 0

This also adds more testing around configuration parsing.

Merge pull request #12 from squeed/host-local-multi

ipam/host-local: support multiple IP ranges

dhcp: add --pidfile option to DHCP client daemon

Merge pull request #14 from rosenhouse/more-ns-warning-redux

pkg/ns readme: expand on danger of ns switching in long-lived programs

ipam/host-local: support multiple IP ranges

This change allows the host-local allocator to allocate multiple IPs.
This is intended to enable dual-stack, but is not limited to only two
subnets or separate address families.

Merge pull request #17 from tklauser/x-sys-unix-setns

pkg/ns: use unix.Setns() instead of open coding it

pkg/ns: use unix.Setns() instead of open coding it

Use unix.Setns() instead of manually re-implementing it using
unix.Syscall().

Merge pull request #15 from rosenhouse/contribute-vagrantfile

Add CONTRIBUTING and Vagrantfile

ptp, macvlan: don't change mac address; send gratuitous arp

vendor: add j-keck/arping

add Vagrantfile to support cross-platform development

Update CONTRIBUTING to reflect plugin split

contributing doc: copy in from cni repo

pkg/ns readme: expand on danger of ns switching in long-lived programs

- redux of https://github.com/containernetworking/cni/pull/331 since the
plugin/lib split
- added link to new blog post about this from Weave folks (thanks!)

Merge pull request #13 from squeed/build-fixes

build: some minor post-repo-split fixes; add fastbuild

build: some minor post-repo-split fixes; add fastbuild

Merge pull request #1 from squeed/portmap-plugin

iptables-based portmapping plugin

plugins/meta/portmap: add an iptables-based host port mapping plugin

vendor: add mattn/go-shellwords

vendor: add libcni

vendor: update coreos/go-iptables

Merge pull request #9 from squeed/move-pkg

Move libraries from cni only used by plugins.

vendor: remove moved cni/pkg/... libraries

move most of cni/pkg to plugins/pkg: delete code staying in cni/pkg

This moves the following packages to this repository:
* ip
* ipam
* ns
* testutils
* utils

Add 'pkg/' from commit 'a11be4d7596203874b742b6597caf255204c56c3'

git-subtree-dir: pkg
git-subtree-mainline: dcf7368eeab15e2affc6256f0bb1e84dd46a34de
git-subtree-split: a11be4d7596203874b742b6597caf255204c56c3

Merge pull request #6 from squeed/merge-docs

Documentation folder merge

Merge pull request #8 from squeed/bump-cni

vendor: bump cni to v0.5.2

vendor: bump cni to v0.5.2

Add references to all plugins to the README

Move documentation folder to individual READMEs

Add 'Documentation/' from commit '5bb59a12b80dc99552bebf0394a067807f72f947'

git-subtree-dir: Documentation
git-subtree-mainline: db8f94a5096c356107873455525f5fa3a02561d5
git-subtree-split: 5bb59a12b80dc99552bebf0394a067807f72f947

Rename build & test, take two

Rename build script to avoid conflict with bazel

Merge pull request #5 from dcbw/add-cni-plugins

plugins: add plugins from containernetworking/cni

plugins: add plugins from containernetworking/cni

Plugins prepared from the containernetworking/cni repo as follows:

1) git reset --hard 1a9288c3c09cea4e580fdb1a636f1c5e185a391f
2) git remove everything not in plugins/
3) git remove plugins/test
4) git merge into containernetworking/plugins repo
5) adjust import paths for containernetworking/cni -> containernetworking/plugins