eolivelli opened a new pull request #10672:
URL: https://github.com/apache/pulsar/pull/10672


   on branch-2.7 NamespacesTest.getNamespaces fails
   
   ```
   java.lang.AssertionError: should have failed
        at org.testng.Assert.fail(Assert.java:99)
        at 
org.apache.pulsar.broker.admin.NamespacesTest.testGetNamespaces(NamespacesTest.java:264)
        at 
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at 
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
   ```
   
   it looks like AdminResource is missing a check, that is present on master 
branch
   
https://github.com/apache/pulsar/blob/5dc5de849b582c7f312764db043da9483c17146a/pulsar-broker/src/main/java/org/apache/pulsar/broker/web/PulsarWebResource.java#L1094
   
   ```
       protected List<String> getListOfNamespaces(String tenant) throws 
Exception {
           List<String> namespaces = Lists.newArrayList();
   
           if (!tenantResources().exists(path(POLICIES, tenant))) {
               throw new RestException(Status.NOT_FOUND, tenant + " doesn't 
exist");
           }
   ```


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to