-
Notifications
You must be signed in to change notification settings - Fork 0
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
CLI-965 Remove broker information in ccloud kafka topic describe
#960
Conversation
Can you write some tests for this? Looks like there are no integration tests 😢 |
I see a few |
@MuweiHe This PR should target |
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.
see above comments
Argh, this is kind of a nit, but it would be nice if we could separate this change from all the other changes that are in master but not in 2.0 yet. Is there any chance you could like rebase the changes you made on top of 2.0, and/or copy the master commits to 2.0 separately (preserving git history/authorship)? Happy to help if that proves to be a pain. |
Checklist
[CRUCIAL] Is the change for CP or CCloud functionalities that are already live in prod?
Did you add/update any commands that accept secrets as args/flags?
What
ccloud kafka topic describe
is listing broker information. It is exposing concepts (partitions, leaders, replicas, ISRs) that Cloud should abstract away. These should be removed.References
Test&Review