You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As @hiddeco pointed out, recording Events on ValidatingWebhook doesn't guarantee that the given event is really happened or not because other ValidatingWebhook in cluster can affect the result of validation.
Proposal
Instead of recording events in ValidatingWebhook,
MutatingWebhook annotates event payload to target resources
Controller records event in initial phase of reconciliation loop if payload presents in annotation and clear from annotations
@devholic I'll leave it up to you whether this needs to be changed in the v0.6.0 timeline or if the current behavior is acceptable for now, making this lower priority. Your call.
Background
As @hiddeco pointed out, recording Events on ValidatingWebhook doesn't guarantee that the given event is really happened or not because other ValidatingWebhook in cluster can affect the result of validation.
Proposal
Instead of recording events in ValidatingWebhook,
References
The text was updated successfully, but these errors were encountered: