Remove unnecessary nolints

The bump to x/tools v0.39.0 changes the behaviour of the gosec linter,
which no longer complains about these valid conversions.

Author: mdbooth

pkg/controllers/infracluster/azure.go

@@ -276,8 +276,7 @@ func (r *InfraClusterController) newAzureCluster(providerSpec *mapiv1beta1.Azure
 			ResourceGroup: providerSpec.ResourceGroup,
 			ControlPlaneEndpoint: clusterv1beta1.APIEndpoint{
 				Host: apiURL.Hostname(),
-				// Port numbers are less than 65536, ignore gosec.
-				Port: int32(port), //nolint:gosec
+				Port: int32(port),
 			},
 		},
 	}

pkg/conversion/capi2mapi/aws.go

@@ -525,7 +525,7 @@ func convertAWSVolumeToMAPI(fldPath *field.Path, volume awsv1.Volume) (mapiv1bet
 		if *volume.Throughput > math.MaxInt32 {
 			return mapiv1beta1.BlockDeviceMappingSpec{}, field.Invalid(fldPath.Child("throughput"), *volume.Throughput, "throughput exceeds maximum int32 value")
 		}
-		//nolint:gosec
+
 		bdm.EBS.ThroughputMib = ptr.To(int32(*volume.Throughput))
 	}
 
@@ -548,8 +548,7 @@ func convertAWSPlacementGroupPartition(in int64) *int32 {
 	if in == 0 {
 		return nil
 	}
-	// We know the value is between 0 and 7 based on API validation. Ignore gosec.
-	//nolint:gosec
+
 	return ptr.To(int32(in))
 }