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.

Sign up for GitHub

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

Jump to bottom

[main] Upgrade to latest dependencies #3291

Closed
wants to merge 1 commit into from
Closed

[main] Upgrade to latest dependencies #3291

wants to merge 1 commit into from

Conversation

knative-automation
Copy link
Contributor

Cron -knative-prow-robot

/cc knative-extensions/eventing-writers
/assign knative-extensions/eventing-writers

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

@knative-prow
Copy link

knative-prow bot commented Aug 17, 2023

@knative-automation: GitHub didn't allow me to request PR reviews from the following users: knative-extensions/eventing-writers.

Note that only knative-extensions members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

Cron -knative-prow-robot

/cc knative-extensions/eventing-writers
/assign knative-extensions/eventing-writers

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

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.

@knative-prow
Copy link

knative-prow bot commented Aug 17, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: knative-automation
Once this PR has been reviewed and has the lgtm label, please assign skonto for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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 added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Aug 17, 2023
@codecov
Copy link

codecov bot commented Aug 17, 2023 •
edited
Loading

Codecov Report

Merging #3291 (03ebee9) into main (c7aa08e) will increase coverage by 0.00%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #3291   +/-   ##
=========================================
  Coverage     61.70%   61.71%           
  Complexity      769      769           
=========================================
  Files           182      182           
  Lines         12312    12312           
  Branches        268      268           
=========================================
+ Hits           7597     7598    +1     
  Misses         4121     4121           
+ Partials        594      593    -1
Flag Coverage Δ
java-unittests 71.05% <ø> (-0.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 3 files with indirect coverage changes

@knative-prow knative-prow bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 18, 2023
@knative-prow knative-prow bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Aug 21, 2023
@knative-prow knative-prow bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 21, 2023
@knative-automation knative-automation force-pushed the auto-updates/update-deps-main branch 3 times, most recently from 8d10543 to d69b997 Compare August 23, 2023 14:03
@matzew
Copy link
Contributor

matzew commented Aug 24, 2023

/retest

@knative-automation knative-automation force-pushed the auto-updates/update-deps-main branch 4 times, most recently from 0f44677 to 6837bb4 Compare August 28, 2023 02:23
@knative-automation
upgrade to latest dependencies
03ebee9 
bumping google.golang.org/api 113082d...85e7e70:
  > 85e7e70 chore(main): release 0.138.0 (# 2116)
  > 9b75278 feat(all): auto-regenerate discovery clients (# 2119)
  > 40ea606 feat(all): auto-regenerate discovery clients (# 2118)
  > 1770219 feat(all): auto-regenerate discovery clients (# 2115)
  > c631c0a chore(main): release 0.137.0 (# 2107)
  > caea956 feat: ok to use S2A with override endpoint (# 2114)
  > 142ffeb feat(all): auto-regenerate discovery clients (# 2113)
  > f7f1c7b feat(all): auto-regenerate discovery clients (# 2112)
  > 0cc62ab feat(all): auto-regenerate discovery clients (# 2111)
  > 4d775db feat(all): auto-regenerate discovery clients (# 2110)
  > 6d29bab chore(all): update module github.com/google/s2a-go to v0.1.5 (# 2109)
  > 80485e0 feat(all): auto-regenerate discovery clients (# 2108)
  > 3f3ed3d feat(all): auto-regenerate discovery clients (# 2106)

Signed-off-by: Knative Automation <automation@knative.team>
@knative-prow
Copy link

knative-prow bot commented Aug 28, 2023

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

Test name Commit Details Required Rerun command
reconciler-tests-loom_eventing-kafka-broker_main 03ebee9 link false /test reconciler-tests-loom
integration-tests_eventing-kafka-broker_main 03ebee9 link true /test integration-tests
reconciler-tests_eventing-kafka-broker_main 03ebee9 link true /test reconciler-tests
channel-reconciler-tests-sasl-ssl_eventing-kafka-broker_main 03ebee9 link true /test channel-reconciler-tests-sasl-ssl
reconciler-tests-namespaced-broker-loom_eventing-kafka-broker_main 03ebee9 link false /test reconciler-tests-namespaced-broker-loom
channel-reconciler-tests-ssl_eventing-kafka-broker_main 03ebee9 link true /test channel-reconciler-tests-ssl
channel-integration-tests-ssl_eventing-kafka-broker_main 03ebee9 link true /test channel-integration-tests-ssl
channel-integration-tests-sasl-plain_eventing-kafka-broker_main 03ebee9 link true /test channel-integration-tests-sasl-plain
reconciler-tests-namespaced-broker_eventing-kafka-broker_main 03ebee9 link true /test reconciler-tests-namespaced-broker
channel-integration-tests-sasl-ssl_eventing-kafka-broker_main 03ebee9 link true /test channel-integration-tests-sasl-ssl
channel-reconciler-tests-sasl-plain_eventing-kafka-broker_main 03ebee9 link true /test channel-reconciler-tests-sasl-plain

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.

@matzew matzew mentioned this pull request Aug 28, 2023
Closed
@matzew
Copy link
Contributor

matzew commented Aug 28, 2023

/close

@knative-prow knative-prow bot closed this Aug 28, 2023
@knative-prow
Copy link

knative-prow bot commented Aug 28, 2023

@matzew: Closed this PR.

In response to this:

/close

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.

@knative-prow-robot knative-prow-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 28, 2023
@knative-prow-robot
Copy link
Contributor

PR needs rebase.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@knative-automation @matzew @knative-prow-robot