containerd / go-cni

A generic CNI library to provide APIs for CNI plugin interactions
Apache License 2.0
146 stars 56 forks source link

Add integration test for linux and update go version from 1.16 to 1.17 #84

Closed fuweid closed 2 years ago

fuweid commented 2 years ago

Signed-off-by: Wei Fu fuweid89@gmail.com

codecov-commenter commented 2 years ago

Codecov Report

Merging #84 (41d9753) into main (e559bd8) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #84   +/-   ##
=======================================
  Coverage   44.41%   44.41%           
=======================================
  Files           9        9           
  Lines         403      403           
=======================================
  Hits          179      179           
  Misses        190      190           
  Partials       34       34           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e559bd8...41d9753. Read the comment docs.

fuweid commented 2 years ago

@dmcgowan @AkihiroSuda Updated. PTAL. Thanks!

fuweid commented 2 years ago

Thanks Mike. I can remove coverage. But I don't quite understand the second part nit :joy: I will discuss with you when you are back. :)

fuweid commented 2 years ago

Paste the result:

go test -v -race  -count=1 ./...
go: downloading github.com/stretchr/testify v1.6.1
go: downloading github.com/stretchr/objx v0.1.0
=== RUN   TestLibCNIType020
--- PASS: TestLibCNIType020 (0.01s)
=== RUN   TestLibCNIType040
--- PASS: TestLibCNIType040 (0.00s)
=== RUN   TestLibCNIType100
--- PASS: TestLibCNIType100 (0.01s)
PASS
ok      github.com/containerd/go-cni    0.043s
bin/integration.test -test.v -test.count=1 -test.root  -test.parallel 8
=== RUN   TestBasicSetupAndRemove
    cni_setup_teardown_linux_test.go:168: [0] ip is 10.[88](https://github.com/containerd/go-cni/runs/5305913820?check_suite_focus=true#step:5:88).0.2
    cni_setup_teardown_linux_test.go:168: [1] ip is 10.88.0.3
--- PASS: TestBasicSetupAndRemove (3.63s)
PASS