Commit 718a7479 authored by Patrick Barker's avatar Patrick Barker

fix missing event report handling in dynamic audit e2e

parent 62edfe8d
...@@ -370,13 +370,13 @@ var _ = SIGDescribe("[Feature:DynamicAudit]", func() { ...@@ -370,13 +370,13 @@ var _ = SIGDescribe("[Feature:DynamicAudit]", func() {
return false, err return false, err
} }
reader := strings.NewReader(logs) reader := strings.NewReader(logs)
missing, err := utils.CheckAuditLines(reader, expectedEvents, auditv1.SchemeGroupVersion) missingReport, err := utils.CheckAuditLines(reader, expectedEvents, auditv1.SchemeGroupVersion)
if err != nil { if err != nil {
framework.Logf("Failed to observe audit events: %v", err) framework.Logf("Failed to observe audit events: %v", err)
} else if len(missing) > 0 { } else if len(missingReport.MissingEvents) > 0 {
framework.Logf("Events %#v not found!", missing) framework.Logf(missingReport.String())
} }
return len(missing) == 0, nil return len(missingReport.MissingEvents) == 0, nil
}) })
framework.ExpectNoError(err, "after %v failed to observe audit events", pollingTimeout) framework.ExpectNoError(err, "after %v failed to observe audit events", pollingTimeout)
err = f.ClientSet.AuditregistrationV1alpha1().AuditSinks().Delete("test", &metav1.DeleteOptions{}) err = f.ClientSet.AuditregistrationV1alpha1().AuditSinks().Delete("test", &metav1.DeleteOptions{})
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment