Jelajahi Sumber

[ML] do not write audit messages for already deleted jobs (#71420)

Audit messages are written on job deletion failure. 

This makes sense unless the failure is caused by the job not being found

closes: https://github.com/elastic/elasticsearch/issues/71414
Benjamin Trent 4 tahun lalu
induk
melakukan
a0e9380e8f

+ 3 - 1
x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/action/TransportDeleteJobAction.java

@@ -176,7 +176,9 @@ public class TransportDeleteJobAction extends AcknowledgedTransportMasterNodeAct
                 ack -> notifyListeners(request.getJobId(), ack, null),
                 e -> {
                     notifyListeners(request.getJobId(), null, e);
-                    auditor.error(request.getJobId(), Messages.getMessage(Messages.JOB_AUDIT_DELETING_FAILED, e.getMessage()));
+                    if ((ExceptionsHelper.unwrapCause(e) instanceof ResourceNotFoundException) == false) {
+                        auditor.error(request.getJobId(), Messages.getMessage(Messages.JOB_AUDIT_DELETING_FAILED, e.getMessage()));
+                    }
                 }
         );