Merge pull request #6290 from seanlaii/fix-ST1005
Fix lint issue ST1005
This commit is contained in:
commit
83a4bb853a
|
@ -70,7 +70,6 @@ linters:
|
||||||
# Will fix the issues in the following PRs.
|
# Will fix the issues in the following PRs.
|
||||||
# Issue: https://github.com/karmada-io/karmada/issues/6273.
|
# Issue: https://github.com/karmada-io/karmada/issues/6273.
|
||||||
- "-QF1008"
|
- "-QF1008"
|
||||||
- "-ST1005"
|
|
||||||
- "-ST1011"
|
- "-ST1011"
|
||||||
- "-QF1001"
|
- "-QF1001"
|
||||||
- "-ST1000"
|
- "-ST1000"
|
||||||
|
|
|
@ -117,7 +117,7 @@ func generateCMDs(module string) ([]*cobra.Command, error) {
|
||||||
cmds = append(cmds, webhookapp.NewWebhookCommand(context.TODO()))
|
cmds = append(cmds, webhookapp.NewWebhookCommand(context.TODO()))
|
||||||
cmds = append(cmds, adapterapp.NewMetricsAdapterCommand(context.TODO()))
|
cmds = append(cmds, adapterapp.NewMetricsAdapterCommand(context.TODO()))
|
||||||
default:
|
default:
|
||||||
return nil, fmt.Errorf("Module %s is not supported", module)
|
return nil, fmt.Errorf("module %s is not supported", module)
|
||||||
}
|
}
|
||||||
|
|
||||||
return cmds, nil
|
return cmds, nil
|
||||||
|
|
|
@ -233,7 +233,7 @@ func (c *FHPAController) reconcileAutoscaler(ctx context.Context, hpa *autoscali
|
||||||
if err := c.updateStatusIfNeeded(ctx, hpaStatusOriginal, hpa); err != nil {
|
if err := c.updateStatusIfNeeded(ctx, hpaStatusOriginal, hpa); err != nil {
|
||||||
utilruntime.HandleError(err)
|
utilruntime.HandleError(err)
|
||||||
}
|
}
|
||||||
return fmt.Errorf("Failed to get scale target reference: %v ", err)
|
return fmt.Errorf("failed to get scale target reference: %v ", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
binding, err := c.getBindingByLabel(ctx, targetResource.GetLabels(), hpa.Spec.ScaleTargetRef)
|
binding, err := c.getBindingByLabel(ctx, targetResource.GetLabels(), hpa.Spec.ScaleTargetRef)
|
||||||
|
|
Loading…
Reference in New Issue