Skip to content

Commit

Permalink
operator: Gateway pods do not need to have INSTANCE_AVAILABILITY_ZONE…
Browse files Browse the repository at this point in the history
… env set in their containers (#9950)
  • Loading branch information
shwetaap authored Jul 17, 2023
1 parent 15af77b commit 97d34ea
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions operator/internal/manifests/node_placement.go
Original file line number Diff line number Diff line change
Expand Up @@ -63,11 +63,13 @@ func configureReplication(podTemplate *corev1.PodTemplateSpec, replication *loki
Env: []corev1.EnvVar{availabilityZoneEnvVar},
}

for i, dst := range podTemplate.Spec.Containers {
if err := mergo.Merge(&dst, src, mergo.WithAppendSlice); err != nil {
return err
if component != LabelGatewayComponent {
for i, dst := range podTemplate.Spec.Containers {
if err := mergo.Merge(&dst, src, mergo.WithAppendSlice); err != nil {
return err
}
podTemplate.Spec.Containers[i] = dst
}
podTemplate.Spec.Containers[i] = dst
}

if err := mergo.Merge(podTemplate, template); err != nil {
Expand Down

0 comments on commit 97d34ea

Please sign in to comment.