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

Upgrade kafka component version to 3.2.0 #1843

Merged
merged 2 commits into from
Feb 3, 2023

Conversation

bitstan-earthyfrodo
Copy link
Contributor

@bitstan-earthyfrodo bitstan-earthyfrodo commented Jan 5, 2023

Change Overview

Pull request type

Please check the type of change your PR introduces:

  • 🚧 Work in Progress
  • 🌈 Refactoring (no functional changes, no api changes)
  • 🐹 Trivial/Minor
  • 🐛 Bugfix
  • 🌻 Feature
  • 🗺️ Documentation
  • 🤖 Test

Issues

Test Plan

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

Manually tested the changes

BACKUP logs

RESTORE logs

@pavannd1
Copy link
Contributor

pavannd1 commented Jan 5, 2023

@bitstan-earthyfrodo Thank you for trying out Kanister and for submitting this pull request 🎉.
Were you able to verify the backup/restore process on this new version?

@bitstan-earthyfrodo
Copy link
Contributor Author

@bitstan-earthyfrodo Thank you for trying out Kanister and for submitting this pull request 🎉. Were you able to verify the backup/restore process on this new version?

@pavannd1 I cannot use the blueprint in my setup because I'm having no Amazon S3 in place. In this case, I cannot test the backup/restore procedure but the current version in the kafka-cluster.yml will definitly not work anymore with strimzi-kafka-operator in Helm v0.32.0(latest as of today).

@pavannd1
Copy link
Contributor

pavannd1 commented Jan 6, 2023

Thank you @bitstan-earthyfrodo!
@akankshakumari393 Could you please help out with testing these changes?

@akankshakumari393
Copy link
Contributor

sure will take a look. Thanks 👍🏻

@akankshakumari393
Copy link
Contributor

Tried testing this and the zookeeper component is failing. Will investigate if there are some other changes that are required.

Copy link
Contributor

@akankshakumari393 akankshakumari393 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @bitstan-earthyfrodo, Thank you for raising the PR, I have verified your changes on S3. Its works fine as described in our example

@mergify mergify bot merged commit 795471e into kanisterio:master Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Kafka version outdated
3 participants