Enable optionalorrequired linter

Author: moko-poi

.golangci-kal.yml

@@ -24,7 +24,7 @@ linters:
               #- "maxlength" # Ensure all strings and arrays have maximum lengths/maximum items.
               #- "nobools" # Bools do not evolve over time, should use enums instead.
               #- "nofloats" # Ensure floats are not used.
-              #- "optionalorrequired" # Every field should be marked as `+optional` or `+required`.
+              - "optionalorrequired" # Every field should be marked as `+optional` or `+required`.
               # - "requiredfields" # Required fields should not be pointers, and should not have `omitempty`.
               - "statussubresource" # All root objects that have a `status` field should have a status subresource.
 

api/v1beta1/awscluster_types.go

@@ -34,9 +34,11 @@ const (
 // AWSClusterSpec defines the desired state of an EC2-based Kubernetes cluster.
 type AWSClusterSpec struct {
 	// NetworkSpec encapsulates all things related to AWS network.
+	// +optional
 	NetworkSpec NetworkSpec `json:"network,omitempty"`
 
 	// The AWS Region the cluster lives in.
+	// +optional
 	Region string `json:"region,omitempty"`
 
 	// SSHKeyName is the name of the ssh key to attach to the bastion host. Valid values are empty string (do not use SSH keys), a valid SSH key name, or omitted (use the default SSH key name)
@@ -81,6 +83,7 @@ type AWSClusterSpec struct {
 	// up machine images when a machine does not specify an AMI. When set, this
 	// will be used for all cluster machines unless a machine specifies a
 	// different ImageLookupBaseOS.
+	// +optional
 	ImageLookupBaseOS string `json:"imageLookupBaseOS,omitempty"`
 
 	// Bastion contains options to configure the bastion host.
@@ -91,6 +94,7 @@ type AWSClusterSpec struct {
 
 	// IdentityRef is a reference to an identity to be used when reconciling the managed control plane.
 	// If no identity is specified, the default identity for this controller will be used.
+	// +optional
 	IdentityRef *AWSIdentityReference `json:"identityRef,omitempty"`
 
 	// S3Bucket contains options to configure a supporting S3 bucket for this
@@ -119,10 +123,12 @@ var (
 type AWSIdentityReference struct {
 	// Name of the identity.
 	// +kubebuilder:validation:MinLength=1
+	// +required
 	Name string `json:"name"`
 
 	// Kind of the identity.
 	// +kubebuilder:validation:Enum=AWSClusterControllerIdentity;AWSClusterRoleIdentity;AWSClusterStaticIdentity
+	// +required
 	Kind AWSIdentityKind `json:"kind"`
 }
 
@@ -146,6 +152,7 @@ type Bastion struct {
 	// InstanceType will use the specified instance type for the bastion. If not specified,
 	// Cluster API Provider AWS will use t3.micro for all regions except us-east-1, where t2.micro
 	// will be the default.
+	// +optional
 	InstanceType string `json:"instanceType,omitempty"`
 
 	// AMI will use the specified AMI to boot the bastion. If not specified,
@@ -200,27 +207,35 @@ type AWSLoadBalancerSpec struct {
 // AWSClusterStatus defines the observed state of AWSCluster.
 type AWSClusterStatus struct {
 	// +kubebuilder:default=false
+	// +required
 	Ready          bool                     `json:"ready"`
+	// +optional
 	Network        NetworkStatus            `json:"networkStatus,omitempty"`
+	// +optional
 	FailureDomains clusterv1.FailureDomains `json:"failureDomains,omitempty"`
+	// +optional
 	Bastion        *Instance                `json:"bastion,omitempty"`
+	// +optional
 	Conditions     clusterv1.Conditions     `json:"conditions,omitempty"`
 }
 
 // S3Bucket defines a supporting S3 bucket for the cluster, currently can be optionally used for Ignition.
 type S3Bucket struct {
 	// ControlPlaneIAMInstanceProfile is a name of the IAMInstanceProfile, which will be allowed
 	// to read control-plane node bootstrap data from S3 Bucket.
+	// +required
 	ControlPlaneIAMInstanceProfile string `json:"controlPlaneIAMInstanceProfile"`
 
 	// NodesIAMInstanceProfiles is a list of IAM instance profiles, which will be allowed to read
 	// worker nodes bootstrap data from S3 Bucket.
+	// +required
 	NodesIAMInstanceProfiles []string `json:"nodesIAMInstanceProfiles"`
 
 	// Name defines name of S3 Bucket to be created.
 	// +kubebuilder:validation:MinLength:=3
 	// +kubebuilder:validation:MaxLength:=63
 	// +kubebuilder:validation:Pattern=`^[a-z0-9][a-z0-9.-]{1,61}[a-z0-9]$`
+	// +required
 	Name string `json:"name"`
 }
 
@@ -237,9 +252,12 @@ type S3Bucket struct {
 // AWSCluster is the schema for Amazon EC2 based Kubernetes Cluster API.
 type AWSCluster struct {
 	metav1.TypeMeta   `json:",inline"`
+	// +optional
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
+	// +optional
 	Spec   AWSClusterSpec   `json:"spec,omitempty"`
+	// +optional
 	Status AWSClusterStatus `json:"status,omitempty"`
 }
 
@@ -249,7 +267,9 @@ type AWSCluster struct {
 // AWSClusterList contains a list of AWSCluster.
 type AWSClusterList struct {
 	metav1.TypeMeta `json:",inline"`
+	// +optional
 	metav1.ListMeta `json:"metadata,omitempty"`
+	// +required
 	Items           []AWSCluster `json:"items"`
 }
 

api/v1beta1/awsclustertemplate_types.go

@@ -24,6 +24,7 @@ import (
 
 // AWSClusterTemplateSpec defines the desired state of AWSClusterTemplate.
 type AWSClusterTemplateSpec struct {
+	// +required
 	Template AWSClusterTemplateResource `json:"template"`
 }
 
@@ -35,8 +36,10 @@ type AWSClusterTemplateSpec struct {
 // AWSClusterTemplate is the schema for Amazon EC2 based Kubernetes Cluster Templates.
 type AWSClusterTemplate struct {
 	metav1.TypeMeta   `json:",inline"`
+	// +optional
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
+	// +optional
 	Spec AWSClusterTemplateSpec `json:"spec,omitempty"`
 }
 
@@ -45,7 +48,9 @@ type AWSClusterTemplate struct {
 // AWSClusterTemplateList contains a list of AWSClusterTemplate.
 type AWSClusterTemplateList struct {
 	metav1.TypeMeta `json:",inline"`
+	// +optional
 	metav1.ListMeta `json:"metadata,omitempty"`
+	// +required
 	Items           []AWSClusterTemplate `json:"items"`
 }
 
@@ -59,5 +64,6 @@ type AWSClusterTemplateResource struct {
 	// More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#metadata
 	// +optional
 	ObjectMeta clusterv1.ObjectMeta `json:"metadata,omitempty"`
+	// +required
 	Spec       AWSClusterSpec       `json:"spec"`
 }

api/v1beta1/awsidentity_types.go

@@ -53,19 +53,24 @@ type AllowedNamespaces struct {
 // AWSRoleSpec defines the specifications for all identities based around AWS roles.
 type AWSRoleSpec struct {
 	// The Amazon Resource Name (ARN) of the role to assume.
+	// +required
 	RoleArn string `json:"roleARN"`
 	// An identifier for the assumed role session
+	// +optional
 	SessionName string `json:"sessionName,omitempty"`
 	// The duration, in seconds, of the role session before it is renewed.
 	// +kubebuilder:validation:Minimum:=900
 	// +kubebuilder:validation:Maximum:=43200
+	// +optional
 	DurationSeconds int32 `json:"durationSeconds,omitempty"`
 	// An IAM policy as a JSON-encoded string that you want to use as an inline session policy.
+	// +optional
 	InlinePolicy string `json:"inlinePolicy,omitempty"`
 
 	// The Amazon Resource Names (ARNs) of the IAM managed policies that you want
 	// to use as managed session policies.
 	// The policies must exist in the same account as the role.
+	// +optional
 	PolicyARNs []string `json:"policyARNs,omitempty"`
 }
 
@@ -77,9 +82,11 @@ type AWSRoleSpec struct {
 // It represents a reference to an AWS access key ID and secret access key, stored in a secret.
 type AWSClusterStaticIdentity struct {
 	metav1.TypeMeta   `json:",inline"`
+	// +optional
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
 	// Spec for this AWSClusterStaticIdentity
+	// +optional
 	Spec AWSClusterStaticIdentitySpec `json:"spec,omitempty"`
 }
 
@@ -89,7 +96,9 @@ type AWSClusterStaticIdentity struct {
 // AWSClusterStaticIdentityList contains a list of AWSClusterStaticIdentity.
 type AWSClusterStaticIdentityList struct {
 	metav1.TypeMeta `json:",inline"`
+	// +optional
 	metav1.ListMeta `json:"metadata,omitempty"`
+	// +required
 	Items           []AWSClusterStaticIdentity `json:"items"`
 }
 
@@ -101,6 +110,7 @@ type AWSClusterStaticIdentitySpec struct {
 	//  AccessKeyID: AKIAIOSFODNN7EXAMPLE
 	//  SecretAccessKey: wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY
 	//  SessionToken: Optional
+	// +required
 	SecretRef string `json:"secretRef"`
 }
 
@@ -112,9 +122,11 @@ type AWSClusterStaticIdentitySpec struct {
 // It is used to assume a role using the provided sourceRef.
 type AWSClusterRoleIdentity struct {
 	metav1.TypeMeta   `json:",inline"`
+	// +optional
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
 	// Spec for this AWSClusterRoleIdentity.
+	// +optional
 	Spec AWSClusterRoleIdentitySpec `json:"spec,omitempty"`
 }
 
@@ -124,7 +136,9 @@ type AWSClusterRoleIdentity struct {
 // AWSClusterRoleIdentityList contains a list of AWSClusterRoleIdentity.
 type AWSClusterRoleIdentityList struct {
 	metav1.TypeMeta `json:",inline"`
+	// +optional
 	metav1.ListMeta `json:"metadata,omitempty"`
+	// +required
 	Items           []AWSClusterRoleIdentity `json:"items"`
 }
 
@@ -146,6 +160,7 @@ type AWSClusterRoleIdentitySpec struct {
 
 	// SourceIdentityRef is a reference to another identity which will be chained to do
 	// role assumption. All identity types are accepted.
+	// +optional
 	SourceIdentityRef *AWSIdentityReference `json:"sourceIdentityRef,omitempty"`
 }
 
@@ -157,9 +172,11 @@ type AWSClusterRoleIdentitySpec struct {
 // It is used to grant access to use Cluster API Provider AWS Controller credentials.
 type AWSClusterControllerIdentity struct {
 	metav1.TypeMeta   `json:",inline"`
+	// +optional
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
 	// Spec for this AWSClusterControllerIdentity.
+	// +optional
 	Spec AWSClusterControllerIdentitySpec `json:"spec,omitempty"`
 }
 
@@ -169,7 +186,9 @@ type AWSClusterControllerIdentity struct {
 // AWSClusterControllerIdentityList contains a list of AWSClusterControllerIdentity.
 type AWSClusterControllerIdentityList struct {
 	metav1.TypeMeta `json:",inline"`
+	// +optional
 	metav1.ListMeta `json:"metadata,omitempty"`
+	// +required
 	Items           []AWSClusterControllerIdentity `json:"items"`
 }
 

api/v1beta1/awsmachine_types.go

@@ -45,12 +45,15 @@ var (
 // AWSMachineSpec defines the desired state of an Amazon EC2 instance.
 type AWSMachineSpec struct {
 	// ProviderID is the unique identifier as specified by the cloud provider.
+	// +optional
 	ProviderID *string `json:"providerID,omitempty"`
 
 	// InstanceID is the EC2 instance ID for this machine.
+	// +optional
 	InstanceID *string `json:"instanceID,omitempty"`
 
 	// AMI is the reference to the AMI from which to create the machine instance.
+	// +optional
 	AMI AMIReference `json:"ami,omitempty"`
 
 	// ImageLookupFormat is the AMI naming format to look up the image for this
@@ -68,14 +71,16 @@ type AWSMachineSpec struct {
 	ImageLookupFormat string `json:"imageLookupFormat,omitempty"`
 
 	// ImageLookupOrg is the AWS Organization ID to use for image lookup if AMI is not set.
+	// +optional
 	ImageLookupOrg string `json:"imageLookupOrg,omitempty"`
 
 	// ImageLookupBaseOS is the name of the base operating system to use for
 	// image lookup the AMI is not set.
+	// +optional
 	ImageLookupBaseOS string `json:"imageLookupBaseOS,omitempty"`
 
 	// InstanceType is the type of instance to create. Example: m4.xlarge
-	// +kubebuilder:validation:Required
+	// +required
 	// +kubebuilder:validation:MinLength:=2
 	InstanceType string `json:"instanceType"`
 
@@ -107,6 +112,7 @@ type AWSMachineSpec struct {
 	// FailureDomain is the failure domain unique identifier this Machine should be attached to, as defined in Cluster API.
 	// For this infrastructure provider, the ID is equivalent to an AWS Availability Zone.
 	// If multiple subnets are matched for the availability zone, the first one returned is picked.
+	// +optional
 	FailureDomain *string `json:"failureDomain,omitempty"`
 
 	// Subnet is a reference to the subnet to use for this instance. If not specified,
@@ -165,6 +171,7 @@ type CloudInit struct {
 	// or AWS Systems Manager Parameter Store to ensure privacy of userdata.
 	// By default, a cloud-init boothook shell script is prepended to download
 	// the userdata from Secrets Manager and additionally delete the secret.
+	// +optional
 	InsecureSkipSecretsManager bool `json:"insecureSkipSecretsManager,omitempty"`
 
 	// SecretCount is the number of secrets used to form the complete secret
@@ -207,6 +214,7 @@ type AWSMachineStatus struct {
 	Interruptible bool `json:"interruptible,omitempty"`
 
 	// Addresses contains the AWS instance associated addresses.
+	// +optional
 	Addresses []clusterv1.MachineAddress `json:"addresses,omitempty"`
 
 	// InstanceState is the state of the AWS instance for this machine.
@@ -269,9 +277,12 @@ type AWSMachineStatus struct {
 // AWSMachine is the schema for Amazon EC2 machines.
 type AWSMachine struct {
 	metav1.TypeMeta   `json:",inline"`
+	// +optional
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
+	// +optional
 	Spec   AWSMachineSpec   `json:"spec,omitempty"`
+	// +optional
 	Status AWSMachineStatus `json:"status,omitempty"`
 }
 
@@ -291,7 +302,9 @@ func (r *AWSMachine) SetConditions(conditions clusterv1.Conditions) {
 // AWSMachineList contains a list of Amazon EC2 machines.
 type AWSMachineList struct {
 	metav1.TypeMeta `json:",inline"`
+	// +optional
 	metav1.ListMeta `json:"metadata,omitempty"`
+	// +required
 	Items           []AWSMachine `json:"items"`
 }
 

api/v1beta1/awsmachinetemplate_types.go

@@ -34,6 +34,7 @@ type AWSMachineTemplateStatus struct {
 
 // AWSMachineTemplateSpec defines the desired state of AWSMachineTemplate.
 type AWSMachineTemplateSpec struct {
+	// +required
 	Template AWSMachineTemplateResource `json:"template"`
 }
 
@@ -44,9 +45,12 @@ type AWSMachineTemplateSpec struct {
 // AWSMachineTemplate is the schema for the Amazon EC2 Machine Templates API.
 type AWSMachineTemplate struct {
 	metav1.TypeMeta   `json:",inline"`
+	// +optional
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
+	// +optional
 	Spec   AWSMachineTemplateSpec   `json:"spec,omitempty"`
+	// +optional
 	Status AWSMachineTemplateStatus `json:"status,omitempty"`
 }
 
@@ -56,7 +60,9 @@ type AWSMachineTemplate struct {
 // AWSMachineTemplateList contains a list of AWSMachineTemplate.
 type AWSMachineTemplateList struct {
 	metav1.TypeMeta `json:",inline"`
+	// +optional
 	metav1.ListMeta `json:"metadata,omitempty"`
+	// +required
 	Items           []AWSMachineTemplate `json:"items"`
 }
 
@@ -68,6 +74,7 @@ type AWSMachineTemplateResource struct {
 	ObjectMeta clusterv1.ObjectMeta `json:"metadata,omitempty"`
 
 	// Spec is the specification of the desired behavior of the machine.
+	// +required
 	Spec AWSMachineSpec `json:"spec"`
 }