Skip to content

Commit cfc596b

Browse files
hvaarak8s-ci-robot
authored andcommitted
Fix golint issues in pkg/apis/autoscaling/validation (kubernetes#85041)
* Fix golint issues in pkg/apis/autoscaling/validation * Fix spelling issue in field of Error
1 parent 0167697 commit cfc596b

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed

hack/.golint_failures

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ pkg/apis/authorization/validation
2020
pkg/apis/autoscaling/v1
2121
pkg/apis/autoscaling/v2beta1
2222
pkg/apis/autoscaling/v2beta2
23-
pkg/apis/autoscaling/validation
2423
pkg/apis/batch/v1
2524
pkg/apis/batch/v1beta1
2625
pkg/apis/batch/v2alpha1

pkg/apis/autoscaling/validation/validation.go

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ import (
3030
"k8s.io/kubernetes/pkg/features"
3131
)
3232

33+
// ValidateScale validates a Scale and returns an ErrorList with any errors.
3334
func ValidateScale(scale *autoscaling.Scale) field.ErrorList {
3435
allErrs := field.ErrorList{}
3536
allErrs = append(allErrs, apivalidation.ValidateObjectMeta(&scale.ObjectMeta, true, apimachineryvalidation.NameIsDNSSubdomain, field.NewPath("metadata"))...)
@@ -41,7 +42,7 @@ func ValidateScale(scale *autoscaling.Scale) field.ErrorList {
4142
return allErrs
4243
}
4344

44-
// ValidateHorizontalPodAutoscaler can be used to check whether the given autoscaler name is valid.
45+
// ValidateHorizontalPodAutoscalerName can be used to check whether the given autoscaler name is valid.
4546
// Prefix indicates this name will be used as part of generation, in which case trailing dashes are allowed.
4647
var ValidateHorizontalPodAutoscalerName = apivalidation.ValidateReplicationControllerName
4748

@@ -67,6 +68,8 @@ func validateHorizontalPodAutoscalerSpec(autoscaler autoscaling.HorizontalPodAut
6768
return allErrs
6869
}
6970

71+
// ValidateCrossVersionObjectReference validates a CrossVersionObjectReference and returns an
72+
// ErrorList with any errors.
7073
func ValidateCrossVersionObjectReference(ref autoscaling.CrossVersionObjectReference, fldPath *field.Path) field.ErrorList {
7174
allErrs := field.ErrorList{}
7275
if len(ref.Kind) == 0 {
@@ -88,6 +91,8 @@ func ValidateCrossVersionObjectReference(ref autoscaling.CrossVersionObjectRefer
8891
return allErrs
8992
}
9093

94+
// ValidateHorizontalPodAutoscaler validates a HorizontalPodAutoscaler and returns an
95+
// ErrorList with any errors.
9196
func ValidateHorizontalPodAutoscaler(autoscaler *autoscaling.HorizontalPodAutoscaler) field.ErrorList {
9297
allErrs := apivalidation.ValidateObjectMeta(&autoscaler.ObjectMeta, true, ValidateHorizontalPodAutoscalerName, field.NewPath("metadata"))
9398

@@ -104,6 +109,8 @@ func ValidateHorizontalPodAutoscaler(autoscaler *autoscaling.HorizontalPodAutosc
104109
return allErrs
105110
}
106111

112+
// ValidateHorizontalPodAutoscalerUpdate validates an update to a HorizontalPodAutoscaler and returns an
113+
// ErrorList with any errors.
107114
func ValidateHorizontalPodAutoscalerUpdate(newAutoscaler, oldAutoscaler *autoscaling.HorizontalPodAutoscaler) field.ErrorList {
108115
allErrs := apivalidation.ValidateObjectMetaUpdate(&newAutoscaler.ObjectMeta, &oldAutoscaler.ObjectMeta, field.NewPath("metadata"))
109116

@@ -121,11 +128,13 @@ func ValidateHorizontalPodAutoscalerUpdate(newAutoscaler, oldAutoscaler *autosca
121128
return allErrs
122129
}
123130

131+
// ValidateHorizontalPodAutoscalerStatusUpdate validates an update to status on a HorizontalPodAutoscaler and
132+
// returns an ErrorList with any errors.
124133
func ValidateHorizontalPodAutoscalerStatusUpdate(newAutoscaler, oldAutoscaler *autoscaling.HorizontalPodAutoscaler) field.ErrorList {
125134
allErrs := apivalidation.ValidateObjectMetaUpdate(&newAutoscaler.ObjectMeta, &oldAutoscaler.ObjectMeta, field.NewPath("metadata"))
126135
status := newAutoscaler.Status
127136
allErrs = append(allErrs, apivalidation.ValidateNonnegativeField(int64(status.CurrentReplicas), field.NewPath("status", "currentReplicas"))...)
128-
allErrs = append(allErrs, apivalidation.ValidateNonnegativeField(int64(status.DesiredReplicas), field.NewPath("status", "desiredReplicasa"))...)
137+
allErrs = append(allErrs, apivalidation.ValidateNonnegativeField(int64(status.DesiredReplicas), field.NewPath("status", "desiredReplicas"))...)
129138
return allErrs
130139
}
131140

0 commit comments

Comments
 (0)