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

Conversation

knative-automation
Copy link
Contributor

@knative-automation knative-automation commented Jun 27, 2023

/assign @knative-sandbox/eventing-wg-leads
/cc @knative-sandbox/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 95cdbaa...ed05a35:
  > ed05a35 Revert "Add loopclosure linter" (# 7083)
  > 838274e [main] Upgrade to latest dependencies (# 7078)
  > a4f0c10 [main] Update community files (# 7085)
  > afcf60b Move ingress & filter TLS secret names into constants (# 7081)
  > 1e96c78 feat!: Add tls support for ingress (# 6986)
  > 6a0495c Add loopclosure linter (# 7079)
  > 2fe1db6 Updated mtping TLS cert test to bind to free port (# 7036)
  > 516a915 Upgrade rekt to latest (# 7076)
  > 6a890e0 Fix flaky unit tests (# 7080)
  > eaf28a7 Add tracing for TestBrokerWithManyTriggers (# 7077)
  > f5b1b12 Send namespace header in MT components (# 7048)
  > 4b5fde8 [main] Update community files (# 7043)
  > 8f74094 Add handler to auto create Event Types (# 7034)
  > 901ef61 Remove check for empty Namespace on resolver (# 7040)

@knative-prow
Copy link

knative-prow bot commented Jun 27, 2023

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

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

In response to this:

/assign @knative-sandbox/eventing-wg-leads
/cc @knative-sandbox/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 95cdbaa...901ef61:
 > 901ef61 Remove check for empty Namespace on resolver (# 7040)

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 Jun 27, 2023
@codecov
Copy link

codecov bot commented Jun 27, 2023

Codecov Report

Merging #3173 (971d48f) into main (9590d86) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #3173      +/-   ##
============================================
- Coverage     63.41%   63.41%   -0.01%     
  Complexity      747      747              
============================================
  Files           167      167              
  Lines         11815    11815              
  Branches        246      246              
============================================
- Hits           7493     7492       -1     
- Misses         3749     3750       +1     
  Partials        573      573              
Flag Coverage Δ
java-unittests 79.66% <ø> (-0.04%) ⬇️

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

see 4 files with indirect coverage changes

@knative-automation knative-automation force-pushed the update-nightly/eventing branch 5 times, most recently from c227310 to 8004705 Compare July 4, 2023 11:05
@knative-automation knative-automation force-pushed the update-nightly/eventing branch 3 times, most recently from 48d411e to c538457 Compare July 11, 2023 11:05
@creydr
Copy link
Contributor

creydr commented Jul 11, 2023

/retest

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 Jul 12, 2023
@knative-prow knative-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 12, 2023
@creydr
Copy link
Contributor

creydr commented Jul 12, 2023

/retest

bumping knative.dev/eventing 95cdbaa...ed05a35:
  > ed05a35 Revert "Add loopclosure linter" (# 7083)
  > 838274e [main] Upgrade to latest dependencies (# 7078)
  > a4f0c10 [main] Update community files (# 7085)
  > afcf60b Move ingress & filter TLS secret names into constants (# 7081)
  > 1e96c78 feat!: Add tls support for ingress (# 6986)
  > 6a0495c Add loopclosure linter (# 7079)
  > 2fe1db6 Updated mtping TLS cert test to bind to free port (# 7036)
  > 516a915 Upgrade rekt to latest (# 7076)
  > 6a890e0 Fix flaky unit tests (# 7080)
  > eaf28a7 Add tracing for TestBrokerWithManyTriggers (# 7077)
  > f5b1b12 Send namespace header in MT components (# 7048)
  > 4b5fde8 [main] Update community files (# 7043)
  > 8f74094 Add handler to auto create Event Types (# 7034)
  > 901ef61 Remove check for empty Namespace on resolver (# 7040)

Signed-off-by: Knative Automation <automation@knative.team>
@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label Jul 12, 2023
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 Jul 12, 2023
@knative-prow
Copy link

knative-prow bot commented Jul 12, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 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

@creydr
Copy link
Contributor

creydr commented Jul 12, 2023

/retest

1 similar comment
@creydr
Copy link
Contributor

creydr commented Jul 12, 2023

/retest

@knative-prow knative-prow bot merged commit 681c7cc into knative-extensions:main Jul 12, 2023
15 of 18 checks passed
Rahul-Kumar-prog pushed a commit to Rahul-Kumar-prog/eventing-kafka-broker that referenced this pull request Jul 31, 2023
bumping knative.dev/eventing 95cdbaa...ed05a35:
  > ed05a35 Revert "Add loopclosure linter" (# 7083)
  > 838274e [main] Upgrade to latest dependencies (# 7078)
  > a4f0c10 [main] Update community files (# 7085)
  > afcf60b Move ingress & filter TLS secret names into constants (# 7081)
  > 1e96c78 feat!: Add tls support for ingress (# 6986)
  > 6a0495c Add loopclosure linter (# 7079)
  > 2fe1db6 Updated mtping TLS cert test to bind to free port (# 7036)
  > 516a915 Upgrade rekt to latest (# 7076)
  > 6a890e0 Fix flaky unit tests (# 7080)
  > eaf28a7 Add tracing for TestBrokerWithManyTriggers (# 7077)
  > f5b1b12 Send namespace header in MT components (# 7048)
  > 4b5fde8 [main] Update community files (# 7043)
  > 8f74094 Add handler to auto create Event Types (# 7034)
  > 901ef61 Remove check for empty Namespace on resolver (# 7040)

Signed-off-by: Knative Automation <automation@knative.team>
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. 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
Development

Successfully merging this pull request may close these issues.

2 participants