style: rename unused variables to _

Signed-off-by: Chris Privitere <[email protected]>

Author: cprivitere

api/v1beta1/packetmachinetemplate_webhook.go

@@ -42,7 +42,7 @@ func (m *PacketMachineTemplate) ValidateCreate() error {
 }
 
 // ValidateUpdate implements webhook.Validator so a webhook will be registered for the type.
-func (m *PacketMachineTemplate) ValidateUpdate(old runtime.Object) error {
+func (m *PacketMachineTemplate) ValidateUpdate(_ runtime.Object) error {
 	machineTemplateLog.Info("validate update", "name", m.Name)
 
 	return nil

controllers/packetcluster_controller.go

@@ -161,7 +161,7 @@ func (r *PacketClusterReconciler) reconcileNormal(ctx context.Context, clusterSc
 	return ctrl.Result{}, nil
 }
 
-func (r *PacketClusterReconciler) reconcileDelete(ctx context.Context, clusterScope *scope.ClusterScope) (ctrl.Result, error) {
+func (r *PacketClusterReconciler) reconcileDelete(_ context.Context, _ *scope.ClusterScope) (ctrl.Result, error) {
 	// Initially I created this handler to remove an elastic IP when a cluster
 	// gets delete, but it does not sound like a good idea.  It is better to
 	// leave to the users the ability to decide if they want to keep and resign

pkg/cloud/packet/client.go

@@ -244,7 +244,7 @@ func (p *Client) GetDeviceByTags(ctx context.Context, project string, tags []str
 
 // CreateIP reserves an IP via Packet API. The request fails straight if no IP are available for the specified project.
 // This prevent the cluster to become ready.
-func (p *Client) CreateIP(ctx context.Context, namespace, clusterName, projectID, facility, metro string) (net.IP, error) {
+func (p *Client) CreateIP(ctx context.Context, _, clusterName, projectID, facility, metro string) (net.IP, error) {
 	failOnApprovalRequired := true
 	req := metal.IPReservationRequestInput{
 		Type:                   "public_ipv4",