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

261 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Michael Bridgen
ddc0ca4791 docs: details on PR acceptance policy 2016-06-01 12:23:21 +01:00
Michael Bridgen
bcc189e1e6 docs: Mention license headers and doc contributions 
Closes #174.
 2016-05-19 13:42:36 +01:00
Michael Bridgen
991176a64a Merge pull request #200 from squaremo/simplify-readme 
Simplify README preamble
 2016-05-19 13:21:40 +01:00
Michael Bridgen
8db1f0d503 README: Simplify preamble 2016-05-19 12:52:09 +01:00
Stefan Junker
74790e55ef Merge pull request #175. 2016-05-19 12:08:41 +02:00
Minhan Xia
89ab9299ab plugins/bridge: add support to set hairpin mode 2016-05-19 12:06:37 +02:00
Dan Williams
f4eb76318e 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-05-19 12:06:37 +02:00
Angus Lees
a80bcc0371 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-19 12:06:37 +02:00
Angus Lees
979044ad2f pkg/ns: use correct syscall number on arm 2016-05-19 12:06:37 +02:00
Jonathan Boulle
ec78e7cf1e MAINTAINERS: add Tom Denham and Gabe Rosenhouse 2016-05-19 12:06:37 +02:00
Stefan Junker
5d67d22caa 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-19 12:06:37 +02:00
Stefan Junker
2c482f433c *: 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-19 12:06:36 +02:00
Brandon Philips
419711fee3 README: fix badge links 2016-05-19 12:06:36 +02:00
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
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
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