I started on implementing this, but a big problem for CAPZ is that CAPI has bumped up to controller-runtime v0.20.3, but ASO imports v0.16.2 v0.19.5. This leads to compilation errors, especially WRT the removal of webhook default and validator.

We can fix up the CAPZ code to use WithDefaulter / WithValidator, but ASO won't compile:

# github.com/Azure/azure-service-operator/v2/api/resources/v1api20200601
../../go/pkg/mod/github.com/!azure/azure-service-operator/v2@v2.11.0/api/resources/v1api20200601/resource_group_types_gen.go:76:17: undefined: admission.Defaulter

cc: @nojnhuh @nawazkh

To clarify, ASO's go.mod at its root is for the v1 module. The v2 go.mod is at v2/go.mod, which is up to v0.19.5, but still not close enough apparrently 😄: https://github.com/Azure/azure-service-operator/blob/614d84503caf2a28ad2c22f97f7a38b50be22e2b/v2/go.mod#L64

Azure/azure-service-operator#4617 should bring that up in ASO's main branch to match what we need.

Aha, thanks for the clarification. But yes, the removal of the webhook.Default and webhook.Validator fields happened in v0.20. CAPZ may have to wait for a new ASO release with the above change in it.

Azure/azure-service-operator#4617 should bring that up in ASO's main branch to match what we need.

Azure/azure-service-operator#4658 seems the be the updated PR that got merged. Adding it here for quick reference.