-
Notifications
You must be signed in to change notification settings - Fork 27
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
Update dependency org.apache.kafka:kafka-clients to v3.6.0 #340
Update dependency org.apache.kafka:kafka-clients to v3.6.0 #340
Conversation
Codecov Report
@@ Coverage Diff @@
## main #340 +/- ##
============================================
+ Coverage 63.55% 63.64% +0.09%
- Complexity 715 718 +3
============================================
Files 82 82
Lines 3298 3298
Branches 303 303
============================================
+ Hits 2096 2099 +3
+ Misses 1014 1011 -3
Partials 188 188
Flags with carried forward coverage won't be shown. Click here to find out more. |
834bec5
to
2584f43
Compare
92db479
to
71928e2
Compare
91e9834
to
dd23ed8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverting approval - @lewijacn pointed out that this change should also include an update to the Kafka version used in the migration console Dockerfile:
opensearch-migrations/TrafficCapture/dockerSolution/src/main/docker/migrationConsole/Dockerfile
Line 20 in 2ec00c0
RUN wget -qO- https://archive.apache.org/dist/kafka/3.5.0/kafka_2.13-3.5.0.tgz | tar --transform 's!^[^/]*!kafka!' -xvz |
a7b4e5e
to
a4cf300
Compare
f6aea02
to
6866e1c
Compare
9c66fb5
to
3f9a620
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated https://github.com/opensearch-project/opensearch-migrations/pull/340/files for consistency.
3f9a620
to
8999f4c
Compare
This PR contains the following updates:
3.5.1
->3.6.0
By merging this PR, the issue #332 will be automatically resolved and closed: