Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add/Update warnings for Zoo and MM1 removal #10828

Merged

Conversation

scholzj
Copy link
Member

@scholzj scholzj commented Nov 12, 2024

Type of change

  • Task

Description

We are now pretty certain the ZooKeeper and MM1 will be gone in Strimzi 0.46.0 as decided in SP|77. This PR:

  • Adds the removal warning for Zoo-based Kafka clusters
  • Updates the warning for MM1 clusters

It also adds the warning to the CHANGELOG.md.

Checklist

  • Write tests
  • Make sure all tests pass
  • Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
  • Update CHANGELOG.md

Signed-off-by: Jakub Scholz <www@scholzj.com>
@scholzj scholzj added this to the 0.45.0 milestone Nov 12, 2024
@scholzj scholzj marked this pull request as ready for review November 12, 2024 22:35
@scholzj
Copy link
Member Author

scholzj commented Nov 12, 2024

/azp run zookeeper-regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@PaulRMellor PaulRMellor left a comment

Choose a reason for hiding this comment

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

Looks good. Suggestions are just around MirrorMaker naming consistency

Signed-off-by: Jakub Scholz <www@scholzj.com>
@scholzj scholzj merged commit 9ae9aea into strimzi:main Nov 15, 2024
13 checks passed
@scholzj scholzj deleted the add-and-update-removal-warnings-for-Zoo-and-MM1 branch November 15, 2024 15:59
OwenCorrigan76 pushed a commit to OwenCorrigan76/strimzi-kafka-operator that referenced this pull request Dec 6, 2024
Signed-off-by: Jakub Scholz <www@scholzj.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants