Closed thaJeztah closed 1 year ago
This looks odd; running go mod tidy does not produce a diff for me, but I'm wondering if it's either trying to use the code not as a separate module. Current run works, but previous one showed this;
Error: /home/runner/go/pkg/mod/github.com/containerd/ttrpc@v1.1.1-0.20220420014843-944ef4a40df3/client.go:30:2: missing go.sum entry for module providing package github.com/sirupsen/logrus (imported by github.com/containerd/nri/plugins/template); to add:
go get github.com/containerd/nri/plugins/template
Error: /home/runner/go/pkg/mod/github.com/containerd/ttrpc@v1.1.1-0.20220420014843-944ef4a40df3/unixcreds_linux.go:27:2: missing go.sum entry for module providing package golang.org/x/sys/unix (imported by github.com/containerd/ttrpc); to add:
go get github.com/containerd/ttrpc@v1.1.1-0.20220420014843-944ef4a40df3
make: *** [Makefile:113: /home/runner/work/nri/nri/build/bin/template] Error 1
Error: Process completed with exit code 2.
Or... something very fishy (full output below, or here: https://pipelinesghubeus24.actions.githubusercontent.com/nAcqJVvhK0M1GSKfg9ZtIaPG34KOIcpFPpt533m0kSjW9jw8oI/_apis/pipelines/1/runs/229/signedlogcontent/2?urlExpires=2023-09-13T08%3A44%3A19.0508573Z&urlSigningMethod=HMACV1&urlSignature=APJ8a64QufPmeEyDpd6xk4WRCZm0HNjTtEv6SThju%2BM%3D);
This project no longer uses vendoring, but the logs output shows multiple vendored packages? Is state not cleaned between runs or something odd?
2023-09-13T08:09:13.5856109Z crypto/dsa
2023-09-13T08:09:13.6241118Z encoding/hex
2023-09-13T08:09:13.6566336Z encoding/pem
2023-09-13T08:09:13.6672236Z crypto/x509/pkix
2023-09-13T08:09:13.7013051Z vendor/golang.org/x/crypto/chacha20
2023-09-13T08:09:13.7225338Z crypto/x509
2023-09-13T08:09:13.7730050Z vendor/golang.org/x/crypto/internal/poly1305
2023-09-13T08:09:13.8461302Z vendor/golang.org/x/sys/cpu
2023-09-13T08:09:13.8994349Z vendor/golang.org/x/crypto/chacha20poly1305
2023-09-13T08:09:14.0305663Z vendor/golang.org/x/crypto/hkdf
2023-09-13T08:09:14.0609146Z vendor/golang.org/x/text/transform
2023-09-13T08:09:14.1358728Z vendor/golang.org/x/text/unicode/bidi
2023-09-13T08:09:14.4065205Z vendor/golang.org/x/text/secure/bidirule
2023-09-13T08:09:14.4425583Z vendor/golang.org/x/text/unicode/norm
2023-09-13T08:09:14.4813185Z crypto/tls
2023-09-13T08:09:14.9576855Z vendor/golang.org/x/net/idna
2023-09-13T08:09:15.3306442Z net/textproto
2023-09-13T08:09:15.4941144Z vendor/golang.org/x/net/http/httpguts
2023-09-13T08:09:15.5702534Z vendor/golang.org/x/net/http/httpproxy
2023-09-13T08:09:15.6687744Z vendor/golang.org/x/net/http2/hpack
2023-09-13T08:09:15.8020145Z mime
2023-09-13T08:09:15.9956854Z mime/quotedprintable
Latest build failed because somehow, somewhere it (maybe cached again?) didn't pick up that the plugin is a separate module, so tried to build it as part of the main module;
Run make install-ginkgo test codecov
make install-ginkgo test codecov
shell: /usr/bin/bash -e {0}
env:
GOPATH: /home/runner/work/nri/nri
go: downloading github.com/containerd/ttrpc v1.1.1-0.20220420014843-944ef4a40df3
go: downloading golang.org/x/sys v0.1.0
go: downloading github.com/opencontainers/runtime-spec v1.0.3-0.20220825212826-86290f6a00fb
go: downloading github.com/sirupsen/logrus v1.8.1
go: downloading github.com/moby/sys/mountinfo v0.6.2
go: downloading github.com/opencontainers/runtime-tools v0.0.0-20221026201742-946c877fa809
go: downloading google.golang.org/protobuf v1.28.1
go: downloading k8s.io/cri-api v0.25.3
go: downloading google.golang.org/genproto v0.0.0-20220502173005-c8bf987b8c21
go: downloading google.golang.org/grpc v1.47.0
go: downloading github.com/gogo/protobuf v1.3.2
go: downloading github.com/syndtr/gocapability v0.0.0-20200815063812-42c35b437635
go: downloading golang.org/x/net v0.1.0
go: downloading github.com/golang/protobuf v1.5.2
go: downloading golang.org/x/text v0.4.0
go: downloading github.com/onsi/ginkgo/v2 v2.5.0
go: downloading github.com/go-task/slim-sprig v0.0.0-20210107165309-348f09dbbbc0
go: downloading golang.org/x/tools v0.2.0
go: downloading github.com/google/pprof v0.0.0-20210407192527-94a9f03dee38
Will skip:
./plugins/ulimit-adjuster
go: downloading github.com/onsi/gomega v1.24.0
go: downloading sigs.k8s.io/yaml v1.3.0
go: downloading github.com/go-logr/logr v1.2.3
go: downloading gopkg.in/yaml.v2 v2.4.0
go: downloading github.com/google/go-cmp v0.5.9
go: downloading gopkg.in/yaml.v3 v3.0.1
[1694595045] NRI Runtime - 81/81 specs ••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••• SUCCESS! 533.630306ms PASS
coverage: 78.5% of statements
go: downloading github.com/stretchr/testify v1.8.0
go: downloading github.com/pmezard/go-difflib v1.0.0
go: downloading github.com/davecgh/go-spew v1.1.1
[1694595045] Net pre-connected Connections and SocketPair - 0/0 specs SUCCESS! 258.505µs PASS
coverage: 59.1% of statements
[1694595045] Connection Multiplexer - 12/12 specs •••••••••••• SUCCESS! 50.669488ms PASS
coverage: 72.0% of statements
[1694595045] Generate Suite - 10/10 specs •••••••••• SUCCESS! 3.368264ms PASS
coverage: 38.3% of statements
composite coverage: 70.9% of statements
Ginkgo ran 4 suites in 32.652418511s
Test Suite Passed
main module (github.com/containerd/nri) does not contain package github.com/containerd/nri/plugins/ulimit-adjuster
make: *** [Makefile:139: test-ulimits] Error 1
Error: Process completed with exit code 2.
Oh, maybe I see; makefile needs a new entry because it's a separate module now. Let me fix
Alright; all green now! I can move the first commit to a separate PR for visibility; also did a quick try in containerd if everything looks good, and so far (CI still running) I think it does;
And opened a separate PR for the first commit;
@klihub @samuelkarp I moved this one out of draft 👍
Happy to do follow-ups where needed! I honestly am not familiar with this module (at all!) and my changes were really just to simplify containerd's complex dependency tree (and to cut circular references where possible)
Happy to do follow-ups where needed! I honestly am not familiar with this module (at all!) and my changes were really just to simplify containerd's complex dependency tree (and to cut circular references where possible)
Sorry if my comment was confusing. I think these changes are fine.
no worries! I think I understood it was not a blocker, so no harm done!
Codecov Report
Patch coverage has no change and project coverage change:
-0.11%
:warning:Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #51 +/- ## ========================================== - Coverage 64.61% 64.50% -0.11% ========================================== Files 9 9 Lines 1834 1834 ========================================== - Hits 1185 1183 -2 - Misses 498 500 +2 Partials 151 151 ``` [see 1 file with indirect coverage changes](https://app.codecov.io/gh/containerd/nri/pull/51/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=containerd):umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.