Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions x-pack/filebeat/input/cel/input.go
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@
func (input) Name() string { return inputName }

func (input) Test(src inputcursor.Source, _ v2.TestContext) error {
cfg := src.(*source).cfg

Check failure on line 103 in x-pack/filebeat/input/cel/input.go

View workflow job for this annotation

GitHub Actions / lint (ubuntu-latest)

Error return value is not checked (errcheck)
if !wantClient(cfg) {
return nil
}
Expand All @@ -110,7 +110,7 @@
// Run starts the input and blocks until it ends completes. It will return on
// context cancellation or type invalidity errors, any other error will be retried.
func (input) Run(env v2.Context, src inputcursor.Source, crsr inputcursor.Cursor, pub inputcursor.Publisher) error {
dataStreamName := src.(*source).cfg.DataStream // May be empty.

Check failure on line 113 in x-pack/filebeat/input/cel/input.go

View workflow job for this annotation

GitHub Actions / lint (ubuntu-latest)

Error return value is not checked (errcheck)

var cursor map[string]interface{}
env.UpdateStatus(status.Starting, dataStreamName)
Expand All @@ -128,7 +128,7 @@
parent: &env,
}
}
err := input{}.run(env, src.(*source), cursor, pub, health)

Check failure on line 131 in x-pack/filebeat/input/cel/input.go

View workflow job for this annotation

GitHub Actions / lint (ubuntu-latest)

Error return value is not checked (errcheck)
if err != nil {
msg := "failed to run: " + err.Error()
if dataStreamName != "" {
Expand Down Expand Up @@ -541,6 +541,25 @@
pubCursor = cursor
}
}
// This is checked prior to the publish attempt since the
// cursor.Publisher interface does not document the behaviour
// related to context cancellation and the context is not
// explicitly passed in, so favour this explicit clarity.
switch err := ctx.Err(); {
case err == nil:
case errors.Is(err, context.Canceled), errors.Is(err, context.DeadlineExceeded):
log.Infow("context cancelled with unpublished events", "unpublished", len(events)-i)
// Don't update status, since we are about to pass
// through the Running state and then fall through
// to the input exit with a change to Stopped.
break

Check failure on line 555 in x-pack/filebeat/input/cel/input.go

View workflow job for this annotation

GitHub Actions / lint (ubuntu-latest)

S1023: redundant break statement (staticcheck)
default:
// This should never happen.
log.Warnw("failed with unpublished events", "error", err, "unpublished", len(events)-i)
health.UpdateStatus(status.Degraded, "error publishing events: "+err.Error())
isDegraded = true
break

Check failure on line 561 in x-pack/filebeat/input/cel/input.go

View workflow job for this annotation

GitHub Actions / lint (ubuntu-latest)

S1023: redundant break statement (staticcheck)
}
err = pub.Publish(beat.Event{
Timestamp: time.Now(),
Fields: event,
Expand Down
Loading