Not allow edit virtualcluster for some field

[jichenjc]

What steps did you take and what happened:
[A clear and concise description on how to REPRODUCE the bug.]

What did you expect to happen:

kubectl edit vc vc-sample-23

spec:
  clusterDomain: cluster.local
  clusterVersionName: cv-sample-np23 ===> cv-sample-np24

success

expected: it should fail, we should consider only allow desired field to be updated and block others ..

Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

Environment:

• cluster-api-provider-nested version:
• Minikube/KIND version:
• Kubernetes version: (use kubectl version):
• OS (e.g. from /etc/os-release):

/kind bug
[One or more /area label. See https://github.com/kubernetes-sigs/cluster-api-provider-nested/labels?q=area for the list of labels]

[jichenjc]

any thoughts on this? @christopherhein @Fei-Guo comments?

[jichenjc]

/assign

[Fei-Guo]

Yes. In current version, we don't consider vc upgrade through changing the cluster version name and we should add webhook to make this field immutable. We defer the upgrade story for cluster version based provisioning because we will design update worklflow based on CAPN in next step.

[christopherhein]

/close

Closing since this PR was implemented and merged.

[k8s-ci-robot]

@christopherhein: Closing this issue.

In response to this:

/close

Closing since this PR was implemented and merged.

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.