[knox] branch master updated: KNOX-2915 - We need reloadDescriptors() when starting the monitors (#761)

[smolnar]

This is an automated email from the ASF dual-hosted git repository.

smolnar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/knox.git


The following commit(s) were added to refs/heads/master by this push:
     new 989b99f33 KNOX-2915 - We need reloadDescriptors() when starting the 
monitors (#761)
989b99f33 is described below

commit 989b99f33d5e0ebabcd7aebd5bfbcb1fe382c5f4
Author: Sandor Molnar <smol...@apache.org>
AuthorDate: Fri Jun 9 13:30:57 2023 +0200

    KNOX-2915 - We need reloadDescriptors() when starting the monitors (#761)
    
    The reason being is that HXR parser might generated new desciptors that 
were not picked up earlier but Knox needs to handle them.
---
 .../knox/gateway/services/topology/impl/DefaultTopologyService.java   | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/gateway-server/src/main/java/org/apache/knox/gateway/services/topology/impl/DefaultTopologyService.java
 
b/gateway-server/src/main/java/org/apache/knox/gateway/services/topology/impl/DefaultTopologyService.java
index fba249eda..37640bbc8 100644
--- 
a/gateway-server/src/main/java/org/apache/knox/gateway/services/topology/impl/DefaultTopologyService.java
+++ 
b/gateway-server/src/main/java/org/apache/knox/gateway/services/topology/impl/DefaultTopologyService.java
@@ -549,9 +549,11 @@ public class DefaultTopologyService extends 
FileAlterationListenerAdaptor implem
         
log.remoteConfigurationMonitorStartFailure(remoteMonitor.getClass().getTypeName(),
 e.getLocalizedMessage());
       }
     }
+
+    // Trigger descriptor discovery (KNOX-2301)
+    reloadDescriptors();
   }
 
-  // Trigger descriptor discovery (KNOX-2301)
   @Override
   public void reloadDescriptors() {
     
log.loadingDescriptorsFromDirectory(descriptorsDirectory.getAbsolutePath());