Skip to content
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

Use "go 1.18" in go.mod files #3456

Closed
wants to merge 2 commits into from
Closed

Conversation

kzys
Copy link
Contributor

@kzys kzys commented Nov 2, 2022

We don't have reasons to use "go 1.12". The agent itself and the dependencies wouldn't work with the version.

Go 1.18 is picked since it is the latest Go version in Amazon Linux 2.

https://go.dev/ref/mod#go-mod-file

Signed-off-by: Kazuyoshi Kato [email protected]

Summary

Use go 1.18 in go.mod files.

Implementation details

  • Updated/added go 1.18 in go.mod files
  • Run go mod tidy and go mod vendor
  • Unpin x/sys to have unix.BPF_STATS_RUN_TIME
  • Upgrade github.com/docker/docker to use the newer version of x/sys

Testing

New tests cover the changes: no

Description for the changelog

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

We don't have reasons to use "go 1.12". The agent itself and
the dependencies wouldn't work with the version.

Go 1.18 is picked since it is the latest Go version in Amazon Linux 2.

https://go.dev/ref/mod#go-mod-file

Signed-off-by: Kazuyoshi Kato <[email protected]>
@kzys kzys requested a review from a team as a code owner November 2, 2022 16:58
@kzys kzys force-pushed the go-version-in-mod branch from 301bfc8 to 4dd65be Compare November 2, 2022 17:10
@kzys kzys marked this pull request as draft November 2, 2022 17:46
@kzys kzys force-pushed the go-version-in-mod branch 3 times, most recently from 04fd7f4 to 612bf98 Compare November 2, 2022 18:12
@kzys kzys force-pushed the go-version-in-mod branch from 612bf98 to cacb089 Compare November 2, 2022 18:21
@sparrc
Copy link
Contributor

sparrc commented Feb 2, 2023

closing as we have bumped to 1.19 in #3551, once we bump the docker library version I will get to #3458

@sparrc sparrc closed this Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants