Merge pull request #460 from aaithal/loadArgsPanic

pkg/types: safer typecasting for TextUnmarshaler when loading args

Merge pull request #473 from tomdee/improve-and-document-release

Update and document release process

Update and document release process

Replace the rkt release process with a docker one.
Document the process that we follow for each release.

Merge pull request #477 from dcbw/fix-interface-index-marshaling

types: fix marshalling of omitted "interfaces" key in IPConfig JSON

types: fix marshalling of omitted "interfaces" key in IPConfig JSON

Plugins that don't have knowledge of interfaces, like host-local or
other IPAM plugins, should not set the 'interfaces' key of their
returned "Result" JSON.  This should then not be translated into
an interface index of 0, which it was due to the int marshaling and
omitempty.

Instead, ensure that an omitted 'interface' in JSON ends up being
nil in the IPConfig structure, and that a nil ensures that no 'interfaces'
key is present in the JSON.

Yes, this means that plugins must call the 'current.Int(x)' method
when setting the Interfaces member.  Oh well.

Fixes: https://github.com/containernetworking/cni/issues/404

Merge pull request #478 from squeed/community-sync

Update note about next Community Sync, 2017-06-21

Update note about next Community Sync, 2017-06-21

Merge pull request #474 from rosenhouse/readme-updates

readme updates

Merge pull request #475 from rosenhouse/backfill-test-delegate

pkg/invoke: backfill tests of delegate add & del

pkg/invoke: backfill tests of delegate add & del

readme: update link for Cloud Foundry CNI integration

readme updates

- update some links to the new plugins repo.
- update guidance on golang versions

Merge pull request #406 from GheRivero/cni_error

Return cni.type.Error instead of plain error

Merge pull request #472 from caniszczyk/add-cncf-ref

Add CNCF reference in README

Merge pull request #471 from rosenhouse/clean-up-vendor

Clean up vendor

Add CNCF reference in README

Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Merge pull request #470 from rosenhouse/fixup-vagrantfile

Vagrantfile updates

Merge pull request #467 from containernetworking/clarify-args-release

Clarify first released spec with 'args' was 0.2.0

vendor: cleanup unused dependencies after plugin split

godeps: update Go version to 1.8

Vagrantfile updates

- installs latest Golang

Merge pull request #464 from squeed/remove-pkg

pkg: move packages from cni to plugins

Merge pull request #469 from mcastelino/document_fix

Documentation: Update README.md to reflect plugin split

Merge pull request #461 from ericchiang/fix-env-order

pkg/invoke: ensure that custom env vars are prepended to the env

Documentation: Update README.md to reflect plugin split

Update README.md to call out how to use priv-net-run.sh and
docker-run.sh scripts post plugin split.

Signed-off-by: Manohar Castelino <manohar.r.castelino@intel.com>

pkg/invoke: add unit test for env var ordering in Args.AsEnv

Clarify first released spec with 'args' was 0.2.0

Merge pull request #465 from caniszczyk/add-coc

Add Code of Conduct

Merge pull request #463 from rosenhouse/readme-add-plugin-silk

readme plugins list: add Silk CNI plugin

Add Code of Conduct

CNCF projects are expected to follow a CoC.

Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

pkg: move packages from cni to plugins

readme: add Silk CNI plugin

pkg/types: modify LoadArgs to return a named error when an unmarshalable condition is detected

Merge pull request #462 from christx2/patch-1

Update Readme.md

Update Readme.md

adding nuage networks cni plugin and repo location

pkg/invoke: ensure that custom env vars are prepended to the env

Ensure that custom values take priority over values defined in the
environment by prepending them to the env, not appending.

Merge pull request #457 from dcbw/move-plugins

plugins: moved to containernetworking/plugins

plugins: moved to containernetworking/plugins

pkg/types: safer typecasting for TextUnmarshaler when loading args

types.LoadArgs does an unsafe typecast for env args key value pairs.
This can cause a panic if the config type fields do not implement
the encoding.TextUnmarshaler interface. This commit modifies it to
do a safe type cast and return an error in such scenarios.

Merge pull request #455 from knobunc/docs/add-openshift-to-readme

Add OpenShift to the list of container runtimes that support CNI

Merge pull request #452 from caniszczyk/add-cni-logo

Add CNI Logo to README

Add OpenShift to the list of container runtimes that support CNI

OpenShift uses CNI, so I thought a link to the appropriate docs made
sense in the top-level readme where it lists the other supporting
runtimes.

Add CNI Logo to README

Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Merge pull request #425 from tomdee/small-build-improvements

scripts/release-with-rkt.sh Minor improvements

Merge pull request #450 from gunjan5/spec-version-mismatch

cniVersion and config structure mismatch

Merge pull request #445 from feiskyer/cniversion

Add cniVersion to Result

cniVersion and config structure mismatch

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

scripts/release-with-rkt.sh Minor improvements

- Use a new Fedora to get a more up to date Go version
- Don't build so many archives, they are not used.
- Handle the new build.sh script
- Just use host networking with rkt

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

Return type.Error instead of plain error

This way, we can use Error.Code attribute to check error type without
needing to regexp the message string

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