canonical / admission-webhook-operator

Admission Webhook Operator
Apache License 2.0
1 stars 4 forks source link

fix: refactor event handling to address certs error #125

Closed NohaIhab closed 6 months ago

NohaIhab commented 6 months ago

closes https://github.com/canonical/admission-webhook-operator/issues/126 debugging details in #124 (edit by @ca-scribner, pretty sure we mean issue 124 in admission-webhook)

NohaIhab commented 6 months ago

thanks @DnPlas for the review, I'm with you on implementing an all-catch event handler and leaving it at that, although I think we still need this check to ensure update_layer is never called unless we have pushed the files successfully.