diff --git a/templates/server-service.yaml b/templates/server-service.yaml index a9c5ede31..216047050 100644 --- a/templates/server-service.yaml +++ b/templates/server-service.yaml @@ -11,10 +11,6 @@ metadata: app.kubernetes.io/instance: {{ .Release.Name }} app.kubernetes.io/managed-by: {{ .Release.Service }} annotations: - # This must be set in addition to publishNotReadyAddresses due - # to an open issue where it may not work: - # https://github.com/kubernetes/kubernetes/issues/58662 - service.alpha.kubernetes.io/tolerate-unready-endpoints: "true" {{- if .Values.server.service.annotations }} {{ toYaml .Values.server.service.annotations | indent 4 }} {{- end }} diff --git a/test/unit/server-service.bats b/test/unit/server-service.bats index c276c4386..6124c3553 100755 --- a/test/unit/server-service.bats +++ b/test/unit/server-service.bats @@ -114,32 +114,6 @@ load _helpers [ "${actual}" = "false" ] } -# This can be seen as testing just what we put into the YAML raw, but -# this is such an important part of making everything work we verify it here. -@test "server/Service: tolerates unready endpoints" { - cd `chart_dir` - local actual=$(helm template \ - -x templates/server-service.yaml \ - --set 'server.dev.enabled=true' \ - . | tee /dev/stderr | - yq -r '.metadata.annotations["service.alpha.kubernetes.io/tolerate-unready-endpoints"]' | tee /dev/stderr) - [ "${actual}" = "true" ] - - local actual=$(helm template \ - -x templates/server-service.yaml \ - --set 'server.ha.enabled=true' \ - . | tee /dev/stderr | - yq -r '.metadata.annotations["service.alpha.kubernetes.io/tolerate-unready-endpoints"]' | tee /dev/stderr) - [ "${actual}" = "true" ] - - local actual=$(helm template \ - -x templates/server-service.yaml \ - --set 'server.standalone.enabled=true' \ - . | tee /dev/stderr | - yq -r '.metadata.annotations["service.alpha.kubernetes.io/tolerate-unready-endpoints"]' | tee /dev/stderr) - [ "${actual}" = "true" ] -} - @test "server/Service: generic annotations" { cd `chart_dir` local actual=$(helm template \