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

[Automated] Update eventing-kafka-broker-eventing nightly #3405

Merged
merged 1 commit into from
Oct 24, 2023
Merged

[Automated] Update eventing-kafka-broker-eventing nightly #3405

merged 1 commit into from
Oct 24, 2023

Conversation

knative-automation
Copy link
Contributor

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

/assign @knative-extensions/eventing-wg-leads
/cc @knative-extensions/eventing-wg-leads

Produced via:

for x in eventing-crds.yaml eventing-core.yaml eventing-tls-networking.yaml; do
  curl https://storage.googleapis.com/knative-nightly/eventing/latest/$x > ${GITHUB_WORKSPACE}/./third_party/eventing-latest/$x
done

Details:

bumping knative.dev/eventing 9de5275...16a3986:
  > 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)
  > 16d75a9 mt-broker-ingress check audience of received token (# 7336)
  > 3ef831c Increase expiration buffer time of OIDCTokenProviders cache to 5 minutes (# 7355)
  > 5168dc3 Update dependencies (# 7378)
  > f9314d8 [main] Upgrade to latest dependencies (# 7372)
  > 4a07fdc Move OIDC e2e tests to test/auth (# 7363)
  > 0933c11 Expose audience of an InMemoryChannel in its status (# 7371)
  > 169542f OIDC - Support auto generation of ApiServerSource identity service account and expose in AuthStatus (# 7330)
  > 62ec33b Support auto generation of Subscriptions identity service account and… (# 7338)
  > 0af3ef6 [main] Upgrade to latest dependencies (# 7370)
  > cf6b140 [main] Update community files (# 7369)
  > de6915a Pass sink namespace to PingSource (# 7368)
  > 6dca8c4 [main] Upgrade to latest dependencies (# 7366)
  > 082d90a Remove unneeded configmapinformer in sinkbinding controller (# 7357)
  > f321ba5 Update dependencies (# 7347)
  > 4c9ad03 Mark new trigger filter tests as beta (# 7349)
  > 494ebb2 bumped x/net to v0.17 to fix cve-2023-44487 (# 7348)
  > da0fba3 Workaround for cert-manager update spec issue (# 7345)
  > a0e4c18 Update `kncloudevents.Dispatcher` to add a JWT (# 7328)
  > c46bdd3 support auto generation of SinkBindings identity service account (# 7327)
  > 029b340 Remove unnecessary filter accumulation logic (# 7325)
  > fb494d1 [main] Update community files (# 7342)
  > ba02f4a [main] Upgrade to latest dependencies (# 7340)

@knative-prow
Copy link

knative-prow bot commented Oct 17, 2023

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

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

In response to this:

/assign @knative-extensions/eventing-wg-leads
/cc @knative-extensions/eventing-wg-leads

Produced via:

for x in eventing-crds.yaml eventing-core.yaml eventing-tls-networking.yaml; do
 curl https://storage.googleapis.com/knative-nightly/eventing/latest/$x > ${GITHUB_WORKSPACE}/./third_party/eventing-latest/$x
done

Details:

bumping knative.dev/eventing 9de5275...6dca8c4:
 > 6dca8c4 [main] Upgrade to latest dependencies (# 7366)
 > 082d90a Remove unneeded configmapinformer in sinkbinding controller (# 7357)
 > f321ba5 Update dependencies (# 7347)
 > 4c9ad03 Mark new trigger filter tests as beta (# 7349)
 > 494ebb2 bumped x/net to v0.17 to fix cve-2023-44487 (# 7348)
 > da0fba3 Workaround for cert-manager update spec issue (# 7345)
 > a0e4c18 Update `kncloudevents.Dispatcher` to add a JWT (# 7328)
 > c46bdd3 support auto generation of SinkBindings identity service account (# 7327)
 > 029b340 Remove unnecessary filter accumulation logic (# 7325)
 > fb494d1 [main] Update community files (# 7342)
 > ba02f4a [main] Upgrade to latest dependencies (# 7340)

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/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 17, 2023
@codecov
Copy link

codecov bot commented Oct 17, 2023 •
edited
Loading

Codecov Report

Merging #3405 (9baaddb) into main (0997c65) will decrease coverage by 9.37%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #3405      +/-   ##
============================================
- Coverage     70.94%   61.57%   -9.37%     
+ Complexity      762      761       -1     
============================================
  Files            90      181      +91     
  Lines          3077    12396    +9319     
  Branches        265      265              
============================================
+ Hits           2183     7633    +5450     
- Misses          728     4161    +3433     
- Partials        166      602     +436
Flag Coverage Δ
java-unittests 70.84% <ø> (-0.10%) ⬇️

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

see 92 files with indirect coverage changes

creydr
creydr approved these changes Oct 18, 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 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 18, 2023
@creydr
Copy link
Contributor

creydr commented Oct 18, 2023

/retest

2 similar comments
@creydr
Copy link
Contributor

creydr commented Oct 18, 2023

/retest

@creydr
Copy link
Contributor

creydr commented Oct 18, 2023

/retest

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

creydr commented Oct 18, 2023

/retest

3 similar comments
@creydr
Copy link
Contributor

creydr commented Oct 18, 2023

/retest

@creydr
Copy link
Contributor

creydr commented Oct 18, 2023

/retest

@creydr
Copy link
Contributor

creydr commented Oct 19, 2023

/retest

@creydr
Copy link
Contributor

creydr commented Oct 19, 2023

/retest

@Cali0707
Copy link
Member

/retest-required

Cali0707
Cali0707 approved these changes Oct 19, 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 19, 2023
@Cali0707
Copy link
Member

/retest-required

@creydr
Copy link
Contributor

creydr commented Oct 20, 2023

/retest

@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 20, 2023
@Cali0707
Copy link
Member

/retest-required

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 24, 2023

/retest

@knative-automation
Update eventing nightly
9baaddb 
bumping knative.dev/eventing 9de5275...16a3986:
  > 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)
  > 16d75a9 mt-broker-ingress check audience of received token (# 7336)
  > 3ef831c Increase expiration buffer time of OIDCTokenProviders cache to 5 minutes (# 7355)
  > 5168dc3 Update dependencies (# 7378)
  > f9314d8 [main] Upgrade to latest dependencies (# 7372)
  > 4a07fdc Move OIDC e2e tests to test/auth (# 7363)
  > 0933c11 Expose audience of an InMemoryChannel in its status (# 7371)
  > 169542f OIDC - Support auto generation of ApiServerSource identity service account and expose in AuthStatus (# 7330)
  > 62ec33b Support auto generation of Subscriptions identity service account and… (# 7338)
  > 0af3ef6 [main] Upgrade to latest dependencies (# 7370)
  > cf6b140 [main] Update community files (# 7369)
  > de6915a Pass sink namespace to PingSource (# 7368)
  > 6dca8c4 [main] Upgrade to latest dependencies (# 7366)
  > 082d90a Remove unneeded configmapinformer in sinkbinding controller (# 7357)
  > f321ba5 Update dependencies (# 7347)
  > 4c9ad03 Mark new trigger filter tests as beta (# 7349)
  > 494ebb2 bumped x/net to v0.17 to fix cve-2023-44487 (# 7348)
  > da0fba3 Workaround for cert-manager update spec issue (# 7345)
  > a0e4c18 Update `kncloudevents.Dispatcher` to add a JWT (# 7328)
  > c46bdd3 support auto generation of SinkBindings identity service account (# 7327)
  > 029b340 Remove unnecessary filter accumulation logic (# 7325)
  > fb494d1 [main] Update community files (# 7342)
  > ba02f4a [main] Upgrade to latest dependencies (# 7340)

Signed-off-by: Knative Automation <[email protected]>
@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 24, 2023
@creydr
Copy link
Contributor

creydr commented Oct 24, 2023

/retest

1 similar comment
@creydr
Copy link
Contributor

creydr commented Oct 24, 2023

/retest

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

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

@Cali0707 Cali0707 Cali0707 approved these changes

@creydr creydr creydr approved these changes

Assignees

@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.
3 participants
@knative-automation @creydr @Cali0707