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

[sm...@apache.org]

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 <sm...@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());