redirect logs into stderr instead of stdout, fix #969

Altinity · Aug 2, 2024 · 1b5b448 · 1b5b448
1 parent f348c51
commit 1b5b448
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/ChangeLog.md b/ChangeLog.md
@@ -1,8 +1,12 @@
+# v2.5.22
+IMPROVEMENTS
+- redirect logs into stderr instead of stdout, fix [969](https://github.com/Altinity/clickhouse-backup/issues/969)
+
 # v2.5.21
 IMPROVEMENTS
 - switch from `docker-compose` (python) to `docker compose` (golang)
 - add parallel integration test execution fix [888](https://github.com/Altinity/clickhouse-backup/issues/888)
-- 
+
 BUG FIXES
 - properly handle log_pointer=1 corner case for `check_replica_before_attach: true`, fix [967](https://github.com/Altinity/clickhouse-backup/issues/967)
 - properly handle empty output for `list` command when `remote_storage: custom`, fix [963](https://github.com/Altinity/clickhouse-backup/issues/963), thanks @straysh

diff --git a/cmd/clickhouse-backup/main.go b/cmd/clickhouse-backup/main.go
@@ -23,7 +23,7 @@ var (
 )
 
 func main() {
-	log.SetHandler(logcli.New(os.Stdout))
+	log.SetHandler(logcli.New(os.Stderr))
 	cliapp := cli.NewApp()
 	cliapp.Name = "clickhouse-backup"
 	cliapp.Usage = "Tool for easy backup of ClickHouse with cloud support"

diff --git a/test/integration/integration_test.go b/test/integration/integration_test.go
@@ -42,7 +42,7 @@ var dockerPool *pool.ObjectPool
 
 // setup log level
 func init() {
-	log.SetHandler(logcli.New(os.Stdout))
+	log.SetHandler(logcli.New(os.Stderr))
 	logLevel := "info"
 	if os.Getenv("LOG_LEVEL") != "" && os.Getenv("LOG_LEVEL") != "info" {
 		logLevel = os.Getenv("LOG_LEVEL")