You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@brooklyn.apache.org by he...@apache.org on 2021/09/14 21:56:42 UTC

[brooklyn-server] 23/27: downgrade pre-pre-managed location warning, as it seems to occur a lot

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 b1a9a3c4d2c05525fa13714f2fc390427a1c3516
Author: Alex Heneveld <al...@cloudsoftcorp.com>
AuthorDate: Tue Sep 14 19:47:40 2021 +0100

    downgrade pre-pre-managed location warning, as it seems to occur a lot
    
    esp on rebind, and not have any ill effects
---
 .../apache/brooklyn/core/mgmt/internal/LocalLocationManager.java   | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/core/src/main/java/org/apache/brooklyn/core/mgmt/internal/LocalLocationManager.java b/core/src/main/java/org/apache/brooklyn/core/mgmt/internal/LocalLocationManager.java
index 2555209..2b4d54c 100644
--- a/core/src/main/java/org/apache/brooklyn/core/mgmt/internal/LocalLocationManager.java
+++ b/core/src/main/java/org/apache/brooklyn/core/mgmt/internal/LocalLocationManager.java
@@ -153,8 +153,11 @@ public class LocalLocationManager implements LocationManagerInternal {
     
     synchronized void prePreManage(Location loc) {
         if (isPreRegistered(loc)) {
-            log.warn(""+this+" redundant call to pre-pre-manage location "+loc+"; skipping", 
-                    new Exception("source of duplicate pre-pre-manage of "+loc));
+            if (log.isTraceEnabled()) {
+                // seems to occur regularly on rebind, and not be a problem; ignore
+                log.trace("" + this + " redundant call to pre-pre-manage location " + loc + "; skipping",
+                        new Exception("source of duplicate pre-pre-manage of " + loc));
+            }
             return;
         }
         preRegisteredLocationsById.put(loc.getId(), loc);