ITQC/containernetworking-plugins
24
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 34 Wiki Activity
208 Commits 6 Branches 48 Tags
Commit Graph

208 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Brandon Philips
04dbf7ce45 README: fix badges 2016-05-19 12:06:36 +02:00
Vipin Jain
61f20da4f2 README: add contiv to cni supported networking 2016-05-19 12:06:36 +02:00
Iago López Galeiras
a993ef5f7f 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.
 2016-05-19 12:06:36 +02:00
Tom Denham
41a5dfac33 Remove reference to policy on accepting PRs 
There is to policy
 2016-05-19 12:06:36 +02:00
Piotr Skamruk
a1dab0aa40 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).
 2016-05-19 12:06:16 +02:00
Piotr Skamruk
4298aa94a5 pkg/utils: add functions to work with sysctl 2016-05-19 12:06:16 +02:00
Stefan Junker
d53ffe1b40 README: clarify what CNI is 
The word "standard" does not represent what CNI is actually trying to
be.
 2016-05-19 12:06:16 +02:00
Gabe Rosenhouse
b5e56b3dab Document use of goroutine and lockosthread in test helpers 2016-05-19 12:06:16 +02:00
Dan Williams
260c7f96bf 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:
008d17ae00

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
 2016-05-19 12:06:16 +02:00
Gabe Rosenhouse
b09af2ff91 Fix issues with MakeNetworkNS test helper 2016-05-19 12:06:16 +02:00
Gabe Rosenhouse
1bbc87ab54 Add basic unit tests of testhelpers 2016-05-19 12:06:16 +02:00
Gabe Rosenhouse
4fc8a90277 Extract inode inspection functions into testhelpers 2016-05-19 12:06:16 +02:00
Gabe Rosenhouse
9044dc2831 Extract testhelpers from loopback test suite 2016-05-19 12:06:16 +02:00
Brandon Philips
5019e283b1 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.
 2016-05-19 12:06:16 +02:00
Brandon Philips
e7ae43e517 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.
 2016-05-19 12:06:16 +02:00
Stefan Junker
27d37cbdf1 travis: don't go get vet 2016-04-08 14:29:04 +02:00
Stefan Junker
a15a372021 Merge pull request #173 from jonboulle/master 
*: add missing license headers + check
 2016-04-02 15:22:40 +02:00
Jonathan Boulle
f6135c7ba3 *: add missing license headers + check 2016-04-01 15:35:21 +02:00
Michael Bridgen
0046767be7 Merge pull request #172 from squaremo/readme_more 
Readme improvements
v0.2.0-rc0 		2016-04-01 13:23:28 +01:00
Michael Bridgen
fcbde68d32 docs: add brief section on contributing, to README 2016-04-01 12:51:39 +01:00
Michael Bridgen
8c860a832f docs: cosmetic edits to README.md 2016-04-01 12:37:39 +01:00
Stefan Junker
42c4cb842d Merge pull request #168 from steveeJ/chain-len 
attempt to ensure chain name is unique and at most 29 chars long
 2016-03-31 19:25:13 +02:00
Stefan Junker
897766d74b pkg/utils: correct the test's expected chain names 2016-03-31 18:01:32 +02:00
Stefan Junker
6aad63055c *: add comment to iptables rules for ipmasq 2016-03-31 18:01:32 +02:00
Stefan Junker
bcef17daac pkg/utils: remove unneeded condition in tests 2016-03-31 18:01:32 +02:00
Stefan Junker
77759626a8 pkg/utils: fix docstring 2016-03-31 18:01:32 +02:00
Stefan Junker
53d9cee00a pkg/utils: split and unexport constants 2016-03-31 18:01:32 +02:00
Stefan Junker
bc44d1227d pkg/utils: fix docstring 2016-03-31 18:01:32 +02:00
Stefan Junker
3e6069cab5 pkg/utils: use constant for chain prefix 2016-03-31 18:01:32 +02:00
Stefan Junker
09248dfad9 pkg/utils: use name+id for hash and extend tests 2016-03-31 18:01:32 +02:00
Abhishek Chanda
c33daf6706 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.
 2016-03-31 18:01:32 +02:00
Zach Gershman
bacaa11d2d Merge pull request #171 from steveeJ/bump-cni 
vendoring: bump coreos/go-iptables to 0.1.0
 2016-03-31 08:46:55 -07:00
Stefan Junker
88d974c562 vendoring: bump coreos/go-iptables to 0.1.0 2016-03-31 17:33:50 +02:00
Michael Bridgen
5362e90c8a Merge pull request #170 from jonboulle/master 
README: add Calico and Weave as CNI users
 2016-03-31 14:51:30 +01:00
Jonathan Boulle
77a5bf44f4 README: add Calico and Weave as CNI users 2016-03-31 15:44:03 +02:00
Stefan Junker
1e0e105b36 Merge pull request #169 from philips/add-users 
README: add a list of CNI users
 2016-03-31 10:21:13 +02:00
Brandon Philips
0b4f7d849f README: add a list of CNI users 2016-03-30 15:59:16 -07:00
Zach Gershman
4146443942 Merge pull request #165 from dcbw/go15-vendor 
scripts: fix build with go-1.5 after switch to vendor/
 2016-03-24 10:44:00 -07:00
Dan Williams
ae2f64ec40 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.
 2016-03-24 11:33:19 -05:00
Zach Gershman
5c79f8bd5d Merge pull request #158 from steveeJ/loadargs 
Add generic IgnoreUnknown to valid CNI_ARGS
 2016-03-22 22:05:57 -07:00
Stefan Junker
8916a7ea5f pkg/types: add tests for args 2016-03-21 20:40:39 +01:00
Stefan Junker
b91aec9a62 Godeps: bump ginkgo for table extensions 2016-03-21 20:36:05 +01:00
Stefan Junker
5a39a168f7 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.
 2016-03-21 20:36:05 +01:00
Stefan Junker
05c5aea77b plugins/host-local: only call LoadArgs() once 2016-03-21 20:17:47 +01:00
Stefan Junker
122abae03a Merge pull request #162 from zachgersh/remove-workspace 
build script: remove the _workspace from GOPATH
 2016-03-21 17:48:25 +01:00
zachgersh
8817b1b946 build script: remove the _workspace from GOPATH 
Now that we no longer use godeps to manage
dependencies we can safely remove this.
Dependencies in the vendor directory will
automagically be picked up.
 2016-03-21 09:32:09 -07:00
Zach Gershman
6ef5ef1ec1 Merge pull request #161 from steveeJ/skel-tests 
pkg/skel: add rudimentary unit tests
 2016-03-19 18:09:07 -07:00
Stefan Junker
a6c5d80661 pkg/skel: add rudimentary unit tests 
This is an attempt to testing the PluginMain() function of the skel pkg.
We should be able to do better by using a mockable interface for the
plugins, but this is a start.
 2016-03-18 22:34:29 +01:00
Stefan Junker
e72674949e Merge pull request #156 from zachgersh/vendor 
Moves to go team vendoring solution
 2016-03-17 10:02:30 +01:00
Zach Gershman
a3038a5bec Merge pull request #157 from achanda/vet 
Fix govet warnings
 2016-03-16 17:01:57 -07:00