capture the optional managed identities proposal using API Model.

Author: machi1990

model/clusters_mgmt/v1/azure_type.model

@@ -97,10 +97,13 @@ struct Azure {
 	// Immutable.
 	NetworkSecurityGroupResourceID String
 
+    // This is renamed to 'required_operators_authentication'.
+	// It brings concerns of; what if one of the operators stop being required anymore all the time?
+	// How do people see the design considering the future evolutions concerns?
 	// Defines how the operators of the cluster authenticate to Azure.
 	// Required during creation.
 	// Immutable.
-	OperatorsAuthentication AzureOperatorsAuthentication
+	RequiredOperatorsAuthentication AzureOperatorsAuthentication
 
 	// NodesOutboundConnectivity defines how the network outbound
 	// configuration of the Cluster's Node Pool's Nodes is performed.

model/clusters_mgmt/v1/cluster_image_registry_type.model

@@ -0,0 +1,4 @@
+struct ClusterImageRegistry {
+	ControlPlaneIdentity AzureControlPlaneManagedIdentity
+	DataPlaneIdentity AzureDataPlaneManagedIdentity
+}

model/clusters_mgmt/v1/cluster_type.model

@@ -247,5 +247,11 @@ class Cluster {
 	RegistryConfig ClusterRegistryConfig
 
 	// OpenShift Cluster Capabilities configuration
-	Capabilities ClusterCapabilities
+	// Commented because it'll be removed from the API as par the proposal in the DDR
+	// Capabilities ClusterCapabilities
+
+    // At the moment it is top level at the cluster's spec.
+	// We could move it to the .Azure field as it contains Azure specific things.
+	// This is the one point we want to solve
+	ImageRegistry ClusterImageRegistry
 }