Merge pull request #518 from porridge/patch-1

Fix indentation

Fix indentation

Merge pull request #514 from rosenhouse/no-vendor

Remove vendor directory

Remove vendor dir, simplify build & test tooling

"Libraries should never vendor their dependencies"
  https://peter.bourgon.org/go-best-practices-2016/#dependency-management
The only thing in there were test-support packages anyhow

Remove Godeps folder, nothing to manage!

Remove build.sh
- not much here to build now that we've split out plugins
   - for non-AMD64 platforms, travis just runs go build directly

Clean up test.sh
- no vendor --> simpler logic for enumerating packages
- use "bash strict mode"
   http://redsymbol.net/articles/unofficial-bash-strict-mode
- coveralls token not required for public repos

Simplify .travis.yml, update to work without vendor
- fewer env vars
- go get -t ./...

Clean up Vagrantfile
- no godep required anymore
- bump Golang version 1.9.2

libcni: fix go vet error

Merge pull request #508 from squeed/cnitool-cid

cnitool: generate container id from the netns path

Merge pull request #515 from rosenhouse/appveyor

Add CI for Windows

ci: add appveyor file for Windows CI coverage

Windows compatibility: all tests pass!

Merge pull request #510 from squeed/containerid

spec, skel: Make the container ID required and unique.

cnitool: generate container id from the netns path, add docs

We shouldn't be creating networks with a blank containerid. Let's
synthesize one from the netns path.

Also, add a basic README.

spec, skel: Make the container ID required and unique.

Previously, the spec did not require the container id to be set.
However, almost every plugin relies on it being unique, including the
CNI-maintained plugins. So, change the spec to require the use of
containerid and that plugins should treat it as a primary key.

Add "spec containes unreleased changes" warning.

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

Golang versions: add 1.9, drop 1.7

pkg/invoke: fix test pollution

Golang versions: add 1.9.1, drop 1.7

Merge pull request #509 from containernetworking/update-community-sync

Add 2017-10-04 community sync

Add 2017-10-04 community sync

Merge pull request #497 from squeed/args-add-ip

Conventions: add the "ips" arg

Merge pull request #505 from dcbw/op-ordering

spec: add notes about ADD/DEL ordering

spec: add notes about ADD/DEL ordering

Merge pull request #504 from dcbw/gov-voting

governance: add notes about public voting

Merge pull request #506 from Intel-Corp/dev/vhostuser3rdparty-pr

README: List vhostuser CNI as a 3rd party plugin

README: List vhostuser as a 3rd party plugin

Merge pull request #500 from matthewdupre/spec-language

Use more RFC2119 style language in specification (must, should...)

Merge pull request #502 from anfernee/version

skel: VERSION shouldn't block on stdin

governance: add notes about public voting

VERSION shouldn't block on stdin

Right now, I have to do the following
  $ echo | CNI_COMMAND=VERSION bridge
or
  CNI_COMMAND=VERSION bridge
  ^D
to be able to get the result back.

But VERSION doesn't need to read from stdin.

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

Test all non-vendored packages

Test all non-vendored packages

Merge pull request #493 from dcbw/governance

docs: add Governance procedure documentation

Merge pull request #492 from dcbw/roadmap

roadmap: add 1.0 items and clean things up

docs: add Governance procedure documentation

Use more RFC2119 style language in specification (must, should...)

Merge pull request #489 from squeed/clean-build

Release: clean the builddir when building.

Conventions: add the "ips" arg

Merge pull request #494 from dcbw/validate-type

libcni: return error if Type is empty

Merge pull request #495 from rosenhouse/spec-list-older-versions

Spec lists git tags pointing to old spec versions

spec: table of git tags to older versions

libcni: return error if Type is empty

A network config missing Type is pretty useless since it means we can't
find the plugin to execute.  So return an error if Type isn't given.

roadmap: add 1.0 items and clean things up

Release: clean the builddir when building.

Merge pull request #487 from John-Lin/readme-add-plugin-linen

README plugins list: add Linen CNI plugin

README plugins list: add Linen CNI plugin

Merge pull request #482 from feiskyer/cnitool

cnitool: add support for CNI_ARGS

cnitool: add support for CNI_ARGS

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

Merge pull request #481 from tomdee/s390x

scripts/release.sh: Add in s390x architecture

scripts/release.sh: Add in s390x architecture

Merge pull request #427 from squeed/convention-chaining

Conventions: add convention around chaining interfaces

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>