Skip to content

Commit 2c8324b

Browse files
committed
fix: Remove changes from routesrv
We should have k8s healthchecks in routesrv to pass e2e load tests. Tests are started with the tip of the dev branch and later the new configuration is applied. There is a timespan when old ingress pods consume routes from the new routesev. If healthcheck is false they will have no healthz endpoint for some time. Signed-off-by: Aleksandr Ponimaskin <[email protected]>
1 parent 5dd6bac commit 2c8324b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

cluster/manifests/skipper/deployment.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -562,7 +562,6 @@ spec:
562562
- "-enable-profile"
563563
- "-kubernetes"
564564
- "-kubernetes-in-cluster"
565-
- "-kubernetes-healthcheck=false" # see -inline-routes of skipper pods
566565
- "-kubernetes-path-mode=path-prefix"
567566
- "-kubernetes-backend-traffic-algorithm={{ .Cluster.ConfigItems.skipper_ingress_backend_traffic_algorithm }}"
568567
- "-kubernetes-default-lb-algorithm={{ .Cluster.ConfigItems.skipper_ingress_default_lb_algorithm }}"
@@ -579,6 +578,7 @@ spec:
579578
- '-kubernetes-annotation-filters-append={{ .Cluster.ConfigItems.skipper_kubernetes_annotation_filters_append }}'
580579
- '-kubernetes-east-west-range-annotation-predicates={{ .Cluster.ConfigItems.skipper_kubernetes_east_west_range_annotation_predicates }}'
581580
- '-kubernetes-east-west-range-annotation-filters-append={{ .Cluster.ConfigItems.skipper_kubernetes_east_west_range_annotation_filters_append }}'
581+
- "-reverse-source-predicate"
582582
- "-default-filters-dir=/etc/config/default-filters"
583583
- '-default-filters-prepend={{ .Cluster.ConfigItems.skipper_default_filters }}'
584584
- '-default-filters-append={{ .Cluster.ConfigItems.skipper_default_filters_authentication }}'

0 commit comments

Comments
 (0)