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

[Transactions] Add pulsar-admin commands to lookup Coordinators #17513

Closed
1 of 2 tasks
eolivelli opened this issue Sep 7, 2022 · 2 comments · Fixed by #17522
Closed
1 of 2 tasks

[Transactions] Add pulsar-admin commands to lookup Coordinators #17513

eolivelli opened this issue Sep 7, 2022 · 2 comments · Fixed by #17522
Labels

Comments

@eolivelli
Copy link
Contributor

Search before asking

  • I searched in the issues and found nothing similar.

Motivation

Actually there is no way using pulsar-admin to get the list of coordinators and to know which broker is running them.

I am using this command
bin/pulsar-admin topics partitioned-lookup pulsar/system/transaction_coordinator_assign

There are a few problems:

  • it is basically impossible to users to know that you can do it (I had to look at Pulsar code)
  • it returns information about only topics that are currently owned

Also I have observed that listing the topics inside pulsar/system returns an empty list even while disabling authn/authz
This should be unblocked in order to allow troubleshooting

Solution

Add a new command
pulsar-admin transactions list-coordinators

the command should list:

  • the coordinator
  • the coordinator id (used by other commands)
  • the broker that is running the service

Alternatives

No response

Anything else?

No response

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@michaeljmarshall
Copy link
Member

What kind of authorization will pulsar-admin transactions list-coordinators require?

@github-actions
Copy link

github-actions bot commented Oct 8, 2022

The issue had no activity for 30 days, mark with Stale label.

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 a pull request may close this issue.

2 participants