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.

Merge pull request #59 from squaremo/factor-out-libcni

Factor an API out into a module

Factor an API out into a module

This takes some of the machinery from CNI and from the rkt networking
code, and turns it into a library that can be linked into go apps.

Included is an example command-line application that uses the library,
called `cnitool`.

Other headline changes:

 * Plugin exec'ing is factored out

The motivation here is to factor out the protocol for invoking
plugins. To that end, a generalisation of the code from api.go and
pkg/plugin/ipam.go goes into pkg/invoke/exec.go.

 * Move argument-handling and conf-loading into public API

The fact that the arguments get turned into an environment for the
plugin is incidental to the API; so, provide a way of supplying them
as a struct or saying "just use the same arguments as I got" (the
latter is for IPAM plugins).

Merge pull request #58 from steveeJ/remove-ipmasq-ifneedbe

ipvlan/macvlan: remove ipMasq related code

ipvlan/macvlan: remove ipMasq related code

Luckily the docs haven't mentioned support for ipMasq for both plugins so far.
Even if anyone has attempted to enable the feature in their configuration files it didn't have the desired effect for the network.

Merge pull request #57 from eyakubovich/maintainers

Add MAINTAINERS file

Add MAINTAINERS file

Merge pull request #53 from NeilW/master

IPAM plugin: improve error messages

plugin/ipam: correct formatting of error message

IPAM plugin: improve error messages

Make it more clear that we failed to find an IPAM plugin.
Check for a missing plugin name and issue a more helpful error.

Merge pull request #55 from jonboulle/travis

*: add basic test script + travis hook

*: add basic test script + travis hook

Adds a simple test script, mostly to perform gofmt and govet checking;
currently tests only exist for the DHCP plugin.

Merge pull request #54 from jonboulle/master

plugin/ipam: fix typo in error message

plugin/ipam: fix typo in error message

Merge pull request #51 from steveeJ/plugins-args

CNI_ARGS: use ';' to split args as documented

CNI_ARGS: use ';' to split args as documented

Merge pull request #42 from steveeJ/plugins-args

host-local: allow ip request via CNI_ARGS

host-local: allow ip request via CNI_ARGS

A specific IP can now be requested via the environment variable CNI_ARGS, e.g.
`CNI_ARGS=ip=1.2.3.4`.
The plugin will try to reserve the specified IP.
If this is not successful the execution will fail.

Merge pull request #49 from eyakubovich/dco

Add DCO and CONTRIBUTING.md

Add DCO and CONTRIBUTING.md

Merge pull request #43 from alexwlchan/master

Fix a few spelling mistakes in the docs

Fix a few spelling mistakes in the docs

Merge pull request #41 from eyakubovich/flannel-route

flannel: set up route for the entire flannel network

flannel: set up route for the entire flannel network

Without it, packets leave via default route iface
and come back via interface added by flannel plugin.
If rp_filter=1, the packets are dropped.

Merge pull request #37 from steveeJ/fix-docs

host-local: fix routes in README examples

host-local: fix routes in README examples

Merge pull request #34 from eyakubovich/document-plugins

Document the plugins

Merge pull request #33 from eyakubovich/veth-to-ptp-rename

Rename veth plugin to ptp and add docs.

Rename veth plugin to ptp and add docs.

ptp (point-to-point) better describes the nature of the plugin
as veth pairs are also used by the bridge plugin.

Document the plugins

Adds a .md file to the Documentation/ folder for each plugin.

Fixes #16

Merge pull request #35 from eyakubovich/ip_forward

enable net.ipv4.ip_forward in plugins that need it

Merge pull request #29 from eyakubovich/spec-fmt-fixes

spec: fix fmt issues and example JSON

Merge pull request #32 from eyakubovich/ipvlan-fix

ipvlan: wrong name was used during link creation

enable net.ipv4.ip_forward in plugins that need it

ipvlan: wrong name was used during link creation

Instead of temp (random) name, the final name (e.g. eth0)
was used during link creation. This would collide on hosts
that already had the an interface with such a name.

Merge pull request #30 from zenlinTechnofreak/master

Fix environment bug,version of golang should be 1.4+, add reminding i…

Fix environment bug,version of golang should be 1.4+, add reminding in Readme.md

spec: fix fmt issues and example JSON

Merge pull request #22 from eyakubovich/netns-err

dhcp: don't swallow err from netns switch

dhcp: don't swallow err from netns switch

Merge pull request #24 from eyakubovich/dhcp-abs-path

dhcp: make netns path absolute prior to RPC

Merge pull request #25 from eyakubovich/dhcp-timeout

dhcp: decrease read timeout and clamp backoff

dhcp: make netns path absolute prior to RPC

The dhcp daemon may be running with a different cwd
and so the netns paths need to be absolute. This patch
also refactors the code to pull out the common, RPC
parts, out into a separate function.

dhcp: decrease read timeout and clamp backoff

Merge pull request #21 from eyakubovich/no-path-rewrite

No more path rewriting

No more path rewriting

Path rewriting causes too many problems when vendoring
vendored code. When CNI code is vendored into rkt,
godep has problems code already vendored by CNI.

Merge pull request #20 from eyakubovich/flannel

add flannel plugin

add flannel plugin

This introduces a notion of a "meta" plugin. A meta plugin
is one that delegates the actual work of setting up the interface
to the main plugin. The meta plugin is used to select and dynamically
configure the main plugin. The sequence of events, is as follows:

Given netconf like:

    {
"name": "mynet",
"type": "flannel",
"delegate": {
    "type": "bridge"
}
    }

flannel fills in values like "mtu", "ipam.subnet" and delegates to
"bridge" main plugin. "bridge" plugin will operate as usual, calling
into ipam module for IP assignment.

Delegate dictionary should not contain "name" field as it will be
filled in by the flannel plugin.

Merge pull request #19 from eyakubovich/err-handling

Propagate json error object to the caller

Propagate json error object to the caller

When plugin errors out, it prints out a JSON object to stdout
describing the failure. This object needs to be propagated out
through the plugins and to the container runtime. This change
also adds Print method to both the result and error structs
for easy serialization to stdout.

Merge pull request #13 from eyakubovich/dhcp

DHCP plugin

add DHCP IPAM plugin

The plugin binary actually functions in two modes. The first mode
is a regular CNI plugin. The second mode (when stared with "daemon" arg)
runs a DHCP client daemon. When executed as a CNI plugin, it issues
an RPC request to the daemon for actual processing. The daemon is
required since a DHCP lease needs to be maintained by periodically
renewing it. One instance of the daemon can server arbitrary number
of containers/leases.

macvlan and ipvlan: rename tmp iface name prior to calling IPAM

fix bridge to call IPAM after link creation

vendoring sys/unix, go-systemd, dhcp libraries

Merge pull request #12 from eyakubovich/git-ignore

adding .gitignore

adding .gitignore

Merge pull request #9 from eyakubovich/errors

report errors via JSON

report errors via JSON

This reflects the latest SPEC draft.

Merge pull request #11 from eyakubovich/nspath

Do not use netns as ID or for entropy

Do not use netns as ID or for entropy

ContainerID is now required so use that
or generate random bytes.

Fixes #5

Merge pull request #2 from jonboulle/master

*: first cut at SPEC and README

Merge pull request #8 from steveeJ/ipvlan

plugins: add ipvlan main plugin

plugins: add ipvlan main plugin

Add the ipvlan main plugin which is heavily based on the macvlan plugin.
Availabile modes for this plugin are l2 and l3, wheres l2 has the higher
compatibility due to support for multicast and broadcasts. L2 has therefore been
chosen as the default mode.
See the official docs at
https://www.kernel.org/doc/Documentation/networking/ipvlan.txt
for more information.

Fixes #6.