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 2016/02/01 18:46:43 UTC

[40/50] brooklyn-library git commit: Cleanup logging

Cleanup logging

- Don't warn about "already deployed" when setting config
  during construction (after setting owner!)
- DynamicCluster.addNode manages it immediately
- ssh logging: trace only in SshjTool and ScriptHelper;
  debug in "script runner", such as SshMachineLocation.execWithLogging
- Drop some log levels
- toStrings on LocalManagementContext and LocalSubscriptionManager
- logging subscriptions: avoid duplicate info in log message
- Repeater: avoid multiple logs when pass first time, and give
  more info about max num attempts etc


Project: http://git-wip-us.apache.org/repos/asf/brooklyn-library/repo
Commit: http://git-wip-us.apache.org/repos/asf/brooklyn-library/commit/50756cae
Tree: http://git-wip-us.apache.org/repos/asf/brooklyn-library/tree/50756cae
Diff: http://git-wip-us.apache.org/repos/asf/brooklyn-library/diff/50756cae

Branch: refs/heads/0.4.0
Commit: 50756cae1e185211ef1f55c20d5fffe5c179df01
Parents: 96e4ab4
Author: Aled Sage <al...@gmail.com>
Authored: Wed Oct 10 21:33:38 2012 +0100
Committer: Aled Sage <al...@gmail.com>
Committed: Thu Oct 11 10:05:46 2012 +0100

----------------------------------------------------------------------
 .../src/main/java/brooklyn/entity/proxy/AbstractController.java | 2 +-
 .../java/brooklyn/entity/proxy/nginx/NginxController.groovy     | 5 ++---
 2 files changed, 3 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/brooklyn-library/blob/50756cae/software/webapp/src/main/java/brooklyn/entity/proxy/AbstractController.java
----------------------------------------------------------------------
diff --git a/software/webapp/src/main/java/brooklyn/entity/proxy/AbstractController.java b/software/webapp/src/main/java/brooklyn/entity/proxy/AbstractController.java
index d1e5de3..cc52e76 100644
--- a/software/webapp/src/main/java/brooklyn/entity/proxy/AbstractController.java
+++ b/software/webapp/src/main/java/brooklyn/entity/proxy/AbstractController.java
@@ -209,7 +209,7 @@ public abstract class AbstractController extends SoftwareProcessEntity implement
     @Override
     protected void postStart() {
         super.postStart();
-        LOG.info("Adding policy {} to {} on AbstractController.start", serverPoolMemberTrackerPolicy, this);
+        LOG.info("Adding policy {} to {}, during start", serverPoolMemberTrackerPolicy, this);
         addPolicy(serverPoolMemberTrackerPolicy);
         if (getUrl()==null) setAttribute(ROOT_URL, inferUrl());
         reset();

http://git-wip-us.apache.org/repos/asf/brooklyn-library/blob/50756cae/software/webapp/src/main/java/brooklyn/entity/proxy/nginx/NginxController.groovy
----------------------------------------------------------------------
diff --git a/software/webapp/src/main/java/brooklyn/entity/proxy/nginx/NginxController.groovy b/software/webapp/src/main/java/brooklyn/entity/proxy/nginx/NginxController.groovy
index dc230df..46a93f2 100644
--- a/software/webapp/src/main/java/brooklyn/entity/proxy/nginx/NginxController.groovy
+++ b/software/webapp/src/main/java/brooklyn/entity/proxy/nginx/NginxController.groovy
@@ -20,13 +20,12 @@ import brooklyn.event.basic.BasicConfigKey
 import brooklyn.util.ResourceUtils
 import brooklyn.util.flags.SetFromFlag
 import brooklyn.util.internal.TimeExtras
+import brooklyn.util.text.Strings
 
 import com.google.common.collect.Iterables
 import com.google.common.collect.LinkedHashMultimap
 import com.google.common.collect.Multimap
 
-import brooklyn.util.text.Strings
-
 /**
  * An entity that represents an Nginx proxy (e.g. for routing requests to servers in a cluster).
  * <p>
@@ -57,7 +56,7 @@ public class NginxController extends AbstractController {
     @SetFromFlag("sticky")
     public static final BasicConfigKey<Boolean> STICKY =
         new BasicConfigKey<Boolean>(Boolean.class, "nginx.sticky", "whether to use sticky sessions", true);
-    
+
     public NginxController(Entity owner) {
         this(new LinkedHashMap(), owner);
     }