add admission webhook validation for service export request

[olamilekan000]

Summary

change moves validatiion logic for  APIServiceExportRequest to validation webhook

What Type of PR Is This?

/kind feature

Related Issue(s)

Fixes 325

Release Notes

add admission webhook validation for service export request

[coderabbitai]

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)

• ai-code-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[mjudeikis]

@olamilekan000 will review this bit later. Need few days to sort some other things. Have you been able to test this while deploying kube-bind backend and trying it?

[olamilekan000]

@olamilekan000 will review this bit later. Need few days to sort some other things. Have you been able to test this while deploying kube-bind backend and trying it?

No problem. Yes I did. Here's a screenshot

[olamilekan000]

I almost forgot about this PR. In line with @xrstf's Tedtalk, I realize that the current PR implementation goes against the point he has raised, which I agree with. To correct this, should the PR pivot to:

• Use CEL for stateless structural validation within the CRD (This will solve the issue with generating certificates)?
• Move the referential validation (checking against BoundSchemas) into the Reconciler, allowing the object to be created but marking it as 'Invalid/Pending' in the status? (If you have a better suggestion, I don't mind)"

cc @mjudeikis @cnvergence

[mjudeikis]

make sense :) +1 on stateless

[cnvergence]

yeah, CEL seems right for it
thanks