chore: nit using lo.FromPtr is safer than directly accessing the poin…

…ter (#305) Co-authored-by: Alex Leites <[email protected]>
Azure · Apr 28, 2024 · aafcbe1 · aafcbe1
1 parent fc0b527
commit aafcbe1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/cloudprovider/cloudprovider.go b/pkg/cloudprovider/cloudprovider.go
@@ -99,7 +99,7 @@ func (c *CloudProvider) Create(ctx context.Context, nodeClaim *corev1beta1.NodeC
 		return nil, fmt.Errorf("creating instance, %w", err)
 	}
 	instanceType, _ := lo.Find(instanceTypes, func(i *cloudprovider.InstanceType) bool {
-		return i.Name == string(*instance.Properties.HardwareProfile.VMSize)
+		return i.Name == string(lo.FromPtr(instance.Properties.HardwareProfile.VMSize))
 	})
 
 	return c.instanceToNodeClaim(ctx, instance, instanceType)
@@ -279,7 +279,7 @@ func (c *CloudProvider) resolveInstanceTypeFromInstance(ctx context.Context, ins
 		return nil, client.IgnoreNotFound(fmt.Errorf("resolving node template, %w", err))
 	}
 	instanceType, _ := lo.Find(instanceTypes, func(i *cloudprovider.InstanceType) bool {
-		return i.Name == string(*instance.Properties.HardwareProfile.VMSize)
+		return i.Name == string(lo.FromPtr(instance.Properties.HardwareProfile.VMSize))
 	})
 	return instanceType, nil
 }