Skip to content

Fix service mismatch

Dmitry Gruzd requested to merge fix-service-mismatch into main

What does this MR do and why?

This MR attempts to fix GitLab.com Chart Bump failing in staging due to... (gitlab-com/gl-infra/delivery#19793 - closed) by keeping the original loadBalancerIP for the stateful set service and by introducing separate configuration for the external service.

How to test

  1. git checkout v0.7.0
  2. helm install gitlab-zoekt . --set service.type=LoadBalancer --set service.loadBalancerIP=$INTERNAL_IP
  3. git checkout v0.8.0
  4. helm upgrade gitlab-zoekt . --set service.type=LoadBalancer --set service.loadBalancerIP=$INTERNAL_IP
  5. Verify that it fails with Service "gitlab-zoekt" is invalid: spec.clusterIPs[0]: Invalid value: []string{"None"}: may not change once set as in GitLab.com Chart Bump failing in staging due to... (gitlab-com/gl-infra/delivery#19793 - closed)
  6. git checkout fix-service-mismatch
  7. helm upgrade gitlab-zoekt . --set service.type=LoadBalancer --set service.loadBalancerIP=$INTERNAL_IP
Edited by Dmitry Gruzd

Merge request reports