From c25593142f931439f37620bddec7fa16d3140bb5 Mon Sep 17 00:00:00 2001 From: "W. Trevor King" Date: Tue, 19 Nov 2019 21:28:49 -0800 Subject: [PATCH] pkg/actuators/machine/actuator: Drop setMachineCloudProviderSpecifics comment The updateProviderID comment was probably a copy/paste error when setMachineCloudProviderSpecifics landed in 716083f839 (Set additional machine annotations/labels to get pretty machine output, 2019-07-23, openshift/cluster-api-provider-aws#242). I haven't read through the function carefully enough to be able to propose a comment that adds much beyond the function name, which is fairly clear on its own. But I know the updateProviderID comment is not a good fit ;). --- pkg/actuators/machine/actuator.go | 1 - 1 file changed, 1 deletion(-) diff --git a/pkg/actuators/machine/actuator.go b/pkg/actuators/machine/actuator.go index 9d5cd68c1c..0316e109aa 100644 --- a/pkg/actuators/machine/actuator.go +++ b/pkg/actuators/machine/actuator.go @@ -130,7 +130,6 @@ func (a *Actuator) Create(context context.Context, cluster *clusterv1.Cluster, m return a.updateStatus(updatedMachine, instance) } -// updateProviderID adds providerID in the machine spec func (a *Actuator) setMachineCloudProviderSpecifics(machine *machinev1.Machine, instance *ec2.Instance) (*machinev1.Machine, error) { if instance == nil { return machine, nil