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

[main] Upgrade to latest dependencies #14588

Conversation

knative-automation
Copy link
Contributor

Cron -knative-prow-robot

/cc knative/serving-writers
/assign knative/serving-writers

Produced by: knative-extensions/knobots/actions/update-deps

@knative-prow knative-prow bot requested a review from a team November 3, 2023 01:29
@knative-prow knative-prow bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Nov 3, 2023
Copy link

codecov bot commented Nov 3, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a72f181) 86.01% compared to head (5f467f7) 85.99%.

❗ Current head 5f467f7 differs from pull request most recent head c300177. Consider uploading reports for the commit c300177 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #14588      +/-   ##
==========================================
- Coverage   86.01%   85.99%   -0.03%     
==========================================
  Files         197      197              
  Lines       14916    14916              
==========================================
- Hits        12830    12827       -3     
- Misses       1775     1779       +4     
+ Partials      311      310       -1     

see 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@ReToCode ReToCode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@knative-prow knative-prow bot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Nov 3, 2023
@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 3, 2023
@nak3
Copy link
Contributor

nak3 commented Nov 6, 2023

/retest
/lgtm
/approve

@knative-prow knative-prow bot added the lgtm Indicates that a PR is ready to be merged. label Nov 6, 2023
Copy link

knative-prow bot commented Nov 6, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: knative-automation, nak3, ReToCode

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 6, 2023
Copy link

knative-prow bot commented Nov 6, 2023

New changes are detected. LGTM label has been removed.

@knative-prow knative-prow bot added area/networking size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 6, 2023
@ReToCode
Copy link
Member

ReToCode commented Nov 6, 2023

/retest

@ReToCode
Copy link
Member

ReToCode commented Nov 6, 2023

There seems to be a difference in the behaviour of gorilla in the new version:

--- FAIL: TestServerShutdown (0.00s)
    server_test.go:142: CloseError not received
FAIL

bumping google.golang.org/api 5e4c19e...0d3b865:
  > 0d3b865 chore(main): release 0.149.0 (# 2230)
  > c84e6ff feat(all): auto-regenerate discovery clients (# 2242)
  > e5a53db chore(all): update all (# 2233)
  > 576f5e1 feat(all): auto-regenerate discovery clients (# 2241)
  > 3151bd7 feat(all): auto-regenerate discovery clients (# 2240)
  > cf09469 feat(all): auto-regenerate discovery clients (# 2239)
  > 8d66391 feat(all): auto-regenerate discovery clients (# 2238)
  > 3cc10bf feat(all): auto-regenerate discovery clients (# 2237)
  > da38ebd feat(all): auto-regenerate discovery clients (# 2236)
  > cc28ceb feat(all): auto-regenerate discovery clients (# 2235)
  > 8bb5867 feat(all): auto-regenerate discovery clients (# 2234)
  > a1fb230 feat(all): auto-regenerate discovery clients (# 2232)
  > 9835f38 feat(all): auto-regenerate discovery clients (# 2231)
  > 199783c feat(all): auto-regenerate discovery clients (# 2229)
bumping knative.dev/hack d4af42e...0d99b7e:
  > 0d99b7e add function to set latest semver to latest (# 339)
bumping knative.dev/networking 3bd0f7d...18529fd:
  > 18529fd upgrade to latest dependencies (# 888)
bumping github.com/gorilla/websocket 9111bb8...ac0789b:
  > ac0789b update GitHub workflows (# 857)
  > 78c3487 update golang.org/x/net (# 856)
  > 666c197  Update go version & add verification/testing tools (# 840)
  > 8039329 Correct way to save memory using write buffer pool and freeing net.http default buffers (# 761)
  > 8983b96 Merge pull request # 839 from gorilla/coreydaley-patch-1
  > 76ecc29 archive mode
  > 931041c Update README.md
  > af47554 check error before GotConn for trace
  > bc7ce89 Check for and report bad protocol in TLSClientConfig.NextProtos (# 788)
  > 27d91a9 drop the versions of go that are no longer supported + add 1.18 to ci
  > 78cf1bc Changed the method name UnderlyingConn to NetConn to align the methods names with Go 1.18 standard library (# 773)
  > 69d0eb9 Add check for Sec-WebSocket-Key header (# 752)
bumping knative.dev/pkg a6c730f...acf0a2d:
  > acf0a2d Bump google.golang.org/api from 0.148.0 to 0.149.0 (# 2891)
  > 5e1261a Bump cloud.google.com/go/storage from 1.33.0 to 1.34.1 (# 2890)
  > 3b3f4e4 Bump github.com/gorilla/websocket from 1.5.0 to 1.5.1 (# 2889)
  > 8bf0f54 Bump golang.org/x/sync from 0.4.0 to 0.5.0 (# 2888)
  > fac3a4f fix closure variable in filtered factory (# 2884)
bumping google.golang.org/genproto d307bd8...49dd2c1:
  > 49dd2c1 chore(all): update all (# 1067)
  > e917dd1 chore(deps): bump golang.org/x/net in /googleapis/api/servicecontrol (# 1063)
  > 8102cb6 chore(deps): bump golang.org/x/net in /googleapis/api/servicemanagement (# 1060)
  > baa7128 chore(deps): bump golang.org/x/net in /googleapis/api/apikeys (# 1062)
  > 67c69aa chore(deps): bump golang.org/x/net in /googleapis/bytestream (# 1061)
  > e298162 chore(deps): bump golang.org/x/net in /googleapis/api/serviceusage (# 1064)
  > 378ed82 chore(deps): bump golang.org/x/net in /googleapis/api (# 1065)
  > a6244dc chore(deps): bump golang.org/x/net from 0.14.0 to 0.17.0 (# 1066)
  > 8bfb1ae chore(all): update all (# 1059)
  > ceb4ed7 chore(all): auto-regenerate .pb.go files (# 1058)
bumping google.golang.org/genproto/googleapis/rpc e917dd1...49dd2c1:
  > 49dd2c1 chore(all): update all (# 1067)
bumping github.com/googleapis/enterprise-certificate-proxy a110527...d928958:
  > d928958 fix: Remove assertion for private key decryption support during pkcs11 init (# 109)
  > 5192b8e feat: Add KeyType API. (# 105)
  > bd11497 fix: Update windows github action to use go 1.20 (# 100)
  > 02fb4ca doc: More certificate config file path options (# 99)
bumping google.golang.org/genproto/googleapis/api d307bd8...49dd2c1:
  > 49dd2c1 chore(all): update all (# 1067)
  > e917dd1 chore(deps): bump golang.org/x/net in /googleapis/api/servicecontrol (# 1063)
  > 8102cb6 chore(deps): bump golang.org/x/net in /googleapis/api/servicemanagement (# 1060)
  > baa7128 chore(deps): bump golang.org/x/net in /googleapis/api/apikeys (# 1062)
  > 67c69aa chore(deps): bump golang.org/x/net in /googleapis/bytestream (# 1061)
  > e298162 chore(deps): bump golang.org/x/net in /googleapis/api/serviceusage (# 1064)
  > 378ed82 chore(deps): bump golang.org/x/net in /googleapis/api (# 1065)
  > a6244dc chore(deps): bump golang.org/x/net from 0.14.0 to 0.17.0 (# 1066)
  > 8bfb1ae chore(all): update all (# 1059)
  > ceb4ed7 chore(all): auto-regenerate .pb.go files (# 1058)

Signed-off-by: Knative Automation <[email protected]>
Copy link

knative-prow bot commented Nov 7, 2023

@knative-automation: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
unit-tests_serving_main c300177 link true /test unit-tests

Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@nak3
Copy link
Contributor

nak3 commented Nov 7, 2023

Yes, it seems SetCloseHandler changed to handle a returned error from WriteControl():
https://github.com/gorilla/websocket/blob/666c197fc9157896b57515c3a3326c3f8c8319fe/conn.go#L1166-L1168
and our test started failing.

I think we should fix upstream gorilla websocket but maybe we should pin to gorilla version to 1.15.0 in go.mod until we can fix it?

@ReToCode
Copy link
Member

ReToCode commented Nov 7, 2023

I think we should fix upstream gorilla websocket but maybe we should pin to gorilla version to 1.15.0 in go.mod until we can fix it?

Yes let's do that 👍

@nak3
Copy link
Contributor

nak3 commented Nov 7, 2023

Thank you. I will do that soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/networking size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants