Merge pull request #445 from feiskyer/cniversion

Add cniVersion to Result

Merge pull request #447 from gunjan5/invalid-spec-json

Fix invalid json

fix invalid json

Fix testings accross the project

Signed-off-by: Pengfei Ni <feiskyer@gmail.com>

Add cniVersion to Result

Signed-off-by: Pengfei Ni <feiskyer@gmail.com>

Merge pull request #433 from dunjut/master

scripts: cleanup() should rm net container in docker-run.sh

Merge pull request #440 from feiskyer/cap

cnitool: support capablity args

Merge pull request #439 from dnardo/netlink-update

vendor: Update vishvanana/netlink dependency.

cnitool: support capablity args

Signed-off-by: Pengfei Ni <feiskyer@gmail.com>

Merge pull request #391 from tangle329/master

Validate rangeStart and rangeEnd specified in conf

Merge pull request #435 from micahhausler/patch-1

Fixed table formatting

Update netlink dependency.

Fixed table formatting

scripts: cleanup() should rm net container in docker-run.sh

Merge pull request #432 from dunjut/master

README: build the plugins using ./build.sh, not ./build

README: build the plugins using ./build.sh, not ./build

Merge pull request #430 from kris-nova/remove-proposal

Wondering if this is still a proposal?

Changing Proposal->Specification in Spec

Merge pull request #419 from asridharan/cnitool

Added documentation for `cnitool`

Documentation: Added documentation for `cnitool`.

Added documentation to simplify the usage of `cnitool` and the
settings expected to be used with `cnitool`.

libcni: Fixed tests that were checking error strings.

Checking error strings makes these tests flaky, especially if the
error string is changed in libcni. Have gone ahead an introduced a new
error type `NoConfigsFoundError` and the Match is against the error
type making it more deterministic.

libcni: Improved error messages.

Improved the error reporting to be more descriptive when configuration
files are not filed while using the cni-tool.

Merge pull request #413 from dcbw/spec-fix-ips

spec/plugins: fix 'ip'->'ips' in the spec, bump to 0.3.1

spec/plugins: fix 'ip'->'ips' in the spec, bump to 0.3.1

Merge pull request #423 from rosenhouse/update-travis

travis: shift forward to Go 1.8 and 1.7

Merge pull request #421 from kshafiee/patch-1

Added entry for CNI-Genie

travis: shift forward to Go 1.8 and 1.7

Merge pull request #400 from lstoll/lstoll-rename-build-script

Rename build script to avoid conflict with bazel

Added entry for CNI-Genie

CNI-Genie enables orchestrators (kubernetes, mesos) for seamless connectivity to choice of CNI plugins (calico, canal, romana, weave) configured on a Node

Update community sync detail

Merge pull request #414 from kad/s390x

Enable s390x build

Enable s390x build

Merge pull request #412 from tomdee/safe-del

plugins/*: Don't error if the device doesn't exist

plugins/*: Don't error if the device doesn't exist

I wasn't able to test or update the dhcp plugin but from a code read it
should be fine. All the other plugins are tested and fixed

Merge pull request #407 from aaronlevy/del-err

plugins/meta/flannel: If net config is missing do not return err on DEL

plugins/meta/flannel: If net config is missing do not return err on DEL

Merge pull request #396 from cf-container-networking/vendored-types-leakage

pkg/ip: do not leak types from vendored netlink package

Merge pull request #401 from chrismarino/patch-1

Added Romana to list of CNI providers...

Added Romana to list of CNI providers...

Added Romana to list of CNI providers.

Rename build script to avoid conflict with bazel

It's difficult to include this repository using bazel, because
the file named "build" conflicts with new_go_repository generation
on case-insensitive filesystems (ref
https://github.com/bazelbuild/rules_go/issues/234). This change
renames the file to something that doesn't conflict, and also
renames the test script for consistency.

pkg/ip: improve docstring for SetupVeth

pkg/ip: SetupVeth returns net.Interface

Validate rangeStart and rangeEnd specified in conf

Signed-off-by: Tang Le <tangle3@wanda.cn>

pkg/ip: do not leak types from vendored netlink package

The exported function SetupVeth now returns a package-defined type.

Signed-off-by: Gabe Rosenhouse <grosenhouse@pivotal.io>

Merge pull request #393 from containernetworking/community-sync-readme

readme.md: Add link to community sync

readme.md: Add link to community sync

Merge pull request #366 from ehazlett/ipam-host-local-initial-reserved-ip

Do not error if last reserved not found after initial creation

fix unrelated failing tests

Signed-off-by: Evan Hazlett <ejhazlett@gmail.com>

add test for ensuring initial subnet creation does not contain an error

Signed-off-by: Evan Hazlett <ejhazlett@gmail.com>

do not error if last_reserved_ip is missing for host local ipam

Signed-off-by: Evan Hazlett <ejhazlett@gmail.com>

Merge pull request #387 from tangle329/master

Check n.IPAM before use it in LoadIPAMConfig function

Merge pull request #373 from dcbw/conflist-runtime-config

spec,libcni: add support for injecting runtimeConfig into plugin stdin data

Check n.IPAM before use it in LoadIPAMConfig function

Signed-off-by: Tang Le <tangle3@wanda.cn>

spec,libcni: add support for injecting runtimeConfig into plugin stdin data

Add a new CapabilityArgs member to the RuntimeConf struct which runtimes can
use to pass arbitrary capability-based keys to the plugin.  Elements of this
member will be filtered against the plugin's advertised capabilities (from
its config JSON) and then added to a new "runtimeConfig" top-level map added
to the config JSON sent to the plugin on stdin.

Also "runtime_config"->"runtimeConfig" in CONVENTIONS.md to make
capitalization consistent with other CNI config keys like "cniVersion".

Merge pull request #382 from rosenhouse/doc-spec-upgrade

Documentation: guidance for upgrading to CNI Spec v0.3.0

Minor rewording about default config version

Merge pull request #383 from dcbw/test-fixes

tests: misc test fixes

Merge pull request #374 from squeed/config-list-upconvert

libcni: up-convert a Config to a ConfigList when no other configs are found.

Merge pull request #377 from squeed/ptp-sandbox

plugins/main/ptp: set the Sandbox property on the response

Merge pull request #380 from bboreham/patch-1

Add Bryan Boreham as maintainer

pkg/types: misc current types testcase cleanups

pkg/testutils: return errors after restoring stdout

Ensures Ginkgo is able to print detailed failure messages instead
of them being captured by the pipe.

Merge pull request #375 from aaithal/complieOnNonLinuxPlatforms

pkg/ns: refactored so that builds succeed on non-linux platforms

docs: table formatting is hard

docs: fill-out and correct version conversion table

docs: minor improvements to spec-upgrades

docs: add small upgrade instructions

docs: minor improvements to 0.3.0 upgrade guidance

docs: Edits to v0.3.0 upgrade guidance

docs: CNI versioning for 0.3.0 upgrade

libcni: up-convert a Config to a ConfigList when no other configs are found.

Merge pull request #379 from Intel-Corp/master

README: List multus as 3rd party plugin

vendor: Update vishvanana/netlink dependency

pkg/ip: refactored so that builds succeed on non-linux platforms

moved functions that depend on linux packages into a separate file and added
nop methods with a build tag for non-linux platforms in a new file.

pkg/ns, pkg/types: refactored non linux build fix code to

Make GetCurrentNS platform specific instead of getCurrentThreadNetNSPath

Merge pull request #371 from rosenhouse/spec-netconf-routes

spec: Remove `routes` from Network Configuration

Replace Michael Bridgen with Bryan Boreham

README: List multus as 3rd party plugin

refer #343 issue

Merge pull request #376 from zackhsi/fix-grammar

Fix grammar

plugins/main/ptp: set the Sandbox property on the response

Fix grammar

pkg/ns: refactored so that builds succeed on non-linux platforms

moved functions that depend on linux packages (sys/unix) into a separate file
and added nop methods with a build tag for non-linux platforms in a new file.

Merge pull request #372 from rosenhouse/consolidate-host-local-docs

docs: consolidate host-local documentation

docs: consolidate host-local documentation

- remove the README.md in the plugins/ipam/host-local (no other plugin
  has this)
- merge that info into the main Documentation/host-local.md

spec: Remove `routes` from Network Configuration

- this is supported by some IPAM plugins but is not required of all of
  them

Merge pull request #369 from containernetworking/port-mapping

CONVENTIONS.md: Update details on port-mappings

More markups

Update with feedback

CONVENTIONS.md: Update details on port-mappings

After discussion with the CNI maintainers and representatives from Mesos and Kubernetes we've settled on this approach for passing the dynamic port mapping information from runtimes to plugins.

Including it in the plugin config allows "operators" to signal that they expect port mapping information to be passed to a specific plugin (or plugins). This follows the model that config passed in the plugin config should be acted upon by a plugin.

Runtimes can optionally return an error to users if they find no plugin in the chain that supports port mappings.

I'm sure we'd all like to see this merged soon so I'll try to turn any feedback quickly (and I'm happy to get any and all feedback - spellings, unclear working, wrong JSON terminology etc..)

Merge pull request #361 from ofiliz/issue-360

invoke: Enable plugin file names with extensions

invoke: Enable plugin file names with extensions

A CNI network configuration file contains the plugin's executable file name.
Some platforms like Windows require a file name extension for executables.
This causes unnecessary burden on admins as they now have to maintain two
versions of each type of netconfig file, which differ only by the ".exe"
extension. A much simpler design is for invoke package to also look for
well-known extensions on platforms that require it. Existing tests are
improved and new tests are added to cover the new behavior.

Fixes #360

Merge pull request #364 from pires/build_tag

pkg/utils/sysctl/sysctl_linux.go: couple fixes

pkg/utils/sysctl/sysctl_linux.go: fix typo.

pkg/utils/sysctl/sysctl_linux.go: fix build tag.

Merge pull request #359 from ofiliz/issue-355

libcni, pkg\invoke: Use OS-specific list separator when parsing CNI_PATH

spec, libcni, pkg/invoke: Use OS-agnostic separator when parsing CNI_PATH

Hardcoding the list separator character as ":" causes CNI to fail when parsing
CNI_PATH on other operating systems. For example, Windows uses ";" as list
separator because ":" can legally appear in paths such as "C:\path\to\file".
This change replaces use of ":" with OS-agnostic APIs or os.PathListSeparator.

Fixes #358

Merge pull request #145 from dcbw/pass-ifname-back

Pass interface name and MAC back to runtime

spec/plugins: return interface details and multiple IP addresses to runtime

Updates the spec and plugins to return an array of interfaces and IP details
to the runtime including:

- interface names and MAC addresses configured by the plugin
- whether the interfaces are sandboxed (container/VM) or host (bridge, veth, etc)
- multiple IP addresses configured by IPAM and which interface they
have been assigned to

Returning interface details is useful for runtimes, as well as allowing
more flexible chaining of CNI plugins themselves.  For example, some
meta plugins may need to know the host-side interface to be able to
apply firewall or traffic shaping rules to the container.

pkg/ipam: add testcases

plugins/flannel: organize test JSON alphabetically

Otherwise the test fails, since Go's JSON marshaller prints
dict items alphabetically in its String() call.