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

486 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
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
72e98b97d6 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.
 2016-05-17 21:11:21 +02:00
Angus Lees
8f5194d9b5 pkg/ns: setns syscall number on arm and evaluate them at build time. 
Dummy merge due to previous manual merge.
 2016-05-17 18:44:33 +02:00
Stefan Junker
31379b8a41 Merge branch 'anguslees-master' 2016-05-17 18:36:38 +02:00
Angus Lees
2b019bfde1 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.
 2016-05-17 18:36:29 +02:00
Angus Lees
a2f23f5a84 pkg/ns: use correct syscall number on arm 2016-05-17 18:36:22 +02:00
Dan Williams
899c0efd42 bridge: leave TX queue length as kernel default, not 0 
Not using NewLinkAttrs() or not initializing TxQLen leaves
the value as 0, which tells the kernel to set a zero-length
tx_queue_len.  That messes up FIFO traffic shapers (like pfifo)
that use the device TX queue length as the default packet
limit.  This leads to a default packet limit of 0, which drops
all packets.
 2016-05-12 10:35:50 -05:00
Michael Bridgen
e1907bdab7 Merge pull request #201 from jonboulle/master 
MAINTAINERS: add Tom Denham and Gabe Rosenhouse
 2016-05-12 14:51:37 +01:00
Jonathan Boulle
463e171614 Merge pull request #205 from steveeJ/teardown-fix 
plugins/{ptp,bridge}: teardown first
 2016-05-12 10:54:47 +02:00
Stefan Junker
d91971cbe0 plugins/{ptp,bridge}: teardown first 
This will allow the IPAM allocations to be cleared in case the
interfaces and iptables rules are non-existent.
 2016-05-11 19:47:55 +02:00
Jonathan Boulle
bf8bde8256 MAINTAINERS: add Tom Denham and Gabe Rosenhouse 2016-05-10 12:12:05 +02:00
Stefan Junker
64791dd233 Merge pull request #198 from steveeJ/org-move 
*: appc/cni -> containernetworking/cni
 2016-05-06 17:58:44 +02:00
Brandon Philips
e021964819 README: fix badge links 2016-05-06 08:39:55 -07:00
Brandon Philips
56ddbfd522 README: fix badges 2016-05-06 08:39:21 -07:00
Stefan Junker
ed5f8dfeb2 *: 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.
 2016-05-06 16:40:27 +02:00
Michael Bridgen
fb34e4a21d Merge pull request #197 from jainvipin/master 
Add Contiv to CNI supported networking
 2016-05-05 09:15:15 +01:00
Vipin Jain
914a85c544 README: add contiv to cni supported networking 2016-05-04 20:29:13 -07:00
Michael Bridgen
dbcecbcec6 Merge branch 'pr195' (clean up masq rules) 2016-05-04 13:50:09 +01:00
Iago López Galeiras
6aa26f9fd2 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-03 12:21:58 +02:00
Dan Williams
99ca41457e 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.
 2016-04-28 11:26:47 -05:00
Michael Bridgen
cae660409f Merge pull request #189 from tomdee/patch-2 
Remove reference to policy on accepting PRs
 2016-04-27 15:47:42 +01:00
Tom Denham
902ee8b992 Remove reference to policy on accepting PRs 
There is to policy
 2016-04-22 12:52:28 -07:00
Stefan Junker
911d941794 Merge pull request #187 from appc/v0.3.0 
merge unneeded branch v0.3.0 into master
 2016-04-22 19:25:54 +02:00
Michael Bridgen
8bb41b9d6f Merge pull request #186 from steveeJ/project-clarification 
README: clarify what CNI is
 2016-04-21 11:54:52 +01:00
Stefan Junker
ef251cc6ce README: clarify what CNI is 
The word "standard" does not represent what CNI is actually trying to
be.
 2016-04-20 20:00:50 +02:00
Zach Gershman
57b28f3936 Merge pull request #182 from rosenhouse/document-lockosthread 
Document use of goroutine and LockOSThread in test helpers
 2016-04-18 10:45:46 -07:00
Zach Gershman
7383809048 Merge pull request #176 from dcbw/ns-test-fix 
ns: fix reading net namespace in multi-threaded processes
 2016-04-18 10:44:08 -07:00
Dan Williams
07a993dc7d 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-04-18 10:35:23 -05:00
Gabe Rosenhouse
94c4b25698 Document use of goroutine and lockosthread in test helpers 2016-04-17 20:27:02 -07:00
Zach Gershman
616702bcf3 Merge pull request #181 from rosenhouse/testhelpers-extraction 
Extract and improve test helpers
 2016-04-17 20:20:43 -07:00
Gabe Rosenhouse
e3fb7ce3a7 Fix issues with MakeNetworkNS test helper 2016-04-17 19:47:54 -07:00
Gabe Rosenhouse
dd3b85c9cc Add basic unit tests of testhelpers 2016-04-17 19:25:39 -07:00
Gabe Rosenhouse
54d7f73092 Extract inode inspection functions into testhelpers 2016-04-17 18:35:49 -07:00
Gabe Rosenhouse
5d932e4716 Extract testhelpers from loopback test suite 2016-04-17 18:28:10 -07:00
Michael Bridgen
9a2801085b Merge pull request #180 from philips/add-more-intro 
README: add some more introductory material
 2016-04-11 10:12:22 +01:00
Brandon Philips
4c778a63a7 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-04-10 18:23:06 -07:00
Brandon Philips
d5a8ec9029 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-04-10 18:22:38 -07:00
Stefan Junker
41ee449fb2 Merge pull request #177 from steveeJ/macvlan-arp-proxy 
Macvlan arp proxy
 2016-04-08 14:53:48 +02:00
Stefan Junker
a06a464235 Merge pull request #178 from steveeJ/master 
travis: don't go get vet
 2016-04-08 14:35:58 +02:00
Stefan Junker
27d37cbdf1 travis: don't go get vet 2016-04-08 14:29:04 +02:00
Stefan Junker
bc482febfb travis: don't go get vet 2016-04-08 14:22:20 +02:00
Piotr Skamruk
b2f492e8b5 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-04-07 21:40:58 +02:00
Piotr Skamruk
72bec92851 pkg/utils: add functions to work with sysctl 2016-04-07 21:40:51 +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