Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

Upgrade Sarama To v0.30.0 #975

Conversation

travis-minke-sap
Copy link
Contributor

@travis-minke-sap travis-minke-sap commented Nov 5, 2021

In general we want to keep up with Sarama Kafka Client library dependencies. Specifically not being up to date was causing problems for eventing e2e tests (see PR #972)

Proposed Changes

  • 🎁 Move Sarama from v0.29.1 to v0.30.0
  • 🧽 Add missing ClusterAdmin interface functions to mocks
  • 🐛 Update common offset tests to return ApiVersionsRequest response

@google-cla google-cla bot added the cla: yes Indicates the PR's author has signed the CLA. label Nov 5, 2021
@knative-prow-robot knative-prow-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Nov 5, 2021
@codecov
Copy link

codecov bot commented Nov 5, 2021

Codecov Report

Merging #975 (8d7a3f5) into main (bb5d3e0) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #975   +/-   ##
=======================================
  Coverage   75.02%   75.02%           
=======================================
  Files         121      121           
  Lines        5650     5650           
=======================================
  Hits         4239     4239           
  Misses       1188     1188           
  Partials      223      223           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb5d3e0...8d7a3f5. Read the comment docs.

@travis-minke-sap
Copy link
Contributor Author

/test pull-knative-sandbox-eventing-kafka-upgrade-tests

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.

Thanks for doing this, Travis!

/lgtm
/approve

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 8, 2021
@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: matzew, travis-minke-sap

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 [matzew,travis-minke-sap]

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

@knative-prow-robot knative-prow-robot merged commit c5e12df into knative-extensions:main Nov 8, 2021
@travis-minke-sap travis-minke-sap deleted the upgrade-sarama-v0.30.0 branch November 8, 2021 15:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants