Commit Graph

1007 Commits

Author SHA1 Message Date
73106f0ece Remove unused pidfilePath const
Moved global var's to BeforeEach()
2018-10-09 16:57:58 -04:00
ef913eadd5 Took out os.Remove() for socketPath and pidfilePath now that os.RemoveAll() is used 2018-10-08 16:01:31 -04:00
3d349e4645 Ensure /run/cni exists in the filesystem before running dhcp tests 2018-10-08 14:08:37 -04:00
3de323f3f0 Enable ginkgo parallelization in test.sh 2018-10-08 11:32:08 -04:00
5fd849ac6d Use tempDir in socket path for ginkgo parallelization 2018-10-08 11:31:03 -04:00
6d3215a256 Allow socket path used by dhcp plugin to be supplied via dhcp ipam configuration
Allow socket path to be supplied as flag when starting dhcp daemon
2018-10-08 11:30:01 -04:00
66837d6f3b Remove -p from test.sh
Running ginkgo tests in parallel causes problems with dhcp_test.go.

BeforeEach() is run once for each spec before any actual dhcp test starts.
This results in setting up two dhcp4servers that run concurrently.
Both try to Listen and use unix socketPath file /run/cni/dhcp.sock at the same time.

AfterEach() for one test runs when test completes, deleting /run/cni/dhcp.sock.
But other test still needs the file resulting in test failing.  Often, the next dhcp
test hasn't started yet.  When test does start it waits 15 seconds for dhcp4server to
create /run/cni/dhcp.sock (which has just been deleted) so test fails.

Other times dhcp tests fail because /run/cni/dhcp.sock is deleted while still being used.
2018-10-01 16:12:07 -04:00
1e4d47fc35 Fix vendor regression in dhcp4server 2018-09-27 11:04:14 -04:00
646dbbace1 Merge pull request #165 from s1061123/dev/static-args
Support CNI_ARGS in static IPAM plugin
2018-09-24 21:00:09 -05:00
9b86f52791 Merge pull request #200 from s1061123/fix/tuning-doc
Add description for mac/mtu/promisc in tuning README.md
2018-09-24 20:53:38 -05:00
8a579a7fbc Merge pull request #204 from mrostecki/always-check-err
Add missing error checks
2018-09-24 20:52:38 -05:00
a8ad12dd7a Merge pull request #193 from thxCode/windows_cni
Windows CNI support
2018-09-20 15:16:52 -05:00
b56ca2fe45 Windows Support
Patch for https://github.com/containernetworking/plugins/pull/85

+ Windows cni plugins are added
   (*) win-bridge (hostgw)
   (*) win-overlay (vxlan)
+ Windows netconf unit test
+ Fix appveyor config to run the test
+ Build release support for windows plugins

Address comments

From:
    - https://github.com/containernetworking/plugins/pull/85
    - 0049c64e3f
2018-09-21 00:34:07 +08:00
e1d29e9fe4 Update Vendor
(*) github.com/Microsoft/hcsshim
    (*) golang.org/x/sys
    (*) github.com/x/cyrpto
    (*) github.com/sirupsen/logrus
    (*) github.com/Microsoft/go-winio
    (*) github.com/juju/errors
    (*) github.com/buger/jsonparser
2018-09-21 00:34:07 +08:00
93178bf026 Fix typo. 2018-09-20 22:18:15 +09:00
f5f787057d plugins/meta: Add missing error checks
Signed-off-by: Michal Rostecki <mrostecki@suse.de>
2018-09-20 11:06:50 +02:00
8c61a2c2f3 plugins/main: Add missing error checks
Signed-off-by: Michal Rostecki <mrostecki@suse.de>
2018-09-20 11:06:28 +02:00
1a3f49c7ae plugins/ipam: Add missing error check
Signed-off-by: Michal Rostecki <mrostecki@suse.de>
2018-09-20 11:05:58 +02:00
947a5881fc pkg/ns: Add missing error checks
Signed-off-by: Michal Rostecki <mrostecki@suse.de>
2018-09-20 11:05:15 +02:00
26834c3e63 pkg/ip: Add missing error check
Signed-off-by: Michal Rostecki <mrostecki@suse.de>
2018-09-20 11:04:13 +02:00
d22e75316f Incorporate comments in PR. 2018-09-13 01:49:47 +09:00
094c903932 Incorporate Dan's comments. 2018-09-13 01:46:11 +09:00
61a412ea77 Align test suite name with others 2018-09-06 15:56:40 +09:00
61c136126f Support multiple IP addresses in CNI_ARGS 2018-09-06 15:50:18 +09:00
35b87a34db Merge pull request #191 from dcbw/portmap-panic-fix
portmap: don't panic if listing a chain returns fewer lines than exected
2018-09-05 10:09:02 -05:00
9048a61dda Add description for mac/mtu/promisc in tuning README.md
This diff adds documents for #177 change (mac/mtu/promisc) in tuning
README.md. Fixes #199.
2018-09-05 16:01:24 +09:00
321467bf1b Merge remote-tracking branch 'upstream/master' into dev/static-args 2018-09-05 14:39:16 +09:00
60a99ca331 Incorporate Dan's comments. 2018-09-05 14:37:31 +09:00
7d329215b0 portmap: don't panic if listing a chain returns fewer lines than expected 2018-08-16 09:50:28 -05:00
6a12a375cd Merge pull request #190 from dongjun666/bandwidth_desc
Correct the bandwidth unit in description
2018-08-15 17:00:24 +01:00
220499db6b Correct the bandwidth unit in description
Replace Kbps with bps and Kb with bits in bandwidth description.
2018-08-15 19:26:13 +08:00
5532950dce Merge pull request #186 from ncdc/fix-release-no-tty
scripts: support building releases without a TTY
2018-08-14 13:22:27 -05:00
fb7a24405e Merge pull request #177 from s1061123/tuninig-iplink
tuning: add some generic link settings (MAC, promisc, MTU)
2018-08-14 13:22:10 -05:00
9689522b4f scripts: support building releases without a TTY
This allows systems such as Jenkins, which do not provide a TTY, to run
scripts/release.sh

Signed-off-by: Andy Goldstein <andy.goldstein@gmail.com>
2018-08-09 10:20:40 -04:00
9425d24c28 Incorporate @jelloneck/@bboreham/@squeed's comments. 2018-08-09 22:28:12 +09:00
635cb22f12 Merge remote-tracking branch 'origin/master' into tuninig-iplink 2018-08-09 21:43:33 +09:00
75d3585862 Merge pull request #174 from jellonek/updatecomment
plugins/ipam/static: Update docstring
2018-08-01 16:28:45 +01:00
3a7f254a63 Introduce iplink(MTU MAC and promiscas) feature into tuning
This change adds 'ip link' command related feature into tuning
meta cni plugin. Currently MTU, MAC and promiscas mode are
supported.
2018-07-31 12:54:15 +09:00
2b819b5571 Merge pull request #129 from mauriciovasquezbernal/fix_ip_leakage_bridge
bridge: release IP in case of error
2018-07-27 15:58:08 -05:00
316489903b bridge: add test case for release IP on error
Signed-off-by: Mauricio Vasquez B <mauricio.vasquez@polito.it>
2018-07-27 07:57:12 -05:00
3a7ee332be bridge: release IP in case of error
If there is an error after an IP has been allocated it is necesary
to release it.

Signed-off-by: Mauricio Vasquez B <mauricio.vasquez@polito.it>
2018-07-27 07:57:04 -05:00
cc3ad26691 Incorporate Casey's comments 2018-07-27 12:47:28 +09:00
4b296ba330 bridge: add random datadir to all testcases 2018-07-26 21:09:30 -05:00
1562a1e60e Merge pull request #171 from containernetworking/build-instructions
Add build instructions
2018-07-25 16:30:56 +01:00
6dc16b9132 plugins/ipam/static: Update docstring 2018-07-19 14:35:10 +02:00
21e3e4ec01 Merge pull request #175 from liucimin/fix_loopback_readme
fix_loopback_readme
2018-07-18 09:02:48 -05:00
427a38e2d7 fix_loopback_readme 2018-07-11 23:59:09 +08:00
f970542036 Merge pull request #173 from sak0/pr
remove duplicated assginment.
2018-07-11 16:32:59 +01:00
9201a5a433 Merge pull request #169 from hustcat/bandwidth-burst-fix
Fix tc-tbf burst value in bytes
2018-07-11 16:31:53 +01:00
3b3622db67 Incorporate jellonek's comments. 2018-07-06 13:17:22 +09:00