Skip to content

Commit 362d805

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 e30bab4 commit 362d805

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

cluster/manifests/skipper/deployment.yaml

Lines changed: 1 addition & 1 deletion
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)