This is an automated email from the ASF dual-hosted git repository. heneveld pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/brooklyn-server.git
commit bead6bced3ec1cdc659c678e9d304052c2550fc6 Author: Alex Heneveld <[email protected]> AuthorDate: Tue Oct 20 16:23:28 2020 +0100 make the external supplier message debug - doesn't need to be info and very prolific when used in tests --- .../mgmt/internal/BasicExternalConfigSupplierRegistry.java | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/core/src/main/java/org/apache/brooklyn/core/mgmt/internal/BasicExternalConfigSupplierRegistry.java b/core/src/main/java/org/apache/brooklyn/core/mgmt/internal/BasicExternalConfigSupplierRegistry.java index fccdec2..a667ce5 100644 --- a/core/src/main/java/org/apache/brooklyn/core/mgmt/internal/BasicExternalConfigSupplierRegistry.java +++ b/core/src/main/java/org/apache/brooklyn/core/mgmt/internal/BasicExternalConfigSupplierRegistry.java @@ -58,18 +58,12 @@ public class BasicExternalConfigSupplierRegistry implements ExternalConfigSuppli public BasicExternalConfigSupplierRegistry(ManagementContext mgmt) { addProvider(DEMO_SAMPLE_PROVIDER, new InPlaceExternalConfigSupplier(mgmt, DEMO_SAMPLE_PROVIDER, - MutableMap.of(DEMO_SAMPLE_PROVIDER_PASSWORD_KEY, DEMO_SAMPLE_PROVIDER_PASSWORD_VALUE)), - /** suppress logging when running this from ide; a nicer way would be to remove this altogether in most tests */ - !BrooklynVersion.isDevelopmentEnvironment()); + MutableMap.of(DEMO_SAMPLE_PROVIDER_PASSWORD_KEY, DEMO_SAMPLE_PROVIDER_PASSWORD_VALUE))); updateFromBrooklynProperties(mgmt); } @Override public void addProvider(String name, ExternalConfigSupplier supplier) { - addProvider(name, supplier, true); - } - - protected void addProvider(String name, ExternalConfigSupplier supplier, boolean logInfo) { synchronized (providersMapMutex) { if (providersByName.containsKey(name) && !DEMO_SAMPLE_PROVIDER.equals(name)) { // allow demo to be overridden @@ -77,7 +71,7 @@ public class BasicExternalConfigSupplierRegistry implements ExternalConfigSuppli } providersByName.put(name, supplier); } - LOG.info("Added external config supplier named '" + name + "': " + supplier); + LOG.debug("Added external config supplier named '" + name + "': " + supplier); } @Override
