Skip to content

Commit c4a03fa

Browse files
committed
test(operator): spell out failurePolicy semantics in webhook assertion comment
Signed-off-by: gshaibi <gshaibi@nvidia.com>
1 parent 19cda79 commit c4a03fa

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

pkg/operator/operands/admission/admission_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,8 @@ var _ = Describe("Admission", func() {
8383
Expect(mutatingWebhook.Webhooks).To(HaveLen(1))
8484
Expect(mutatingWebhook.Webhooks[0].ClientConfig.CABundle).To(Equal(secret.Data[certKey]))
8585

86-
// Webhooks: pods (Fail), pods/resize (Ignore), topology (Fail).
86+
// failurePolicy per webhook: pods=Fail, topology=Fail, pods/resize=Ignore
87+
// (fail-open: the apiserver admits the request if the webhook call fails).
8788
Expect(validatingWebhook.Webhooks).To(HaveLen(3))
8889
for _, wh := range validatingWebhook.Webhooks {
8990
Expect(wh.ClientConfig.CABundle).To(Equal(secret.Data[certKey]))

0 commit comments

Comments
 (0)