Skip to content

Commit

Permalink
fix: avoid gotestsum import in libbeat es output (#39835)
Browse files Browse the repository at this point in the history
gotestsum is being imported causing downstream apps to include it
in the dependency tree.
Drop the import and use the client logger.
  • Loading branch information
kruskall authored Jun 10, 2024
1 parent 35f8d09 commit a2ab85d
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions libbeat/outputs/elasticsearch/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ import (
"time"

"go.elastic.co/apm/v2"
"gotest.tools/gotestsum/log"

"github.com/elastic/beats/v7/libbeat/beat"
"github.com/elastic/beats/v7/libbeat/beat/events"
Expand Down Expand Up @@ -433,7 +432,7 @@ func (client *Client) bulkCollectPublishFails(bulkResult bulkResult) ([]publishe

if client.applyItemStatus(events[i], itemStatus, itemMessage, &stats) {
eventsToRetry = append(eventsToRetry, events[i])
log.Debugf("Bulk item insert failed (i=%v, status=%v): %s", i, itemStatus, itemMessage)
client.log.Debugf("Bulk item insert failed (i=%v, status=%v): %s", i, itemStatus, itemMessage)
}
}

Expand Down

0 comments on commit a2ab85d

Please sign in to comment.