Skip to content

Commit 5adc63f

Browse files
authored
feat: require explicit targetValue in scaling metric specs (#1597)
1 parent 36bb1a3 commit 5adc63f

2 files changed

Lines changed: 6 additions & 8 deletions

File tree

config/crd/bases/http.keda.sh_interceptorroutes.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -159,11 +159,12 @@ spec:
159159
description: Scale based on concurrent request count.
160160
properties:
161161
targetValue:
162-
default: 100
163162
description: Target concurrent request count per replica.
164163
format: int32
165164
minimum: 1
166165
type: integer
166+
required:
167+
- targetValue
167168
type: object
168169
requestRate:
169170
description: Scale based on request rate.
@@ -173,7 +174,6 @@ spec:
173174
description: Bucket size for rate calculation within the window.
174175
type: string
175176
targetValue:
176-
default: 100
177177
description: Target request rate per replica.
178178
format: int32
179179
minimum: 1
@@ -183,6 +183,8 @@ spec:
183183
description: Sliding time window over which the request rate
184184
is calculated.
185185
type: string
186+
required:
187+
- targetValue
186188
type: object
187189
type: object
188190
x-kubernetes-validations:

operator/apis/http/v1beta1/interceptorroute_types.go

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -63,10 +63,8 @@ type RoutingRule struct {
6363
// ConcurrencyTargetSpec defines concurrency-based scaling.
6464
type ConcurrencyTargetSpec struct {
6565
// Target concurrent request count per replica.
66-
// +kubebuilder:default=100
6766
// +kubebuilder:validation:Minimum=1
68-
// +optional
69-
TargetValue int32 `json:"targetValue,omitzero"`
67+
TargetValue int32 `json:"targetValue"`
7068
}
7169

7270
// RequestRateTargetSpec defines rate-based scaling.
@@ -76,10 +74,8 @@ type RequestRateTargetSpec struct {
7674
// +optional
7775
Granularity metav1.Duration `json:"granularity,omitzero"`
7876
// Target request rate per replica.
79-
// +kubebuilder:default=100
8077
// +kubebuilder:validation:Minimum=1
81-
// +optional
82-
TargetValue int32 `json:"targetValue,omitzero"`
78+
TargetValue int32 `json:"targetValue"`
8379
// Sliding time window over which the request rate is calculated.
8480
// +kubebuilder:default="1m"
8581
// +optional

0 commit comments

Comments
 (0)