Setup validation and defaulting webhook

[ahg-g]

No description provided.

[danielvegamyhre]

/assign

[ahg-g]

/unassign @danielvegamyhre

[kannon92]

I remember that Kueue had some questions around CEL and webhooks. Do we want to go forward with the assumption that we could use CEL?

kubernetes-sigs/kueue#463

Not sure what backwards capability we want to target so curious if we could include some of these?

[ahg-g]

Using CEL would certainly be better, lets start with that and see how far we can go. Does it cover defaulting too?

[ahg-g]

Things to validate:

• EnableDNSHostnames is allowed only with completion mode Indexed.

[kannon92]

Should we close this issue?

[ahg-g]

Yes

[ahg-g]

/reopen

we need to enforce immutability of most fields in the JobSetSpec, I think the only field we should allow for now is suspend

[k8s-ci-robot]

@ahg-g: Reopened this issue.

In response to this:

/reopen

we need to enforce immutability of most fields in the JobSetSpec, I think the only field we should allow for now is suspend

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[ahg-g]

We may need to setup a defaulting webhook if we are to default things within the job template, like Indexed mode.

[danielvegamyhre]

/assign