-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
chore: Update go to v1.23.0 #15766
chore: Update go to v1.23.0 #15766
Conversation
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.
Have min
and max
now become a reserved word or something? 😂
@Hipska yes. Those are builtin functions since go v1.21... |
Thanks so much for the pull request! |
I guess the issue may be related to golangci/golangci-lint#4837 (comment)
|
@mskonovalov the issue is that the CircleCI instance runs out-of-memory unfortunately. In the past we were able to tune the garbage collection params but this doesn't seem to work this time... |
It looks like But I found something else. When I started playing around with By default, Confirmation here: And here: Using only as many CPUs as are actually available to the guest (4 instead of 36) speeds up the jobs (less CPU time spent in garbage collection) and reduces memory usage/makes garbage collection more efficient (with parallel linter threads, using fewer processors allows for a more efficient "stop the world" phase). |
This is really amazing! Thanks for hunting down and fixing the issue @zak-pawel! |
- Add --timeout=30m for golangci-lint (linux only) - Add no_output_timeout: 30m for golangci-lint (linux only) - Fix deprecated reflect.PtrTo (use reflect.PointerTo instead) - Remove exportloopref linter (deprecated) - Install [email protected] (instead of [email protected]) in Makefile - Add few //nolint's
- Get back to GOGC=80 GOMEMLIMIT=6656MiB (from: GOGC=70 GOMEMLIMIT=5120MiB)
…trange, because Docker.Large should have 4 - check this)
…ndows, remove `madvdontneed=1` from Linux
Download PR build artifacts for linux_amd64.tar.gz, darwin_arm64.tar.gz, and windows_amd64.zip. 🥳 This pull request decreases the Telegraf binary size by -1.27 % for linux amd64 (new size: 260.7 MB, nightly size 264.0 MB) 📦 Click here to get additional PR build artifactsArtifact URLs |
Co-authored-by: Pawel Zak <[email protected]>
Summary
Checklist
Related issues
resolves #15746