Merge pull request #1781 from CecileRobertMichon/audit-api

Audit API to follow CAPI conventions

api/v1beta1/azurecluster_types.go

@@ -34,6 +34,7 @@ const (
 // AzureClusterSpec defines the desired state of AzureCluster.
 type AzureClusterSpec struct {
 	// NetworkSpec encapsulates all things related to Azure network.
+	// +optional
 	NetworkSpec NetworkSpec `json:"networkSpec,omitempty"`
 
 	// +optional
@@ -46,7 +47,7 @@ type AzureClusterSpec struct {
 
 	// ControlPlaneEndpoint represents the endpoint used to communicate with the control plane.
 	// +optional
-	ControlPlaneEndpoint clusterv1.APIEndpoint `json:"controlPlaneEndpoint"`
+	ControlPlaneEndpoint clusterv1.APIEndpoint `json:"controlPlaneEndpoint,omitempty"`
 
 	// AdditionalTags is an optional set of tags to add to Azure resources managed by the Azure provider, in addition to the
 	// ones added by default.
@@ -87,6 +88,7 @@ type AzureClusterStatus struct {
 	// the cluster is more resilient to failure.
 	// See: https://docs.microsoft.com/en-us/azure/availability-zones/az-overview
 	// This list will be used by Cluster API to try and spread the machines across the failure domains.
+	// +optional
 	FailureDomains clusterv1.FailureDomains `json:"failureDomains,omitempty"`
 
 	// Ready is true when the provider resource is ready.

api/v1beta1/azuremachine_types.go

@@ -40,6 +40,7 @@ type AzureMachineSpec struct {
 
 	// FailureDomain is the failure domain unique identifier this Machine should be attached to,
 	// as defined in Cluster API. This relates to an Azure Availability Zone
+	// +optional
 	FailureDomain *string `json:"failureDomain,omitempty"`
 
 	// Image is used to provide details of an image to use during VM creation.
@@ -74,6 +75,7 @@ type AzureMachineSpec struct {
 	OSDisk OSDisk `json:"osDisk"`
 
 	// DataDisk specifies the parameters that are used to add one or more data disks to the machine
+	// +optional
 	DataDisks []DataDisk `json:"dataDisks,omitempty"`
 
 	SSHPublicKey string `json:"sshPublicKey"`
@@ -128,6 +130,7 @@ type AzureMachineStatus struct {
 	Ready bool `json:"ready"`
 
 	// Addresses contains the Azure instance associated addresses.
+	// +optional
 	Addresses []v1.NodeAddress `json:"addresses,omitempty"`
 
 	// VMState is the provisioning state of the Azure virtual machine.