New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Split out cleanups from #10806 #10823
Conversation
test-me-please |
test-with-kernel |
restart-ginkgo |
The large list of files unnecessarily clutters the log. Just print the name of the check. Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Get rid of some superflous horizontal spacing in the rst source. Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Dockerfile.builder
Outdated
cd /go/src/github.com/gordonklaus/ineffassign && \ | ||
git checkout -b 1003c8bd00dc2869cb5ca5282e6ce33834fed514 1003c8bd00dc2869cb5ca5282e6ce33834fed514 && \ | ||
go install | ||
GO111MODULE=on go get github.com/gordonklaus/ineffassign@1003c8bd00dc2869cb5ca5282e6ce33834fed514 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we might need to update the docker image tags in this PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. I only updated it for #10806 (cilium-builder:2020-04-01
).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Image is currently building and I updated the tag in Dockerfile
to cilium-builder:2020-04-02
. While at it I also noticed that we can reduce the image size a bit by running go clean -cache -modcache
after go install
to get rid of the build and module download caches.
…ssign Also clean the Go build and module download caches to reduce the container image size. Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
940f2e1
to
ab4ffca
Compare
restart-ginkgo |
test-with-kernel |
test-docs-please |
Cleanup commits split out of #10806
See individual commit messages.