Skip to content

Commit

Permalink
feat(finalizer): don't create events for add/remove operations
Browse files Browse the repository at this point in the history
These events are somewhat spammy and not really actionable anyway.
  • Loading branch information
tronghn committed Oct 3, 2023
1 parent 673ecf2 commit 40ee100
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions pkg/reconciler/finalizer/finalizer.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ func (f finalizer) register(tx transaction.Transaction) error {
return fmt.Errorf("error when registering finalizer: %w", err)
}

f.ReportEvent(tx, corev1.EventTypeNormal, v1.EventAddedFinalizer, "Object finalizer is added")
return nil
}

Expand Down Expand Up @@ -90,7 +89,6 @@ func (f finalizer) finalize(tx transaction.Transaction) error {
return fmt.Errorf("failed to remove finalizer from list: %w", err)
}

f.ReportEvent(tx, corev1.EventTypeNormal, v1.EventDeletedFinalizer, "Object finalizer is deleted")
metrics.IncWithNamespaceLabel(metrics.AzureAppsDeletedCount, tx.Instance.Namespace)

return nil
Expand Down

0 comments on commit 40ee100

Please sign in to comment.