Skip to content

Commit

Permalink
fix(kafka-publish): don't fail silently (#3343)
Browse files Browse the repository at this point in the history
Signed-off-by: Benjamin Coenen <benjamin.coenen@corp.ovh.com>
  • Loading branch information
bnjjj authored and yesnault committed Sep 20, 2018
1 parent 2486ab4 commit cee0195
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions contrib/grpcplugins/action/kafka-publish/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -199,10 +199,7 @@ func (actPlugin *kafkaPublishActionPlugin) Run(ctxBack context.Context, q *actio

Logf("Ack Received: %+v\n", ack)

return &actionplugin.ActionResult{
Status: sdk.StatusSuccess.String(),
Details: fmt.Sprintf("Plugin failed with ACK.Result:%s", ack.Result),
}, nil
return fail("Plugin failed with ACK.Result:%s", ack.Result)
}

func (actPlugin *kafkaPublishActionPlugin) WorkerHTTPPort(ctx context.Context, q *actionplugin.WorkerHTTPPortQuery) (*empty.Empty, error) {
Expand Down

0 comments on commit cee0195

Please sign in to comment.