openyurtio / openyurt

OpenYurt - Extending your native Kubernetes to edge(project under CNCF)

Home Page:https://openyurt.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[feature request] Add related validation into yurtappset webhook

vie-serendipity opened this issue · comments

What would you like to be added:
It is necessary to add related validation about yurtappset. Like native kubernetes resources, add validation of yurtappset created by users into webhook.

Why is this needed:
First, kubernetes do validation about native resources in the webhook. It is natural to extend this style.
Second, it is complicated to handle problems caused by invalid resources which may be produced by users. Subsequently, related controllerrevision handling will be tricky. We have to check whether it is invalid.

if isRevisionInvalid(expecteddRevison) {

Finally, It is also unreasonable that a user need to check conditions of status after creating an invalid resource instead of just failing to create it.

others
/kind feature

@vie-serendipity yes, I agree with you that yurtappset webhook should validate the deployment template instead of K8s native validation.

commented

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.