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

update: clarify KRaft metrics removal #691

Merged

Conversation

harshini-rangaswamy
Copy link
Contributor

@harshini-rangaswamy harshini-rangaswamy commented Feb 12, 2025

Describe your changes

Clarifies the removal of certain metrics in KRaft mode for Aiven for Apache Kafka®. Ensures only relevant metrics are exposed, aligning with the transition from ZooKeeper to KRaft.

Checklist

  • The first paragraph of the page is on one line.
  • The other lines have a line break at 90 characters.
  • I checked the output.
  • I applied the style guide.
  • My links start with /docs/.

Copy link

cloudflare-workers-and-pages bot commented Feb 12, 2025

Deploying aiven-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 368d83a
Status: ✅  Deploy successful!
Preview URL: https://b6a76114.aiven-docs.pages.dev
Branch Preview URL: https://doc-1317-document-deprecated.aiven-docs.pages.dev

View logs

@harshini-rangaswamy harshini-rangaswamy self-assigned this Feb 12, 2025
@harshini-rangaswamy harshini-rangaswamy added the review/SME An SME must review the PR. label Feb 12, 2025
Copy link
Contributor

@juha-aiven juha-aiven left a comment

Choose a reason for hiding this comment

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

Looking good now.

@harshini-rangaswamy harshini-rangaswamy marked this pull request as ready for review February 13, 2025 10:43
@harshini-rangaswamy harshini-rangaswamy requested a review from a team as a code owner February 13, 2025 10:43
Copy link
Contributor

@juha-aiven juha-aiven left a comment

Choose a reason for hiding this comment

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

LGTM

@harshini-rangaswamy harshini-rangaswamy added review/TW A TW must review the PR. Comes after review/SME. and removed review/SME An SME must review the PR. labels Feb 13, 2025
Copy link
Contributor

@staceysalamon-aiven staceysalamon-aiven left a comment

Choose a reason for hiding this comment

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

LGTM!

@harshini-rangaswamy harshini-rangaswamy merged commit 7184345 into main Feb 13, 2025
4 checks passed
@harshini-rangaswamy harshini-rangaswamy deleted the DOC-1317-document-deprecated-metrics-in-kafka-3-9 branch February 13, 2025 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review/TW A TW must review the PR. Comes after review/SME. service/kafka
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants