cni.git
9 years agorelease script: don't run the tests
Stefan Junker [Fri, 6 May 2016 15:50:01 +0000 (17:50 +0200)]
release script: don't run the tests

For two reasons:

1. They're not functional within rkt
2. They rebuild the binaries dynamically

9 years agobuild/release: link all release binaries statically
Stefan Junker [Thu, 28 Apr 2016 20:40:59 +0000 (22:40 +0200)]
build/release: link all release binaries statically

9 years agoscripts: build static releases and create an ACI
Stefan Junker [Fri, 22 Apr 2016 22:12:50 +0000 (00:12 +0200)]
scripts: build static releases and create an ACI

* use SHA1 instead of MD5

9 years agoscripts: add "release with rkt"
Stefan Junker [Fri, 22 Apr 2016 16:05:19 +0000 (18:05 +0200)]
scripts: add "release with rkt"

This script uses rkt and a fedora image to build release tarballs.

9 years agoMerge pull request #200 from squaremo/simplify-readme
Michael Bridgen [Thu, 19 May 2016 12:21:40 +0000 (13:21 +0100)]
Merge pull request #200 from squaremo/simplify-readme

Simplify README preamble

9 years agoREADME: Simplify preamble
Michael Bridgen [Thu, 5 May 2016 15:18:45 +0000 (16:18 +0100)]
README: Simplify preamble

9 years agoMerge pull request #175.
Stefan Junker [Thu, 19 May 2016 10:07:30 +0000 (12:07 +0200)]
Merge pull request #175.

9 years agoplugins/bridge: add support to set hairpin mode
Minhan Xia [Tue, 5 Apr 2016 17:50:28 +0000 (10:50 -0700)]
plugins/bridge: add support to set hairpin mode

9 years agomacvlan: sysctl must be set in macvlan interface's namespace
Dan Williams [Thu, 28 Apr 2016 16:26:47 +0000 (11:26 -0500)]
macvlan: sysctl must be set in macvlan interface's namespace

The macvlan is initially created in a separate network namespace
and the sysctl must be set in that namespace too.

9 years agopkg/ns: evaluate syscall number at compile-time
Angus Lees [Fri, 13 May 2016 07:32:40 +0000 (17:32 +1000)]
pkg/ns: evaluate syscall number at compile-time

Previously this code used a run-time map lookup keyed by
runtime.GOOS/GOARCH.  This version uses conditional compilation to make
this choice at compile time, giving immediate feedback for unsupported
platforms.

9 years agopkg/ns: use correct syscall number on arm
Angus Lees [Fri, 13 May 2016 07:18:39 +0000 (17:18 +1000)]
pkg/ns: use correct syscall number on arm

9 years agoMAINTAINERS: add Tom Denham and Gabe Rosenhouse
Jonathan Boulle [Tue, 10 May 2016 10:12:05 +0000 (12:12 +0200)]
MAINTAINERS: add Tom Denham and Gabe Rosenhouse

9 years agoplugins/{ptp,bridge}: teardown first
Stefan Junker [Wed, 11 May 2016 16:50:25 +0000 (18:50 +0200)]
plugins/{ptp,bridge}: teardown first

This will allow the IPAM allocations to be cleared in case the
interfaces and iptables rules are non-existent.

9 years ago*: appc/cni -> containernetworking/cni
Stefan Junker [Wed, 4 May 2016 14:06:05 +0000 (16:06 +0200)]
*: appc/cni -> containernetworking/cni

The project has been moved so internally we simply rename everything.
Consumers are recommended to update their vendored version of cni.

9 years agoREADME: fix badge links
Brandon Philips [Fri, 6 May 2016 15:39:55 +0000 (08:39 -0700)]
README: fix badge links

9 years agoREADME: fix badges
Brandon Philips [Fri, 6 May 2016 15:39:21 +0000 (08:39 -0700)]
README: fix badges

9 years agoREADME: add contiv to cni supported networking
Vipin Jain [Thu, 5 May 2016 03:29:13 +0000 (20:29 -0700)]
README: add contiv to cni supported networking

9 years agoplugins/bridge: clean masquerading rules
Iago López Galeiras [Tue, 3 May 2016 10:21:58 +0000 (12:21 +0200)]
plugins/bridge: clean masquerading rules

In the Add command we set up masquerading rules that didn't have a
corresponding clean-up code in Del.

Add the clean-up code.

9 years agoRemove reference to policy on accepting PRs
Tom Denham [Fri, 22 Apr 2016 19:52:28 +0000 (12:52 -0700)]
Remove reference to policy on accepting PRs

There is to policy

9 years agomacvlan: set proxy_arp in time of creating interface
Piotr Skamruk [Mon, 7 Mar 2016 15:41:04 +0000 (16:41 +0100)]
macvlan: set proxy_arp in time of creating interface

Resolves CNI part of https://github.com/coreos/rkt/issues/1765
Second part would be adding similar lines into kvm flavored macvlan
support (in time of creating macvtap device).

9 years agopkg/utils: add functions to work with sysctl
Piotr Skamruk [Mon, 7 Mar 2016 15:40:27 +0000 (16:40 +0100)]
pkg/utils: add functions to work with sysctl

9 years agoREADME: clarify what CNI is
Stefan Junker [Wed, 20 Apr 2016 15:54:12 +0000 (17:54 +0200)]
README: clarify what CNI is

The word "standard" does not represent what CNI is actually trying to
be.

9 years agoDocument use of goroutine and lockosthread in test helpers
Gabe Rosenhouse [Mon, 18 Apr 2016 03:27:02 +0000 (20:27 -0700)]
Document use of goroutine and lockosthread in test helpers

9 years agons: fix reading net namespace in multi-threaded processes
Dan Williams [Wed, 6 Apr 2016 16:03:31 +0000 (11:03 -0500)]
ns: fix reading net namespace in multi-threaded processes

/proc/self/ns/net gives the main thread's namespace, not necessarily
the namespace of the thread that's running the testcases.  This causes
sporadic failures of the tests.

For example, with a testcase reading inodes after switching netns:

/proc/27686/task/27689/ns/net 4026532565
/proc/self/ns/net 4026531969
/proc/27686/task/27689/ns/net 4026532565

See also:
https://github.com/vishvananda/netns/commit/008d17ae001344769b031375bdb38a86219154c6

Running Suite: pkg/ns Suite
===========================
Random Seed: 1459953577
Will run 6 of 6 specs

• Failure [0.028 seconds]
Linux namespace operations
/cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:167
  WithNetNS
  /cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:166
    executes the callback within the target network namespace [It]
    /cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:97

    Expected
        <uint64>: 4026531969
    to equal
        <uint64>: 4026532565

    /cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:96
------------------------------
•••••

Summarizing 1 Failure:

[Fail] Linux namespace operations WithNetNS [It] executes the callback within the target network namespace
/cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:96

Ran 6 of 6 Specs in 0.564 seconds
FAIL! -- 5 Passed | 1 Failed | 0 Pending | 0 Skipped --- FAIL: TestNs (0.56s)
FAIL

9 years agoFix issues with MakeNetworkNS test helper
Gabe Rosenhouse [Mon, 18 Apr 2016 02:44:00 +0000 (19:44 -0700)]
Fix issues with MakeNetworkNS test helper

9 years agoAdd basic unit tests of testhelpers
Gabe Rosenhouse [Mon, 18 Apr 2016 01:48:50 +0000 (18:48 -0700)]
Add basic unit tests of testhelpers

9 years agoExtract inode inspection functions into testhelpers
Gabe Rosenhouse [Mon, 18 Apr 2016 01:35:49 +0000 (18:35 -0700)]
Extract inode inspection functions into testhelpers

9 years agoExtract testhelpers from loopback test suite
Gabe Rosenhouse [Mon, 18 Apr 2016 01:28:10 +0000 (18:28 -0700)]
Extract testhelpers from loopback test suite

9 years agoREADME: add some material on what CNI isn't
Brandon Philips [Mon, 11 Apr 2016 01:23:06 +0000 (18:23 -0700)]
README: add some material on what CNI isn't

We get some questions about this stuff from time to time and I want to
acknowledge we have thought about them but aren't currently tackling
them.

9 years agoREADME: add some more details on what CNI is
Brandon Philips [Mon, 11 Apr 2016 01:22:38 +0000 (18:22 -0700)]
README: add some more details on what CNI is

We needed a bit more intro material to scope what CNI does. Here is my
attempt.

9 years agoThe macvlan is initially created in a separate network namespaceand the sysctl must...
Stefan Junker [Tue, 17 May 2016 19:11:21 +0000 (21:11 +0200)]
The macvlan is initially created in a separate network namespaceand the sysctl must be set in that namespace too.Fixes #208.

The macvlan is initially created in a separate network namespace
and the sysctl must be set in that namespace too.

Fixes #208.

9 years agopkg/ns: setns syscall number on arm and evaluate them at build time.
Angus Lees [Tue, 17 May 2016 16:44:33 +0000 (02:44 +1000)]
pkg/ns: setns syscall number on arm and evaluate them at build time.

Dummy merge due to previous manual merge.

9 years agoMerge branch 'anguslees-master'
Stefan Junker [Tue, 17 May 2016 16:36:38 +0000 (18:36 +0200)]
Merge branch 'anguslees-master'

9 years agopkg/ns: evaluate syscall number at compile-time
Angus Lees [Fri, 13 May 2016 07:32:40 +0000 (17:32 +1000)]
pkg/ns: evaluate syscall number at compile-time

Previously this code used a run-time map lookup keyed by
runtime.GOOS/GOARCH.  This version uses conditional compilation to make
this choice at compile time, giving immediate feedback for unsupported
platforms.

9 years agopkg/ns: use correct syscall number on arm
Angus Lees [Fri, 13 May 2016 07:18:39 +0000 (17:18 +1000)]
pkg/ns: use correct syscall number on arm

9 years agoMerge pull request #201 from jonboulle/master
Michael Bridgen [Thu, 12 May 2016 13:51:37 +0000 (14:51 +0100)]
Merge pull request #201 from jonboulle/master

MAINTAINERS: add Tom Denham and Gabe Rosenhouse

9 years agoMerge pull request #205 from steveeJ/teardown-fix
Jonathan Boulle [Thu, 12 May 2016 08:54:47 +0000 (10:54 +0200)]
Merge pull request #205 from steveeJ/teardown-fix

plugins/{ptp,bridge}: teardown first

9 years agoplugins/{ptp,bridge}: teardown first
Stefan Junker [Wed, 11 May 2016 16:50:25 +0000 (18:50 +0200)]
plugins/{ptp,bridge}: teardown first

This will allow the IPAM allocations to be cleared in case the
interfaces and iptables rules are non-existent.

9 years agoMAINTAINERS: add Tom Denham and Gabe Rosenhouse
Jonathan Boulle [Tue, 10 May 2016 10:12:05 +0000 (12:12 +0200)]
MAINTAINERS: add Tom Denham and Gabe Rosenhouse

9 years agoMerge pull request #198 from steveeJ/org-move
Stefan Junker [Fri, 6 May 2016 15:58:44 +0000 (17:58 +0200)]
Merge pull request #198 from steveeJ/org-move

*: appc/cni -> containernetworking/cni

9 years agoREADME: fix badge links
Brandon Philips [Fri, 6 May 2016 15:39:55 +0000 (08:39 -0700)]
README: fix badge links

9 years agoREADME: fix badges
Brandon Philips [Fri, 6 May 2016 15:39:21 +0000 (08:39 -0700)]
README: fix badges

9 years ago*: appc/cni -> containernetworking/cni
Stefan Junker [Wed, 4 May 2016 14:06:05 +0000 (16:06 +0200)]
*: appc/cni -> containernetworking/cni

The project has been moved so internally we simply rename everything.
Consumers are recommended to update their vendored version of cni.

9 years agoMerge pull request #197 from jainvipin/master
Michael Bridgen [Thu, 5 May 2016 08:15:15 +0000 (09:15 +0100)]
Merge pull request #197 from jainvipin/master

Add Contiv to CNI supported networking

9 years agoREADME: add contiv to cni supported networking
Vipin Jain [Thu, 5 May 2016 03:29:13 +0000 (20:29 -0700)]
README: add contiv to cni supported networking

9 years agoMerge branch 'pr195' (clean up masq rules)
Michael Bridgen [Wed, 4 May 2016 12:50:09 +0000 (13:50 +0100)]
Merge branch 'pr195' (clean up masq rules)

9 years agoplugins/bridge: clean masquerading rules
Iago López Galeiras [Tue, 3 May 2016 10:21:58 +0000 (12:21 +0200)]
plugins/bridge: clean masquerading rules

In the Add command we set up masquerading rules that didn't have a
corresponding clean-up code in Del.

Add the clean-up code.

9 years agomacvlan: sysctl must be set in macvlan interface's namespace
Dan Williams [Thu, 28 Apr 2016 16:26:47 +0000 (11:26 -0500)]
macvlan: sysctl must be set in macvlan interface's namespace

The macvlan is initially created in a separate network namespace
and the sysctl must be set in that namespace too.

9 years agoMerge pull request #189 from tomdee/patch-2
Michael Bridgen [Wed, 27 Apr 2016 14:47:42 +0000 (15:47 +0100)]
Merge pull request #189 from tomdee/patch-2

Remove reference to policy on accepting PRs

9 years agoRemove reference to policy on accepting PRs
Tom Denham [Fri, 22 Apr 2016 19:52:28 +0000 (12:52 -0700)]
Remove reference to policy on accepting PRs

There is to policy

9 years agoMerge pull request #187 from appc/v0.3.0
Stefan Junker [Fri, 22 Apr 2016 17:25:54 +0000 (19:25 +0200)]
Merge pull request #187 from appc/v0.3.0

merge unneeded branch v0.3.0 into master

9 years agoMerge pull request #186 from steveeJ/project-clarification
Michael Bridgen [Thu, 21 Apr 2016 10:54:52 +0000 (11:54 +0100)]
Merge pull request #186 from steveeJ/project-clarification

README: clarify what CNI is

9 years agoREADME: clarify what CNI is
Stefan Junker [Wed, 20 Apr 2016 15:54:12 +0000 (17:54 +0200)]
README: clarify what CNI is

The word "standard" does not represent what CNI is actually trying to
be.

9 years agoMerge pull request #182 from rosenhouse/document-lockosthread
Zach Gershman [Mon, 18 Apr 2016 17:45:46 +0000 (10:45 -0700)]
Merge pull request #182 from rosenhouse/document-lockosthread

Document use of goroutine and LockOSThread in test helpers

9 years agoMerge pull request #176 from dcbw/ns-test-fix
Zach Gershman [Mon, 18 Apr 2016 17:44:08 +0000 (10:44 -0700)]
Merge pull request #176 from dcbw/ns-test-fix

ns: fix reading net namespace in multi-threaded processes

9 years agons: fix reading net namespace in multi-threaded processes
Dan Williams [Wed, 6 Apr 2016 16:03:31 +0000 (11:03 -0500)]
ns: fix reading net namespace in multi-threaded processes

/proc/self/ns/net gives the main thread's namespace, not necessarily
the namespace of the thread that's running the testcases.  This causes
sporadic failures of the tests.

For example, with a testcase reading inodes after switching netns:

/proc/27686/task/27689/ns/net 4026532565
/proc/self/ns/net 4026531969
/proc/27686/task/27689/ns/net 4026532565

See also:
https://github.com/vishvananda/netns/commit/008d17ae001344769b031375bdb38a86219154c6

Running Suite: pkg/ns Suite
===========================
Random Seed: 1459953577
Will run 6 of 6 specs

• Failure [0.028 seconds]
Linux namespace operations
/cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:167
  WithNetNS
  /cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:166
    executes the callback within the target network namespace [It]
    /cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:97

    Expected
        <uint64>: 4026531969
    to equal
        <uint64>: 4026532565

    /cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:96
------------------------------
•••••

Summarizing 1 Failure:

[Fail] Linux namespace operations WithNetNS [It] executes the callback within the target network namespace
/cni/gopath/src/github.com/appc/cni/pkg/ns/ns_test.go:96

Ran 6 of 6 Specs in 0.564 seconds
FAIL! -- 5 Passed | 1 Failed | 0 Pending | 0 Skipped --- FAIL: TestNs (0.56s)
FAIL

9 years agoDocument use of goroutine and lockosthread in test helpers
Gabe Rosenhouse [Mon, 18 Apr 2016 03:27:02 +0000 (20:27 -0700)]
Document use of goroutine and lockosthread in test helpers

9 years agoMerge pull request #181 from rosenhouse/testhelpers-extraction
Zach Gershman [Mon, 18 Apr 2016 03:20:43 +0000 (20:20 -0700)]
Merge pull request #181 from rosenhouse/testhelpers-extraction

Extract and improve test helpers

9 years agoFix issues with MakeNetworkNS test helper
Gabe Rosenhouse [Mon, 18 Apr 2016 02:44:00 +0000 (19:44 -0700)]
Fix issues with MakeNetworkNS test helper

9 years agoAdd basic unit tests of testhelpers
Gabe Rosenhouse [Mon, 18 Apr 2016 01:48:50 +0000 (18:48 -0700)]
Add basic unit tests of testhelpers

9 years agoExtract inode inspection functions into testhelpers
Gabe Rosenhouse [Mon, 18 Apr 2016 01:35:49 +0000 (18:35 -0700)]
Extract inode inspection functions into testhelpers

9 years agoExtract testhelpers from loopback test suite
Gabe Rosenhouse [Mon, 18 Apr 2016 01:28:10 +0000 (18:28 -0700)]
Extract testhelpers from loopback test suite

9 years agoMerge pull request #180 from philips/add-more-intro
Michael Bridgen [Mon, 11 Apr 2016 09:12:22 +0000 (10:12 +0100)]
Merge pull request #180 from philips/add-more-intro

README: add some more introductory material

9 years agoREADME: add some material on what CNI isn't
Brandon Philips [Mon, 11 Apr 2016 01:23:06 +0000 (18:23 -0700)]
README: add some material on what CNI isn't

We get some questions about this stuff from time to time and I want to
acknowledge we have thought about them but aren't currently tackling
them.

9 years agoREADME: add some more details on what CNI is
Brandon Philips [Mon, 11 Apr 2016 01:22:38 +0000 (18:22 -0700)]
README: add some more details on what CNI is

We needed a bit more intro material to scope what CNI does. Here is my
attempt.

9 years agoMerge pull request #177 from steveeJ/macvlan-arp-proxy
Stefan Junker [Fri, 8 Apr 2016 12:53:48 +0000 (14:53 +0200)]
Merge pull request #177 from steveeJ/macvlan-arp-proxy

Macvlan arp proxy

9 years agoMerge pull request #178 from steveeJ/master
Stefan Junker [Fri, 8 Apr 2016 12:35:58 +0000 (14:35 +0200)]
Merge pull request #178 from steveeJ/master

travis: don't go get vet

9 years agotravis: don't go get vet
Stefan Junker [Fri, 8 Apr 2016 12:22:20 +0000 (14:22 +0200)]
travis: don't go get vet

9 years agotravis: don't go get vet
Stefan Junker [Fri, 8 Apr 2016 12:22:20 +0000 (14:22 +0200)]
travis: don't go get vet

9 years agomacvlan: set proxy_arp in time of creating interface
Piotr Skamruk [Mon, 7 Mar 2016 15:41:04 +0000 (16:41 +0100)]
macvlan: set proxy_arp in time of creating interface

Resolves CNI part of https://github.com/coreos/rkt/issues/1765
Second part would be adding similar lines into kvm flavored macvlan
support (in time of creating macvtap device).

9 years agopkg/utils: add functions to work with sysctl
Piotr Skamruk [Mon, 7 Mar 2016 15:40:27 +0000 (16:40 +0100)]
pkg/utils: add functions to work with sysctl

9 years agoMerge pull request #173 from jonboulle/master
Stefan Junker [Sat, 2 Apr 2016 13:22:40 +0000 (15:22 +0200)]
Merge pull request #173 from jonboulle/master

*: add missing license headers + check

9 years ago*: add missing license headers + check
Jonathan Boulle [Fri, 1 Apr 2016 13:35:21 +0000 (15:35 +0200)]
*: add missing license headers + check

9 years agoMerge pull request #172 from squaremo/readme_more
Michael Bridgen [Fri, 1 Apr 2016 12:23:28 +0000 (13:23 +0100)]
Merge pull request #172 from squaremo/readme_more

Readme improvements

9 years agodocs: add brief section on contributing, to README
Michael Bridgen [Fri, 1 Apr 2016 11:34:49 +0000 (12:34 +0100)]
docs: add brief section on contributing, to README

9 years agodocs: cosmetic edits to README.md
Michael Bridgen [Fri, 1 Apr 2016 09:58:25 +0000 (10:58 +0100)]
docs: cosmetic edits to README.md

9 years agoMerge pull request #168 from steveeJ/chain-len
Stefan Junker [Thu, 31 Mar 2016 17:25:13 +0000 (19:25 +0200)]
Merge pull request #168 from steveeJ/chain-len

attempt to ensure chain name is unique and at most 29 chars long

9 years agopkg/utils: correct the test's expected chain names
Stefan Junker [Thu, 31 Mar 2016 15:11:11 +0000 (17:11 +0200)]
pkg/utils: correct the test's expected chain names

9 years ago*: add comment to iptables rules for ipmasq
Stefan Junker [Thu, 31 Mar 2016 13:44:54 +0000 (15:44 +0200)]
*: add comment to iptables rules for ipmasq

9 years agopkg/utils: remove unneeded condition in tests
Stefan Junker [Thu, 31 Mar 2016 09:55:46 +0000 (11:55 +0200)]
pkg/utils: remove unneeded condition in tests

9 years agopkg/utils: fix docstring
Stefan Junker [Thu, 31 Mar 2016 09:50:18 +0000 (11:50 +0200)]
pkg/utils: fix docstring

9 years agopkg/utils: split and unexport constants
Stefan Junker [Thu, 31 Mar 2016 08:08:52 +0000 (10:08 +0200)]
pkg/utils: split and unexport constants

9 years agopkg/utils: fix docstring
Stefan Junker [Wed, 30 Mar 2016 17:40:31 +0000 (19:40 +0200)]
pkg/utils: fix docstring

9 years agopkg/utils: use constant for chain prefix
Stefan Junker [Wed, 30 Mar 2016 17:40:20 +0000 (19:40 +0200)]
pkg/utils: use constant for chain prefix

9 years agopkg/utils: use name+id for hash and extend tests
Stefan Junker [Wed, 30 Mar 2016 17:17:37 +0000 (19:17 +0200)]
pkg/utils: use name+id for hash and extend tests

9 years agopkg: add a function to generate chain names
Abhishek Chanda [Wed, 23 Mar 2016 00:16:59 +0000 (17:16 -0700)]
pkg: add a function to generate chain names

Adds a function to generate chain names for use in iptables and
ports all drivers to use that function. Also adds tests for the
said function.

9 years agoMerge pull request #171 from steveeJ/bump-cni
Zach Gershman [Thu, 31 Mar 2016 15:46:55 +0000 (08:46 -0700)]
Merge pull request #171 from steveeJ/bump-cni

vendoring: bump coreos/go-iptables to 0.1.0

9 years agovendoring: bump coreos/go-iptables to 0.1.0
Stefan Junker [Thu, 31 Mar 2016 15:33:50 +0000 (17:33 +0200)]
vendoring: bump coreos/go-iptables to 0.1.0

9 years agoMerge pull request #170 from jonboulle/master
Michael Bridgen [Thu, 31 Mar 2016 13:51:30 +0000 (14:51 +0100)]
Merge pull request #170 from jonboulle/master

README: add Calico and Weave as CNI users

9 years agoREADME: add Calico and Weave as CNI users
Jonathan Boulle [Thu, 31 Mar 2016 13:06:49 +0000 (15:06 +0200)]
README: add Calico and Weave as CNI users

9 years agoMerge pull request #169 from philips/add-users
Stefan Junker [Thu, 31 Mar 2016 08:21:13 +0000 (10:21 +0200)]
Merge pull request #169 from philips/add-users

README: add a list of CNI users

9 years agoREADME: add a list of CNI users
Brandon Philips [Wed, 30 Mar 2016 22:59:16 +0000 (15:59 -0700)]
README: add a list of CNI users

9 years agoMerge pull request #165 from dcbw/go15-vendor
Zach Gershman [Thu, 24 Mar 2016 17:44:00 +0000 (10:44 -0700)]
Merge pull request #165 from dcbw/go15-vendor

scripts: fix build with go-1.5 after switch to vendor/

9 years agoscripts: fix build with go-1.5 after switch to vendor/
Dan Williams [Thu, 24 Mar 2016 16:33:19 +0000 (11:33 -0500)]
scripts: fix build with go-1.5 after switch to vendor/

go-1.6 enables vendor by default, but go-1.5 needs an environment variable.

9 years agoMerge pull request #158 from steveeJ/loadargs
Zach Gershman [Wed, 23 Mar 2016 05:05:57 +0000 (22:05 -0700)]
Merge pull request #158 from steveeJ/loadargs

Add generic IgnoreUnknown to valid CNI_ARGS

9 years agopkg/types: add tests for args
Stefan Junker [Thu, 17 Mar 2016 13:09:54 +0000 (14:09 +0100)]
pkg/types: add tests for args

9 years agoGodeps: bump ginkgo for table extensions
Stefan Junker [Thu, 17 Mar 2016 17:32:56 +0000 (18:32 +0100)]
Godeps: bump ginkgo for table extensions

9 years agopkg/types: add IgnoreUnknown arg and logic
Stefan Junker [Thu, 17 Mar 2016 13:08:29 +0000 (14:08 +0100)]
pkg/types: add IgnoreUnknown arg and logic

This commit adds a struct type CommonArgs that is to be embedded in
every plugin's argument struct. It contains a field named
"IgnoreUnknown" which will be parsed as a boolean and can be provided to
ignore unknown arguments passed to the plugin.

9 years agoplugins/host-local: only call LoadArgs() once
Stefan Junker [Thu, 17 Mar 2016 12:57:28 +0000 (13:57 +0100)]
plugins/host-local: only call LoadArgs() once

9 years agoMerge pull request #162 from zachgersh/remove-workspace
Stefan Junker [Mon, 21 Mar 2016 16:48:25 +0000 (17:48 +0100)]
Merge pull request #162 from zachgersh/remove-workspace

build script: remove the _workspace from GOPATH