merlimat opened a new pull request #1584: On tenants cli tool, default tenant to all clusters if not specified URL: https://github.com/apache/incubator-pulsar/pull/1584 ### Motivation Currently, the create tenant CLI tool is requiring 2 arguments to be passed: * List of admin roles (principal names that can act as administrators for the tenant) * List of clusters the tenant is allowed to use In many cases, these parameters are not important to pass. For example: * If authentication is disabled, admin roles would be meaningless * If there's 1 single cluster or if you want to white list tenants to use all clusters, the CLI args become repetitive ### Modifications Let the 2 arguments to be optional. When issuing a command like: ```shell bin/pulsar-admin tenants create my-tenant ``` This will create a tenant with: * No admin roles defined * Allowed to use *all* available clusters (at the tenant creation time, if more clusters are added later, the tenant would need to be given permission to use)
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services