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

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

knative-automation
Copy link
Contributor

@knative-automation knative-automation commented May 1, 2024

/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 4a3f65d...68920d2:
  > 68920d2 Fix log message for recordevents receiver (# 7936)
  > 9c0f9cc Update eventtype CRD to include `.spec.reference.address` (# 7935)
  > b5f92c2 Copy message before passing it to eventtype autocreate handler (# 7934)
  > 0e14da4 fix(test): the attribute filter transform test correctly compares slices now (# 7931)
  > 5355171 Fix trust bundle propagation (# 7924)
  > 15be248 CrossNamespace: Data Plane Changes for Broker Filter (# 7913)
  > 34f9e43 CrossNamespace: Control Plane Changes for Trigger (# 7889)
  > 308e6b6 SubjectAccessReviews resources correction (# 7925)
  > 9cdc7ba CrossNamespace: Update knsubscribe permissions (# 7926)
  > dbaaedc [main] Update actions (# 7923)
  > 3ea1900 [main] Upgrade to latest dependencies (# 7922)
  > 3189fc2 Add attribute filter transform function (# 7909)
  > 3541408 Make transport-encryption a Beta feature (# 7915)
  > 884f0da Adding uname name mapping for x86_64 in hack/install.sh (# 7917)
  > 6b6f6d1 Sequence and Parallel: announce correct OIDC identities in authstatus (# 7902)
  > 58d7920 Fix IMC metrics for transport encryption (# 7904)
  > 4358574 CrossNamespace: Testing Code Style (# 7918)
  > 4951b74 [main] Upgrade to latest dependencies (# 7920)
  > 1572967 Map subscription to graph edges (# 7880)
  > 72585c9 CrossNamespace: Control Plane Changes for Subscription (# 7882)
  > 599c548 Assert EventTypes references for IMC test (# 7899)
  > 1cc8179 Remove SelfSigned issuer from released `transport-encryption` artifacts (# 7905)
  > 4f32823 Refactor NewFiltersFeatureSet to prevent starvation (# 7911)
  > fa46a45 feat: added fake client reactor for subject access reviews (# 7908)
  > 1ce7074 Fix trust bundle propagation already exists error (# 7906)
  > e9b3af6 Update e2e tests for OIDC sender identity matching to be a bit more resilient (# 7903)
  > 1e6e235 fix: trigger tests set annotations correctly now (# 7887)
  > 159d6fc [main] Upgrade to latest dependencies (# 7901)
  > c7401a1 Add e2e test to verify used sender identity is correct (# 7900)
  > 96e5c18 [main] Update community files (# 7897)
  > c3a1c19 EventType tests verify the type under spec.reference  (# 7888)
  > c104df5 [main] Upgrade to latest dependencies (# 7894)
  > 07ebf11 [main] Upgrade to latest dependencies (# 7891)
  > 0e165f2 Containersource use OIDC identity of corresponding SinkBinding (# 7890)
  > 3dfe973 Add SubscriptionsAPI filters to APIServerSource (# 7799)
  > 11a1ad4 feat: added sources to event lineage graph constructor (# 7881)
  > 217f7a3 Create validation function for cross namespace referencing (# 7812)

Copy link

knative-prow bot commented May 1, 2024

@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 4a3f65d...217f7a3:
 > 217f7a3 Create validation function for cross namespace referencing (# 7812)

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 May 1, 2024
Copy link

codecov bot commented May 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.48%. Comparing base (aacc057) to head (e9c8433).
Report is 20 commits behind head on main.

Current head e9c8433 differs from pull request most recent head aae61a2

Please upload reports for the commit aae61a2 to get more accurate results.

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #3869       +/-   ##
=============================================
- Coverage     73.70%   49.48%   -24.22%     
=============================================
  Files           100      247      +147     
  Lines          3419    14911    +11492     
  Branches        292        0      -292     
=============================================
+ Hits           2520     7379     +4859     
- Misses          723     6773     +6050     
- Partials        176      759      +583     
Flag Coverage Δ
java-unittests ?

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

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

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

/retest

@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 May 2, 2024
@matzew
Copy link
Contributor

matzew commented May 2, 2024

/test reconciler-tests-namespaced-broker

@creydr
Copy link
Contributor

creydr commented May 3, 2024

/retest-required

@matzew
Copy link
Contributor

matzew commented May 3, 2024

/test upgrade-tests

@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label May 3, 2024
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 May 3, 2024
@matzew
Copy link
Contributor

matzew commented May 6, 2024

/retest

@creydr
Copy link
Contributor

creydr commented May 6, 2024

/test reconciler-tests-namespaced-broker
/lgtm
/approve

@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label May 6, 2024
@pierDipi
Copy link
Member

pierDipi commented May 7, 2024

/retest

Copy link
Member

@pierDipi pierDipi 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 the lgtm Indicates that a PR is ready to be merged. label May 7, 2024
Copy link

knative-prow bot commented May 7, 2024

[APPROVALNOTIFIER] This PR is APPROVED

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

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 [creydr,matzew,pierDipi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link

knative-prow bot commented May 7, 2024

New changes are detected. LGTM label has been removed.

@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label May 7, 2024
@knative-automation knative-automation force-pushed the update-nightly/eventing branch 5 times, most recently from 9e64067 to e9daba1 Compare May 15, 2024 11:05
@knative-automation knative-automation force-pushed the update-nightly/eventing branch 5 times, most recently from e9c8433 to f5a3cb0 Compare May 22, 2024 11:05
bumping knative.dev/eventing 4a3f65d...68920d2:
  > 68920d2 Fix log message for recordevents receiver (# 7936)
  > 9c0f9cc Update eventtype CRD to include `.spec.reference.address` (# 7935)
  > b5f92c2 Copy message before passing it to eventtype autocreate handler (# 7934)
  > 0e14da4 fix(test): the attribute filter transform test correctly compares slices now (# 7931)
  > 5355171 Fix trust bundle propagation (# 7924)
  > 15be248 CrossNamespace: Data Plane Changes for Broker Filter (# 7913)
  > 34f9e43 CrossNamespace: Control Plane Changes for Trigger (# 7889)
  > 308e6b6 SubjectAccessReviews resources correction (# 7925)
  > 9cdc7ba CrossNamespace: Update knsubscribe permissions (# 7926)
  > dbaaedc [main] Update actions (# 7923)
  > 3ea1900 [main] Upgrade to latest dependencies (# 7922)
  > 3189fc2 Add attribute filter transform function (# 7909)
  > 3541408 Make transport-encryption a Beta feature (# 7915)
  > 884f0da Adding uname name mapping for x86_64 in hack/install.sh (# 7917)
  > 6b6f6d1 Sequence and Parallel: announce correct OIDC identities in authstatus (# 7902)
  > 58d7920 Fix IMC metrics for transport encryption (# 7904)
  > 4358574 CrossNamespace: Testing Code Style (# 7918)
  > 4951b74 [main] Upgrade to latest dependencies (# 7920)
  > 1572967 Map subscription to graph edges (# 7880)
  > 72585c9 CrossNamespace: Control Plane Changes for Subscription (# 7882)
  > 599c548 Assert EventTypes references for IMC test (# 7899)
  > 1cc8179 Remove SelfSigned issuer from released `transport-encryption` artifacts (# 7905)
  > 4f32823 Refactor NewFiltersFeatureSet to prevent starvation (# 7911)
  > fa46a45 feat: added fake client reactor for subject access reviews (# 7908)
  > 1ce7074 Fix trust bundle propagation already exists error (# 7906)
  > e9b3af6 Update e2e tests for OIDC sender identity matching to be a bit more resilient (# 7903)
  > 1e6e235 fix: trigger tests set annotations correctly now (# 7887)
  > 159d6fc [main] Upgrade to latest dependencies (# 7901)
  > c7401a1 Add e2e test to verify used sender identity is correct (# 7900)
  > 96e5c18 [main] Update community files (# 7897)
  > c3a1c19 EventType tests verify the type under spec.reference  (# 7888)
  > c104df5 [main] Upgrade to latest dependencies (# 7894)
  > 07ebf11 [main] Upgrade to latest dependencies (# 7891)
  > 0e165f2 Containersource use OIDC identity of corresponding SinkBinding (# 7890)
  > 3dfe973 Add SubscriptionsAPI filters to APIServerSource (# 7799)
  > 11a1ad4 feat: added sources to event lineage graph constructor (# 7881)
  > 217f7a3 Create validation function for cross namespace referencing (# 7812)

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

knative-prow bot commented May 24, 2024

@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-keda_eventing-kafka-broker_main aae61a2 link true /test reconciler-tests-keda
channel-reconciler-tests-sasl-plain_eventing-kafka-broker_main aae61a2 link true /test channel-reconciler-tests-sasl-plain
channel-reconciler-tests-ssl_eventing-kafka-broker_main aae61a2 link true /test channel-reconciler-tests-ssl
channel-reconciler-tests-sasl-ssl_eventing-kafka-broker_main aae61a2 link true /test channel-reconciler-tests-sasl-ssl
reconciler-tests_eventing-kafka-broker_main aae61a2 link true /test reconciler-tests
reconciler-tests-namespaced-broker_eventing-kafka-broker_main aae61a2 link true /test reconciler-tests-namespaced-broker

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-sigs/prow repository. I understand the commands that are listed here.

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. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. skip-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants