Skip to content

Commit

Permalink
Merge pull request #6592 from zalando-incubator/skipper/canary
Browse files Browse the repository at this point in the history
skipper: enable manual canary deployment
  • Loading branch information
AlexanderYastrebov authored Dec 6, 2023
2 parents 2d2391b + 9ef924c commit 6e9eea7
Show file tree
Hide file tree
Showing 4 changed files with 199 additions and 143 deletions.
8 changes: 8 additions & 0 deletions cluster/config-defaults.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,14 @@ skipper_ingress_min_replicas: "2"
{{end}}
skipper_ingress_cpu: "1000m"
skipper_ingress_memory: "1500Mi"

# Enables deployment of canary version
skipper_ingress_canary_enabled: "false"

# Optional canary arguments separated by "[cf724afc]" to allow whitespaces, e.g.:
# skipper_ingress_canary_args: "-foo=has a whitespace[cf724afc]-baz=qux"
skipper_ingress_canary_args: ""

# When set to true (and dedicated node pool for skipper is also true) the
# daemonset overhead will be subtracted from the cpu settings such
# that skipper will perfectly fit on the node.
Expand Down
6 changes: 6 additions & 0 deletions cluster/manifests/deletions.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,12 @@ post_apply:
kind: ClusterRoleBinding
{{- end }}

{{ if ne .Cluster.ConfigItems.skipper_ingress_canary_enabled "true" }}
- name: skipper-ingress-canary
namespace: kube-system
kind: Deployment
{{ end }}

{{ if eq .Cluster.ConfigItems.skipper_routesrv_enabled "false" }}
- name: skipper-ingress-routesrv
namespace: kube-system
Expand Down
Loading

0 comments on commit 6e9eea7

Please sign in to comment.