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

Update Go to 1.17 #494

Merged
merged 3 commits into from
Aug 23, 2021
Merged

Update Go to 1.17 #494

merged 3 commits into from
Aug 23, 2021

Conversation

tklauser
Copy link
Member

Split into multiple commits to separate the version update from the go.mod and vendor update due to the new module graph pruning (see https://golang.org/doc/go1.17#graph-pruning).

Signed-off-by: Tobias Klauser <[email protected]>
Bump the go version in go.mod to 1.17. This will change the module graph
to include only the immediate dependencies of other go 1.17 modules, not
their full transitive dependencies. As a consequence of this, the
newly-added requirements on indirect dependencies are added by `go mod
tidy` in a separate require block.

See https://golang.org/doc/go1.17#graph-pruning and
https://golang.org/ref/mod#graph-pruning for details.

Signed-off-by: Tobias Klauser <[email protected]>
The privileged_tests build tag is only used in cilium/cilium, we can
omit these in cilium/cilium-cli.

Found when running Go 1.17 `gofmt` which added a `//go:build` line.

Signed-off-by: Tobias Klauser <[email protected]>
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 23, 2021
@tklauser tklauser merged commit 698aecb into master Aug 23, 2021
@tklauser tklauser deleted the pr/tklauser/update-go-1.17 branch August 23, 2021 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants