fix(core): logger level error as default when output is not human (#2790)

Author: Monitob

docs/developer.md

@@ -54,6 +54,10 @@ go test ./internal/namespaces/init -debug
 
 When you are developing tests, you can also use the `Logger` field that is available in the different contexts to write your own logs.
 
+### Checking the version
+
+When you are running CLI commands the version check is made in every action. You can avoid these output setting `SCW_DISABLE_CHECK_VERSION` to false.
+
 #### Targeting specific tests
 
 The test suite is design to run quickly, but when you are recording interactions you probably don't want to record interactions for all the tests in the test suite.

internal/core/bootstrap.go

@@ -66,6 +66,10 @@ type BootstrapConfig struct {
 	BetaMode bool
 }
 
+const (
+	defaultOutput = "human"
+)
+
 // Bootstrap is the main entry point. It is directly called from main.
 // BootstrapConfig.Args is usually os.Args
 // BootstrapConfig.Commands is a list of command available in CLI.
@@ -79,7 +83,7 @@ func Bootstrap(config *BootstrapConfig) (exitCode int, result interface{}, err e
 	flags := pflag.NewFlagSet(config.Args[0], pflag.ContinueOnError)
 	flags.StringVarP(&profileFlag, "profile", "p", "", "The config profile to use")
 	flags.StringVarP(&configPathFlag, "config", "c", "", "The path to the config file")
-	flags.StringVarP(&outputFlag, "output", "o", "human", "Output format: json or human")
+	flags.StringVarP(&outputFlag, "output", "o", defaultOutput, "Output format: json or human")
 	flags.BoolVarP(&debug, "debug", "D", os.Getenv("SCW_DEBUG") == "true", "Enable debug mode")
 	// Ignore unknown flag
 	flags.ParseErrorsWhitelist.UnknownFlags = true
@@ -96,11 +100,15 @@ func Bootstrap(config *BootstrapConfig) (exitCode int, result interface{}, err e
 
 	// If debug flag is set enable debug mode in SDK logger
 	logLevel := logger.LogLevelWarning
+	if outputFlag != defaultOutput {
+		logLevel = logger.LogLevelError
+	}
+
 	if debug {
 		logLevel = logger.LogLevelDebug // enable debug mode
 	}
 
-	// We force log to os.Stderr because we dont have a scoped logger feature and it create
+	// We force log to os.Stderr because we don't have a scoped logger feature, and it creates
 	// concurrency situation with golden files
 	log := config.Logger
 	if log == nil {