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
run go.mod tidy - drop compat with 1.16 #12911
Conversation
go mod tidy -go=1.16 && go mod tidy -go=1.17 then go mod tidy -compat=1.17 This was causing our downstream-test in knative.dev/pkg to fail since go mod tidy was existing with an error code 1 Example output ``` knative.dev/serving/pkg/queue imports github.com/prometheus/client_golang/prometheus imports github.com/prometheus/procfs loaded from github.com/prometheus/procfs@v0.6.0, but go 1.16 would select v0.7.3 knative.dev/serving/cmd/activator imports go.uber.org/zap tested by go.uber.org/zap.test imports go.uber.org/goleak loaded from go.uber.org/goleak@v1.1.11-0.20210813005559-691160354723, but go 1.16 would select v1.1.12 knative.dev/serving/pkg/queue imports github.com/prometheus/client_golang/prometheus imports github.com/prometheus/procfs imports github.com/prometheus/procfs/internal/fs loaded from github.com/prometheus/procfs@v0.6.0, but go 1.16 would select v0.7.3 knative.dev/serving/pkg/queue imports github.com/prometheus/client_golang/prometheus imports github.com/prometheus/procfs imports github.com/prometheus/procfs/internal/util loaded from github.com/prometheus/procfs@v0.6.0, but go 1.16 would select v0.7.3 knative.dev/serving/cmd/activator imports go.uber.org/zap tested by go.uber.org/zap.test imports go.uber.org/goleak imports go.uber.org/goleak/internal/stack loaded from go.uber.org/goleak@v1.1.11-0.20210813005559-691160354723, but go 1.16 would select v1.1.12 knative.dev/serving/hack imports github.com/tsenart/vegeta/v12 imports github.com/tsenart/vegeta/v12/internal/resolver tested by github.com/tsenart/vegeta/v12/internal/resolver.test imports github.com/miekg/dns loaded from github.com/miekg/dns@v1.1.35, but go 1.16 would select v1.1.41 ```
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dprotaso The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold surprised to see a |
Codecov Report
@@ Coverage Diff @@
## main #12911 +/- ##
=======================================
Coverage 87.04% 87.04%
=======================================
Files 197 197
Lines 14437 14437
=======================================
Hits 12567 12567
Misses 1576 1576
Partials 294 294 Continue to review full report at Codecov.
|
Removed it here: prometheus/procfs#447 in the meantime |
do you need to wait for that? |
/lgtm |
go mod tidy -go=1.16 && go mod tidy -go=1.17
then
go mod tidy -compat=1.17
This was causing our downstream-test in
knative.dev/pkg
to failsince go mod tidy was exiting with an error code 1
Example output