Skip to content

Commit

Permalink
istio shared cm per version and update svc cluster rollout order
Browse files Browse the repository at this point in the history
the istio shared config map must exist for the targeted version prior to
upgrading the service
  • Loading branch information
tony-schndr committed Feb 28, 2025
1 parent 633d69e commit 17ca35b
Show file tree
Hide file tree
Showing 5 changed files with 20 additions and 13 deletions.
20 changes: 10 additions & 10 deletions dev-infrastructure/svc-pipeline.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,16 @@ resourceGroups:
subscription: {{ .svc.subscription }}
aksCluster: {{ .svc.aks.name }}
steps:
# configure istio
- name: istio-config
action: Shell
command: make -C ../istio deploy
dryRun:
variables:
- name: DRY_RUN
value: "true"
dependsOn:
- istio-upgrade
# - updates workload to use istio on version svc.istio.targetVersion
# - configures istio IP tag usage
- name: istio-upgrade
Expand All @@ -120,16 +130,6 @@ resourceGroups:
configRef: svc.rg
dependsOn:
- svc
# configure istio
- name: istio-config
action: Shell
command: make -C ../istio deploy
dryRun:
variables:
- name: DRY_RUN
value: "true"
dependsOn:
- istio-upgrade
# Install ACRpull
- name: acrpull
action: Shell
Expand Down
1 change: 1 addition & 0 deletions istio/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ HELM_CMD ?= helm upgrade --install
deploy:
${HELM_CMD} istio \
deploy/helm/istio/ \
--set "istioVersions={${ISTIO_VERSIONS}}" \
--namespace aks-istio-system
.PHONY: deploy

Expand Down
7 changes: 5 additions & 2 deletions istio/deploy/helm/istio/templates/istio-shared-configmap.yml
Original file line number Diff line number Diff line change
@@ -1,9 +1,11 @@
{{ range $version := .Values.istioVersions }}
---
kind: ConfigMap
apiVersion: v1
metadata:
labels:
istio.io/rev: asm-1-23
name: istio-shared-configmap-asm-1-23
istio.io/rev: {{ $version }}
name: istio-shared-configmap-{{ $version }}
namespace: aks-istio-system
data:
mesh: |-
Expand All @@ -14,3 +16,4 @@ data:
port: "8080"
includeRequestHeadersInCheck: ["x-ext-authz"]
pathPrefix: "/v1/EnvoyValidateRequest"
{{ end }}
2 changes: 1 addition & 1 deletion istio/deploy/helm/istio/values.yaml
Original file line number Diff line number Diff line change
@@ -1 +1 @@
# This file is intentionally left blank.
istioVersions: []
3 changes: 3 additions & 0 deletions istio/pipeline.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,3 +13,6 @@ resourceGroups:
variables:
- name: DRY_RUN
value: "true"
variables:
- name: ISTIO_VERSIONS
configRef: svc.istio.versions

0 comments on commit 17ca35b

Please sign in to comment.