Skip to content

Commit 5507072

Browse files
authored
[CLI-3530] Only display produce/consume info messages when the verbosity level is at least info (#3208)
1 parent f2982d2 commit 5507072

File tree

2 files changed

+18
-4
lines changed

2 files changed

+18
-4
lines changed

internal/asyncapi/command_export.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -708,7 +708,7 @@ func addComponents(reflector asyncapi.Reflector, messages map[string]spec.Messag
708708
}
709709

710710
func createConsumer(broker string, clusterCreds *config.APIKeyPair, group string) (*ckgo.Consumer, error) {
711-
consumer, err := ckgo.NewConsumer(&ckgo.ConfigMap{
711+
configMap := &ckgo.ConfigMap{
712712
"bootstrap.servers": broker,
713713
"sasl.mechanisms": "PLAIN",
714714
"security.protocol": "SASL_SSL",
@@ -717,7 +717,11 @@ func createConsumer(broker string, clusterCreds *config.APIKeyPair, group string
717717
"group.id": group,
718718
"auto.offset.reset": "earliest",
719719
"enable.auto.commit": "false",
720-
})
720+
}
721+
if err := kafka.SetConsumerDebugOption(configMap); err != nil {
722+
return nil, fmt.Errorf("failed to create Kafka consumer: %w", err)
723+
}
724+
consumer, err := ckgo.NewConsumer(configMap)
721725
if err != nil {
722726
return nil, fmt.Errorf("failed to create Kafka consumer: %w", err)
723727
}

internal/kafka/confluent_kafka_configs.go

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -334,23 +334,33 @@ func getPartitionsByIndex(partitions []ckgo.TopicPartition, partitionFilter Part
334334
}
335335

336336
func SetProducerDebugOption(configMap *ckgo.ConfigMap) error {
337+
// Note: log_levels are based on syslog levels
337338
switch log.CliLogger.Level {
339+
case log.WARN:
340+
return configMap.Set("log_level=4") // Warn level and above
341+
case log.INFO:
342+
return configMap.Set("log_level=6") // Info level and above
338343
case log.DEBUG:
339344
return configMap.Set("debug=broker, topic, msg, protocol")
340345
case log.TRACE, log.UNSAFE_TRACE:
341346
return configMap.Set("debug=all")
342347
}
343-
return nil
348+
return configMap.Set("log_level=3") // error level and above
344349
}
345350

346351
func SetConsumerDebugOption(configMap *ckgo.ConfigMap) error {
352+
// Note: log_levels are based on syslog levels
347353
switch log.CliLogger.Level {
354+
case log.WARN:
355+
return configMap.Set("log_level=4") // Warn level and above
356+
case log.INFO:
357+
return configMap.Set("log_level=6") // Info level and above
348358
case log.DEBUG:
349359
return configMap.Set("debug=broker, topic, msg, protocol, consumer, cgrp, fetch")
350360
case log.TRACE, log.UNSAFE_TRACE:
351361
return configMap.Set("debug=all")
352362
}
353-
return nil
363+
return configMap.Set("log_level=3") // error level and above
354364
}
355365

356366
func newProducerWithOverwrittenConfigs(configMap *ckgo.ConfigMap, configPath string, configStrings []string) (*ckgo.Producer, error) {

0 commit comments

Comments
 (0)