[fix] metricpath warning was too verbose. Fixing overall log level also (#173)

filipecosta90 · web-flow · commit f9388150c208 · 2021-08-03T16:35:53.000+01:00
* [add] metricpath warning was too verbose.
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "redisbench-admin"
-version = "0.4.0"
+version = "0.4.1"
 description = "Redis benchmark run helper. A wrapper around Redis and Redis Modules benchmark tools ( ftsb_redisearch, memtier_benchmark, redis-benchmark, aibench, etc... )."
 authors = ["filipecosta90 <filipecosta.90@gmail.com>"]
 readme = "README.md"
diff --git a/redisbench_admin/cli.py b/redisbench_admin/cli.py
@@ -26,9 +26,9 @@
 from redisbench_admin.watchdog.watchdog import watchdog_command_logic
 
 
-LOG_LEVEL = logging.INFO
+LOG_LEVEL = logging.DEBUG
 if os.getenv("VERBOSE", "1") == "0":
-    LOG_LEVEL = logging.WARN
+    LOG_LEVEL = logging.INFO
 LOG_FORMAT = "%(asctime)s %(levelname)-4s %(message)s"
 LOG_DATEFMT = "%Y-%m-%d %H:%M:%S"
 
diff --git a/redisbench_admin/utils/remote.py b/redisbench_admin/utils/remote.py
@@ -532,7 +532,7 @@ def extract_perversion_timeseries_from_results(
                 }
             else:
                 logging.warning(
-                    "Unable to find metric path {} in {}".format(jsonpath, results_dict)
+                    "Unable to find metric path {} in result dict".format(jsonpath)
                 )
     return True, branch_time_series_dict
 
@@ -599,7 +599,7 @@ def extract_perbranch_timeseries_from_results(
                 }
             else:
                 logging.warning(
-                    "Unable to find metric path {} in {}".format(jsonpath, results_dict)
+                    "Unable to find metric path {} in result dict".format(jsonpath)
                 )
     return True, branch_time_series_dict
 

Original file line number	Diff line number	Diff line change
`@@ -532,7 +532,7 @@ def extract_perversion_timeseries_from_results(`
`532`	`532`	`}`
`533`	`533`	`else:`
`534`	`534`	`logging.warning(`
`535`		`- "Unable to find metric path {} in {}".format(jsonpath, results_dict)`
	`535`	`+ "Unable to find metric path {} in result dict".format(jsonpath)`
`536`	`536`	`)`
`537`	`537`	`return True, branch_time_series_dict`
`538`	`538`
`@@ -599,7 +599,7 @@ def extract_perbranch_timeseries_from_results(`
`599`	`599`	`}`
`600`	`600`	`else:`
`601`	`601`	`logging.warning(`
`602`		`- "Unable to find metric path {} in {}".format(jsonpath, results_dict)`
	`602`	`+ "Unable to find metric path {} in result dict".format(jsonpath)`
`603`	`603`	`)`
`604`	`604`	`return True, branch_time_series_dict`
`605`	`605`