dont set replica when hpa is set

This is no longer needed after #2816 was merged Relates to #4983 (comment) Signed-off-by: Arko Dasgupta <[email protected]>
envoyproxy · Jan 7, 2025 · 8200af5 · 8200af5
1 parent 2a5ecaf
commit 8200af5
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 10 deletions.
diff --git a/internal/infrastructure/kubernetes/proxy/resource_provider.go b/internal/infrastructure/kubernetes/proxy/resource_provider.go
@@ -328,11 +328,6 @@ func (r *ResourceRender) Deployment() (*appsv1.Deployment, error) {
 
 	provider := proxyConfig.GetEnvoyProxyProvider()
 
-	// omit the deployment replicas if HPA is being set
-	if provider.GetEnvoyProxyKubeProvider().EnvoyHpa != nil {
-		deployment.Spec.Replicas = nil
-	}
-
 	// apply merge patch to deployment
 	if deployment, err = deploymentConfig.ApplyMergePatch(deployment); err != nil {
 		return nil, err

diff --git a/internal/infrastructure/kubernetes/ratelimit/resource_provider.go b/internal/infrastructure/kubernetes/ratelimit/resource_provider.go
@@ -290,11 +290,6 @@ func (r *ResourceRender) Deployment() (*appsv1.Deployment, error) {
 		}
 	}
 
-	// omit the deployment replicas if HPA is being set
-	if r.rateLimitHpa != nil {
-		deployment.Spec.Replicas = nil
-	}
-
 	// apply merge patch to deployment
 	var err error
 	if deployment, err = r.rateLimitDeployment.ApplyMergePatch(deployment); err != nil {