Skip to content

Commit

Permalink
fix: missing sync wave for external
Browse files Browse the repository at this point in the history
  • Loading branch information
kirinnee committed Oct 20, 2023
1 parent d585768 commit 2891c4f
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 6 deletions.
4 changes: 2 additions & 2 deletions infra/root_chart/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ Root Chart to a single Service
| Key | Type | Default | Description |
|-----|------|---------|-------------|
| api.affinity | object | `{}` | |
| api.annotations."argocd.argoproj.io/sync-wave" | string | `"3"` | |
| api.annotations."argocd.argoproj.io/sync-wave" | string | `"4"` | |
| api.appSettings.App.Mode | string | `"Server"` | |
| api.autoscaling | object | `{}` | |
| api.configMountPath | string | `"/app/Config"` | |
Expand Down Expand Up @@ -76,7 +76,7 @@ Root Chart to a single Service
| maindb.nameOverride | string | `"main-database"` | |
| maindb.primary.persistence.enabled | bool | `false` | |
| migration.affinity | object | `{}` | |
| migration.annotations."argocd.argoproj.io/sync-wave" | string | `"2"` | |
| migration.annotations."argocd.argoproj.io/sync-wave" | string | `"3"` | |
| migration.appSettings.App.Mode | string | `"Migration"` | |
| migration.aspNetEnv | string | `"Development"` | |
| migration.backoffLimit | int | `4` | |
Expand Down
3 changes: 1 addition & 2 deletions infra/root_chart/templates/externalsecret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@ metadata:
name: {{ template "root-chart.fullname-with-suffix" (dict "arg" .Values.bromine.target "root" .) }}
labels: {{- include "root-chart.labels" . | nindent 4 }}
annotations: {{- include "root-chart.annotations" . | nindent 4 }}
"helm.sh/hook": pre-install,pre-upgrade
"helm.sh/hook-weight": "-2"
argocd.argoproj.io/sync-wave: "2"
spec:
secretStoreRef:
kind: SecretStore
Expand Down
4 changes: 2 additions & 2 deletions infra/root_chart/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ bromine:
api:
enabled: true
annotations:
argocd.argoproj.io/sync-wave: "3"
argocd.argoproj.io/sync-wave: "4"
nameOverride: "api"
fullnameOverride: ""

Expand Down Expand Up @@ -103,7 +103,7 @@ api:

migration:
annotations:
argocd.argoproj.io/sync-wave: "2"
argocd.argoproj.io/sync-wave: "3"
enabled: false
nameOverride: "migration"
fullnameOverride: ""
Expand Down

0 comments on commit 2891c4f

Please sign in to comment.