tjouni opened a new pull request, #14395:
URL: https://github.com/apache/kafka/pull/14395

   kafka-acls cli prints the following error message on invalid operation:
   ```
   $ kafka-acls --bootstrap-server xxx:9095 --remove --allow-principal 
"User:abc" --allow-host * --operation DESCRIBE_CONFIGS --topic xyz
   ResourceType TOPIC only supports operations 
WRITE,DESCRIBE,ALL,READ,CREATE,ALTER,DELETE,DESCRIBE_CONFIGS,ALTER_CONFIGS
   ```
   The following command actually works:
   ```
   $ kafka-acls --bootstrap-server xxx:9095 --remove --allow-principal 
"User:abc" --allow-host * --operation DescribeConfigs --topic xyz
   ```
   
   The operations are listed correctly in the cli --help:
   ```
   --operation <String>                     Operation that is being allowed or  
   
                                              denied. Valid operation names 
are:   
                                                Describe                        
      
                                                DescribeConfigs                 
      
                                                Alter                           
      
                                                IdempotentWrite                 
      
                                                Read                            
      
                                                Delete                          
      
                                                Create                          
      
                                                ClusterAction                   
      
                                                All                             
      
                                                Write                           
      
                                                AlterConfigs                    
      
                                                CreateTokens                    
      
                                                DescribeTokens    
   ```
   
   This PR fixes the invalid formating of operations in the error message and 
adds a space after the comma.
   
   
   ### Committer Checklist (excluded from commit message)
   - [x] Verify design and implementation 
   - [x] Verify test coverage and CI build status
   - [x] Verify documentation (including upgrade notes)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to