Skip to content

Commit

Permalink
fix(siren): service selector (#61)
Browse files Browse the repository at this point in the history
  • Loading branch information
mabdh authored Oct 24, 2023
1 parent 1f411e6 commit 79255c9
Show file tree
Hide file tree
Showing 3 changed files with 43 additions and 39 deletions.
34 changes: 17 additions & 17 deletions stable/siren/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,22 +1,22 @@
apiVersion: v2
name: siren
description: Siren Helm chart
version: 0.1.8
appVersion: v0.6.8
version: 0.1.9
appVersion: v0.6.9
home: https://github.com/goto/siren
dependencies:
- name: app
version: "0.5.10"
repository: "https://goto.github.io/charts/"
alias: app
condition: app.enabled
- name: app
version: "0.5.10"
repository: "https://goto.github.io/charts/"
alias: notification-worker
condition: notification-worker.enabled
- name: app
version: "0.5.10"
repository: "https://goto.github.io/charts/"
alias: notification-dlq-worker
condition: notification-dlq-worker.enabled
- name: app
version: "0.5.10"
repository: "https://goto.github.io/charts/"
alias: app
condition: app.enabled
- name: app
version: "0.5.10"
repository: "https://goto.github.io/charts/"
alias: notification-worker
condition: notification-worker.enabled
- name: app
version: "0.5.10"
repository: "https://goto.github.io/charts/"
alias: notification-dlq-worker
condition: notification-dlq-worker.enabled
4 changes: 3 additions & 1 deletion stable/siren/templates/services.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@ spec:
name: {{ $port.name }}
{{- end }}
selector:
{{- include "app.selectorLabels" $dot | nindent 4 }}
{{- range $k, $v := $service.selectorLabels }}
{{ $k }}: {{ $v }}
{{- end }}
{{- end -}}
{{- end -}}
44 changes: 23 additions & 21 deletions stable/siren/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,15 +26,18 @@ app:
- mountPath: /work-dir
name: workdir
initContainers:
- args: ["wget -c https://github.com/goto/siren/releases/download/v0.6.7/siren-cortex_Linux_x86_64.tar.gz -O - | sudo tar -xz -C /work-dir/"]
- args:
[
"wget -c https://github.com/goto/siren/releases/download/v0.6.9/siren-cortex_Linux_x86_64.tar.gz -O - | sudo tar -xz -C /work-dir/",
]
command: ["/bin/sh", "-c"]
image: busybox:latest
imagePullPolicy: IfNotPresent
name: init-siren-app
name: init-cortex
volumeMounts:
- mountPath: /work-dir
name: workdir
- mountPath: /work-dir
name: workdir

podLabels: {}

migration:
Expand Down Expand Up @@ -81,7 +84,7 @@ app:
NOTIFICATION_MESSAGE_HANDLER_ENABLED: false
NOTIFICATION_DLQ_HANDLER_ENABLED: false
# PROVIDERS_CORTEX_WEBHOOK_BASE_API: http://localhost:8080/v1beta1/alerts/cortex

secretConfig:
DB_URL: postgres://postgres:@localhost:5432/siren?sslmode=disable
SERVICE_ENCRYPTION_KEY: ____STRING_OF_32_CHARACTERS_____
Expand All @@ -90,7 +93,6 @@ app:
- name: workdir
emptyDir: {}


notification-worker:
image:
repository: gotocompany/siren
Expand All @@ -104,17 +106,17 @@ notification-worker:
livenessProbe:
exec:
command:
- /bin/sh
- -c
- ps -ef | grep siren | grep -v grep
- /bin/sh
- -c
- ps -ef | grep siren | grep -v grep
initialDelaySeconds: 10
periodSeconds: 10
readinessProbe:
exec:
command:
- /bin/sh
- -c
- ps -ef | grep siren | grep -v grep
- /bin/sh
- -c
- ps -ef | grep siren | grep -v grep
initialDelaySeconds: 10
periodSeconds: 10
podLabels: {}
Expand All @@ -130,7 +132,7 @@ notification-worker:
NOTIFICATION_QUEUE_KIND: postgres
NOTIFICATION_MESSAGE_HANDLER_ENABLED: false
NOTIFICATION_DLQ_HANDLER_ENABLED: false

secretConfig:
DB_URL: postgres://postgres:@localhost:5432/siren?sslmode=disable
SERVICE_ENCRYPTION_KEY: ____STRING_OF_32_CHARACTERS_____
Expand All @@ -148,17 +150,17 @@ notification-dlq-worker:
livenessProbe:
exec:
command:
- /bin/sh
- -c
- ps -ef | grep siren | grep -v grep
- /bin/sh
- -c
- ps -ef | grep siren | grep -v grep
initialDelaySeconds: 10
periodSeconds: 10
readinessProbe:
exec:
command:
- /bin/sh
- -c
- ps -ef | grep siren | grep -v grep
- /bin/sh
- -c
- ps -ef | grep siren | grep -v grep
initialDelaySeconds: 10
periodSeconds: 10
podLabels: {}
Expand All @@ -168,7 +170,7 @@ notification-dlq-worker:

ingress:
enabled: false

config:
DB_DRIVER: postgres
NOTIFICATION_QUEUE_KIND: postgres
Expand Down

0 comments on commit 79255c9

Please sign in to comment.