You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by eo...@apache.org on 2021/04/29 08:17:47 UTC
[pulsar] branch branch-2.7 updated: Suppress printing "skip
Processing" rows in startup scripts (#10275)
This is an automated email from the ASF dual-hosted git repository.
eolivelli pushed a commit to branch branch-2.7
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/branch-2.7 by this push:
new 2623241 Suppress printing "skip Processing" rows in startup scripts (#10275)
2623241 is described below
commit 262324182ad52a95ab389a8edd0bf633848ae094
Author: Lari Hotari <lh...@users.noreply.github.com>
AuthorDate: Tue Apr 20 10:40:09 2021 +0300
Suppress printing "skip Processing" rows in startup scripts (#10275)
### Motivation
The startup logging of the broker (and bookie) is very verbose after the changes made in #8709 .
[example of typical output](https://github.com/apache/pulsar/runs/2378390309?check_suite_focus=true#step:13:751):
```
[conf/broker.conf] skip Processing #
[conf/broker.conf] skip Processing # Licensed to the Apache Software Foundation (ASF) under one
[conf/broker.conf] skip Processing # or more contributor license agreements. See the NOTICE file
[conf/broker.conf] skip Processing # distributed with this work for additional information
[conf/broker.conf] skip Processing # regarding copyright ownership. The ASF licenses this file
[conf/broker.conf] skip Processing # to you under the Apache License, Version 2.0 (the
[conf/broker.conf] skip Processing # "License"); you may not use this file except in compliance
[conf/broker.conf] skip Processing # with the License. You may obtain a copy of the License at
[conf/broker.conf] skip Processing #
[conf/broker.conf] skip Processing # http://www.apache.org/licenses/LICENSE-2.0
[conf/broker.conf] skip Processing #
[conf/broker.conf] skip Processing # Unless required by applicable law or agreed to in writing,
[conf/broker.conf] skip Processing # software distributed under the License is distributed on an
[conf/broker.conf] skip Processing # "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
[conf/broker.conf] skip Processing # KIND, either express or implied. See the License for the
[conf/broker.conf] skip Processing # specific language governing permissions and limitations
[conf/broker.conf] skip Processing # under the License.
[conf/broker.conf] skip Processing #
[conf/broker.conf] skip Processing ### --- General broker settings --- ###
[conf/broker.conf] skip Processing # Zookeeper quorum connection string
[conf/broker.conf] skip Processing # Configuration Store connection string
[conf/broker.conf] skip Processing # Broker data port
[conf/broker.conf] skip Processing # Broker data port for TLS - By default TLS is disabled
[conf/broker.conf] skip Processing # Port to use to server HTTP request
[conf/broker.conf] skip Processing # Port to use to server HTTPS request - By default TLS is disabled
[conf/broker.conf] skip Processing # Hostname or IP address the service binds on, default is 0.0.0.0.
```
### Modifications
Suppress printing the `skip Processing` lines unless `PF_ENV_DEBUG` environment variable is set to `1`.
(cherry picked from commit 762fea3cfd49d783aa73541c7e2914c3d7414bb1)
---
docker/pulsar/scripts/apply-config-from-env-with-prefix.py | 4 +++-
docker/pulsar/scripts/apply-config-from-env.py | 4 +++-
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/docker/pulsar/scripts/apply-config-from-env-with-prefix.py b/docker/pulsar/scripts/apply-config-from-env-with-prefix.py
index 670ba9a..a3a4ada 100755
--- a/docker/pulsar/scripts/apply-config-from-env-with-prefix.py
+++ b/docker/pulsar/scripts/apply-config-from-env-with-prefix.py
@@ -35,6 +35,7 @@ if len(sys.argv) < 3:
prefix = sys.argv[1]
conf_files = sys.argv[2:]
+PF_ENV_DEBUG = (os.environ.get('PF_ENV_DEBUG','0') == '1')
for conf_filename in conf_files:
lines = [] # List of config file lines
@@ -51,7 +52,8 @@ for conf_filename in conf_files:
k,v = line.split('=', 1)
keys[k] = len(lines) - 1
except:
- print("[%s] skip Processing %s" % (conf_filename, line))
+ if PF_ENV_DEBUG:
+ print("[%s] skip Processing %s" % (conf_filename, line))
# Update values from Env
for k in sorted(os.environ.keys()):
diff --git a/docker/pulsar/scripts/apply-config-from-env.py b/docker/pulsar/scripts/apply-config-from-env.py
index 803c4f1..6e2eb74 100755
--- a/docker/pulsar/scripts/apply-config-from-env.py
+++ b/docker/pulsar/scripts/apply-config-from-env.py
@@ -35,6 +35,7 @@ if len(sys.argv) < 2:
conf_files = sys.argv[1:]
PF_ENV_PREFIX = 'PULSAR_PREFIX_'
+PF_ENV_DEBUG = (os.environ.get('PF_ENV_DEBUG','0') == '1')
for conf_filename in conf_files:
lines = [] # List of config file lines
@@ -53,7 +54,8 @@ for conf_filename in conf_files:
k = k[1:]
keys[k.strip()] = len(lines) - 1
except:
- print("[%s] skip Processing %s" % (conf_filename, line))
+ if PF_ENV_DEBUG:
+ print("[%s] skip Processing %s" % (conf_filename, line))
# Update values from Env
for k in sorted(os.environ.keys()):