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

979 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Gabe Rosenhouse
b2ef4aa1a7 docs: on contributing plugins to this repo 2016-10-02 23:44:59 -07:00
Gabe Rosenhouse
b07b226c97 testing: adds basic test of backwards compatibility 2016-10-02 22:18:53 -07:00
Gabe Rosenhouse
51b51a0182 testing: adds basic test of backwards compatibility 2016-10-02 22:18:53 -07:00
Dan Williams
a29fc24f11 Merge pull request #296 from rosenhouse/plugins-require-versioned-config 
Plugins validate cniVersion of NetConf
 2016-09-22 17:11:34 -05:00
Dan Williams
b5e0a4ef2e Merge pull request #298 from freehan/fix-host-local-30 
host-local: fix allocation of last address in a /30
 2016-09-22 09:13:32 -05:00
Minhan Xia
52e4358cbd host-local: fix allocation of last address in a /30 2016-09-21 13:10:05 -07:00
Brandon Philips
e70e6034d2 Merge pull request #299 from tgraf/cilium-link 
List cilium as 3rd party plugin
 2016-09-19 14:31:54 -07:00
Gabe Rosenhouse
f721eaf6ed skel: use named constant for Incompatible CNI Version error code 2016-09-19 13:25:33 -07:00
Gabe Rosenhouse
5b696f3307 skel: use named constant for Incompatible CNI Version error code 2016-09-19 13:25:33 -07:00
Gabe Rosenhouse
0135e2751e versioning: ipam config is not versioned 2016-09-19 13:25:28 -07:00
Gabe Rosenhouse
04ba7c7761 versioning: plugins require version match with config 
infer version 0.1.0 when config is missing an explicit "cniVersion" field
 2016-09-19 13:00:49 -07:00
Gabe Rosenhouse
fba37620e0 versioning: plugins require version match with config 
infer version 0.1.0 when config is missing an explicit "cniVersion" field
 2016-09-19 13:00:49 -07:00
Gabe Rosenhouse
1e4e8fdb5d versioning: add basic version decode for network config 2016-09-19 12:57:08 -07:00
Gabe Rosenhouse
64bbcd8cf7 versioning: add basic version decode for network config 2016-09-19 12:57:08 -07:00
Thomas Graf
75d983ec25 README: List cilium as 3rd party plugin 
Signed-off-by: Thomas Graf <tgraf@suug.ch>
 2016-09-19 13:45:39 +02:00
Gabe Rosenhouse
921292ff77 skel: Plugins require a cniVersion in the NetConf 2016-09-18 21:30:57 -07:00
Gabe Rosenhouse
fd150a4c97 skel: Plugins require a cniVersion in the NetConf 2016-09-18 21:30:57 -07:00
Gabe Rosenhouse
56032390fe Merge pull request #287 from rosenhouse/multi-version 
Plugins report a list of supported versions
 2016-09-12 13:44:14 -07:00
Gabe Rosenhouse
3f4011b3ec Merge pull request #295 from rosenhouse/update-readme-cloudfoundry-link 
readme: update link to CloudFoundry usage of CNI
 2016-09-08 00:47:04 -04:00
Gabe Rosenhouse
d32ccb1899 readme: update link to CloudFoundry usage of CNI 2016-09-06 23:48:48 -04:00
Gabe Rosenhouse
de91f31ae7 versioning: revert spec version to 0.2.0 2016-09-06 15:37:42 -04:00
Gabe Rosenhouse
7958b9f0cc versioning: revert spec version to 0.2.0 2016-09-06 15:37:42 -04:00
Gabe Rosenhouse
bf6948da19 versioning: misc cleanups 
highlights:
 - NetConf struct finally includes cniVersion field
 - improve test coverage of current version report behavior
 - godoc a few key functions
 - allow tests to control version list reported by no-op plugin
 2016-09-06 15:35:58 -04:00
Gabe Rosenhouse
d5e2e375d4 versioning: misc cleanups 
highlights:
 - NetConf struct finally includes cniVersion field
 - improve test coverage of current version report behavior
 - godoc a few key functions
 - allow tests to control version list reported by no-op plugin
 2016-09-06 15:35:58 -04:00
Gabe Rosenhouse
07a8a28637 Merge pull request #294 from dcbw/host-local-ranges 
Fix handling of host-local IP address ranges
 2016-09-05 23:24:42 -04:00
Dan Williams
95a9ea0bd2 host-local: don't allocate the broadcast address or allow invalid networks 
There aren't any IPs to allocate in /32 or /31 networks, so don't allow them.
 2016-09-02 17:20:09 -05:00
Dan Williams
959af1e6ab host-local: ensure requested IPs are within the given range 
And also make sure that RangeStart and RangeEnd are sane.
 2016-09-02 16:52:29 -05:00
Dan Williams
dfc4f7cd2a host-local: don't allocate past RangeEnd 
When RangeEnd is given, a.end = RangeEnd+1.

If when getSearchRange() is called and lastReservedIP equals
RangeEnd, a.nextIP() only compares lastReservedIP (which in this
example is RangeEnd) against a.end (which in this example is
RangeEnd+1) and they clearly don't match, so a.nextIP() returns
start=RangeEnd+1 and end=RangeEnd.

Get() happily allocates RangeEnd+1 because it only compares 'cur'
to the end returned by getSearchRange(), not to a.end, and thus
allocates past RangeEnd.

Since a.end is inclusive (eg, host-local will allocate a.end) the
fix is to simply set a.end equal to RangeEnd.
 2016-09-02 16:37:34 -05:00
Gabe Rosenhouse
2f97407396 invoke: better name and unit test coverage for GetVersionInfo 2016-09-02 16:39:01 -04:00
Gabe Rosenhouse
bf31ed1591 invoke: better name and unit test coverage for GetVersionInfo 2016-09-02 16:39:01 -04:00
Gabe Rosenhouse
f467134838 invoke: correctly infer version for 0.1.0-vintage plugins 
Older plugins return a known error when issued the VERSION command.
Capture this error and report it as a 0.1.0 version plugin.
 2016-09-02 16:02:07 -04:00
Gabe Rosenhouse
97192fc979 invoke: correctly infer version for 0.1.0-vintage plugins 
Older plugins return a known error when issued the VERSION command.
Capture this error and report it as a 0.1.0 version plugin.
 2016-09-02 16:02:07 -04:00
Gabe Rosenhouse
2142c076ab versioning: adds tooling to compile a program against a given old CNI version 
Allows us to write tests that cover interactions between components of
differing versions
 2016-09-02 16:02:02 -04:00
Gabe Rosenhouse
deb4466041 versioning: adds tooling to compile a program against a given old CNI version 
Allows us to write tests that cover interactions between components of
differing versions
 2016-09-02 16:02:02 -04:00
Gabe Rosenhouse
bd3ade0c54 pkg/invoke: refactor plugin exec and backfill unit tests 2016-09-02 15:59:25 -04:00
Gabe Rosenhouse
dea1c6e44d pkg/invoke: refactor plugin exec and backfill unit tests 2016-09-02 15:59:25 -04:00
Gabe Rosenhouse
383c84031e versioning: document meaning of 'Legacy' version support 2016-09-02 11:26:17 -04:00
Gabe Rosenhouse
adf28a84c6 versioning: document meaning of 'Legacy' version support 2016-09-02 11:26:17 -04:00
Gabe Rosenhouse
b1e254d901 versioning: plugins report a list of supported versions 
Further progress on versioning support (Issue #266).
Bump CNI spec version to 0.3.0
 2016-09-02 11:26:17 -04:00
Gabe Rosenhouse
536cb5b99b versioning: plugins report a list of supported versions 
Further progress on versioning support (Issue #266).
Bump CNI spec version to 0.3.0
 2016-09-02 11:26:17 -04:00
Gabe Rosenhouse
c5e39a87f7 Merge pull request #293 from rosenhouse/backfill-invoke-exec-tests 
Backfill tests for plugin execution
 2016-08-31 20:49:08 -04:00
Gabe Rosenhouse
63ace9e496 Merge pull request #293 from rosenhouse/backfill-invoke-exec-tests 
Backfill tests for plugin execution
 2016-08-31 20:49:08 -04:00
Dan Williams
e0ea82b229 Merge pull request #290 from asridharan/dcos 
Fixed the check for network namespace path.
 2016-08-31 18:19:02 -05:00
Dan Williams
26241a517a Merge pull request #290 from asridharan/dcos 
Fixed the check for network namespace path.
 2016-08-31 18:19:02 -05:00
Avinash Sridharan
7281d5792a pkg/ns: fixed the check for network namespace path. 
The expectation on older kernels (< 3.19) was to have the network
namespace always be a directory. This is not true if the network
namespace is bind mounted to a file, and will make the plugin fail
erroneously in such cases. The fix is to remove this assumption
completely and just do a basic check on the file system types being
returned.

Fixes #288
 2016-08-31 21:41:00 +00:00
Avinash Sridharan
1ce5e54e99 pkg/ns: fixed the check for network namespace path. 
The expectation on older kernels (< 3.19) was to have the network
namespace always be a directory. This is not true if the network
namespace is bind mounted to a file, and will make the plugin fail
erroneously in such cases. The fix is to remove this assumption
completely and just do a basic check on the file system types being
returned.

Fixes #288
 2016-08-31 21:41:00 +00:00
Dan Williams
4d77bc177c Merge pull request #291 from tomdee/SetupVeth 
pkg/ip: Ensure that SetupVeth returns correct hostVeth
 2016-08-31 16:32:09 -05:00
Tom Denham
31ef82276a pkg/ip: Ensure that SetupVeth returns correct hostVeth 
The veth is moved from the container NS to the host NS.
This is handled by the code that sets the link to UP but the wrong
hostVeth is returned to the calling code.
 2016-08-31 14:17:25 -07:00
Tom Denham
726c6b4578 pkg/ip: Ensure that SetupVeth returns correct hostVeth 
The veth is moved from the container NS to the host NS.
This is handled by the code that sets the link to UP but the wrong
hostVeth is returned to the calling code.
 2016-08-31 14:17:25 -07:00
Gabe Rosenhouse
d3ecadb860 invoke: backfill tests for plugin execution 2016-08-30 23:52:47 -04:00