Skip to content

Revert "Make the AuditEventStreamingWorker backward compatible temporarily"

Huzaifa Iftikhar requested to merge 362573_revert_temp_change_for_audit_event_streaming_worker into master

What does this MR do and why?

  1. The first commit reverts the temporary changes introduced in MR !87243 (merged)
  2. The second commit removes the if condition that checks if audit_operation is present or not. Since we always set the value of audit_operation while creating a job for this worker, it should be safe to assume that the worker will always have a non nil value for audit_operation. We also need to make sure that the payload of the streaming event is consistent so we should always set the event_type property in the JSON body and not conditionally.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Relates to #362573 (closed)

Merge request reports