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 #3410

Merged
merged 1 commit into from
Oct 24, 2023
Merged

[main] Upgrade to latest dependencies #3410

merged 1 commit into from
Oct 24, 2023

Conversation

knative-automation
Copy link
Contributor

@knative-automation knative-automation commented Oct 19, 2023 •
edited
Loading

Cron -knative-prow-robot

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

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

@knative-automation knative-automation force-pushed the auto-updates/update-deps-main branch from 4285764 to 940a4b1 Compare October 19, 2023 02:10
@knative-prow
Copy link

knative-prow bot commented Oct 19, 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 knative-prow bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 19, 2023
@codecov
Copy link

codecov bot commented Oct 19, 2023 •
edited
Loading

Codecov Report

Merging #3410 (5857c11) into main (57c4d1e) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #3410   +/-   ##
=========================================
  Coverage     61.55%   61.56%           
- Complexity      761      762    +1     
=========================================
  Files           181      181           
  Lines         12396    12396           
  Branches        265      265           
=========================================
+ Hits           7630     7631    +1     
  Misses         4163     4163           
+ Partials        603      602    -1
Flag Coverage Δ
java-unittests 70.88% <ø> (+0.03%) ⬆️

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

see 1 file with indirect coverage changes

@matzew
Copy link
Contributor

matzew commented Oct 19, 2023

/retest

@matzew
Copy link
Contributor

matzew commented Oct 19, 2023

/test channel-integration-tests-sasl-plain

matzew
matzew approved these changes Oct 19, 2023
View reviewed changes
Copy link
Contributor

@matzew matzew 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 Oct 19, 2023
@creydr
Copy link
Contributor

creydr commented Oct 19, 2023

/retest

1 similar comment
@creydr
Copy link
Contributor

creydr commented Oct 19, 2023

/retest

@knative-automation knative-automation force-pushed the auto-updates/update-deps-main branch from 940a4b1 to c861140 Compare October 19, 2023 13:53
@knative-prow knative-prow bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 19, 2023
@creydr creydr mentioned this pull request Oct 19, 2023
Merged
@creydr
Copy link
Contributor

creydr commented Oct 19, 2023

#3412 fixes the build issues

@knative-automation knative-automation force-pushed the auto-updates/update-deps-main branch from c861140 to 34c4c27 Compare October 20, 2023 02:06
@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 Oct 20, 2023
@creydr
Copy link
Contributor

creydr commented Oct 20, 2023

/retest

creydr
creydr reviewed Oct 20, 2023
View reviewed changes
go.mod Outdated Show resolved Hide resolved
@knative-automation knative-automation force-pushed the auto-updates/update-deps-main branch from 34c4c27 to 13a20c0 Compare October 23, 2023 02:02
@creydr
Copy link
Contributor

creydr commented Oct 23, 2023

/retest

1 similar comment
@creydr
Copy link
Contributor

creydr commented Oct 23, 2023

/retest

@creydr
Copy link
Contributor

creydr commented Oct 23, 2023

/retest-required

@creydr
Copy link
Contributor

creydr commented Oct 23, 2023

/hold
not sure about #3410 (comment)

@knative-prow knative-prow bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 23, 2023
@creydr creydr mentioned this pull request Oct 23, 2023
Closed
@knative-prow knative-prow bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 23, 2023
@knative-prow knative-prow bot added the lgtm Indicates that a PR is ready to be merged. label Oct 23, 2023
@creydr
Copy link
Contributor

creydr commented Oct 23, 2023

/hold

@knative-prow knative-prow bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 23, 2023
@creydr
Copy link
Contributor

creydr commented Oct 23, 2023

/unhold

@knative-prow knative-prow bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 23, 2023
@knative-automation knative-automation force-pushed the auto-updates/update-deps-main branch from 13a20c0 to db7c625 Compare October 23, 2023 13:42
@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 23, 2023
Cali0707
Cali0707 approved these changes Oct 23, 2023
View reviewed changes
Copy link
Member

@Cali0707 Cali0707 left a comment

Choose a reason for hiding this comment

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

/lgtm

@knative-prow knative-prow bot added the lgtm Indicates that a PR is ready to be merged. label Oct 23, 2023
@creydr
Copy link
Contributor

creydr commented Oct 23, 2023

/retest

1 similar comment
@creydr
Copy link
Contributor

creydr commented Oct 23, 2023

/retest

@creydr
Copy link
Contributor

creydr commented Oct 23, 2023

/test reconciler-tests

@knative-automation knative-automation force-pushed the auto-updates/update-deps-main branch from db7c625 to adc630e Compare October 23, 2023 16:54
@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 23, 2023
@knative-automation
upgrade to latest dependencies
5857c11 
bumping knative.dev/eventing 16d75a9...b5fd264:
  > b5fd264 Shell executor logs through testing.T in upgrade tests (# 7367)
  > 5848584 [main] Upgrade to latest dependencies (# 7388)
  > 16a3986 Don't override default values when applying partial features.yaml configmap  (# 7379)
  > 4d14c21 Added test for multiple sinks, multiple triggers (# 7350)
  > a261e06 OIDC - Support auto generation of PingSource identity service account and expose in AuthStatus (# 7344)
bumping knative.dev/pkg d6ab729...29775d7:
  > 29775d7 [release-1.12] [CVE-2023-44487] Disable http2 for webhooks (# 2876)
bumping knative.dev/reconciler-test 317033b...e3a9c2e:
  > e3a9c2e Improve error message when deleting resources (# 617)

Signed-off-by: Knative Automation <[email protected]>
@knative-automation knative-automation force-pushed the auto-updates/update-deps-main branch from adc630e to 5857c11 Compare October 24, 2023 02:04
creydr
creydr approved these changes Oct 24, 2023
View reviewed changes
Copy link
Contributor

@creydr creydr left a comment

Choose a reason for hiding this comment

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

/lgtm

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

knative-prow bot commented Oct 24, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Cali0707, creydr, knative-automation, matzew

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:
  • OWNERS [Cali0707,creydr,matzew]

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 merged commit 1643cf3 into knative-extensions:main Oct 24, 2023
19 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pierDipi pierDipi pierDipi left review comments

@Cali0707 Cali0707 Cali0707 approved these changes

@creydr creydr creydr approved these changes

@matzew matzew matzew approved these changes

Assignees

@matzew matzew

@creydr creydr

@Cali0707 Cali0707

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@knative-automation @matzew @creydr @pierDipi @Cali0707