Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Helm Migration for Staging no TGB (DO NOT MERGE) #3189

Closed
wants to merge 9 commits into from
2 changes: 1 addition & 1 deletion helmfile/charts/notify-celery/templates/hpa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ spec:
scaleTargetRef:
apiVersion: apps/v1
kind: Deployment
name: celery-{{ $node.name }}
name: notify-celery-{{ $node.name }}
{{- end }}
---
{{- end }}
Expand Down
22 changes: 22 additions & 0 deletions helmfile/charts/notify-celery/templates/service.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
{{- range $node := .Values.nodes }}
{{- with $ -}}
{{ if $node.service.serviceEnabled }}
apiVersion: v1
kind: Service
metadata:
name: {{ include "celery.fullname" . }}-{{ $node.name }}
labels:
app: {{ include "celery.fullname" . }}-{{ $node.name }}
namespace: notification-canada-ca
spec:
type: ClusterIP
# Putting a dummy port because we need to specify ports
ports:
- port: 80
targetPort: 80
selector:
app: notify-celery-{{ $node.name }}
{{- end }}
{{- end }}
---
{{- end }}
18 changes: 17 additions & 1 deletion helmfile/charts/notify-celery/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,8 @@ nodes:
pdbEnabled: false
autoscaling:
hpaEnabled: false
service:
serviceEnabled: true
other-beat:
name: beat
newRelicAppName: "notification-celery-beat-dev"
Expand All @@ -72,6 +74,8 @@ nodes:
pdbEnabled: false
autoscaling:
hpaEnabled: false
service:
serviceEnabled: true
# PRIMARY CONFIGS
primary-main:
name: primary
Expand All @@ -96,6 +100,8 @@ nodes:
minAvailable: 2
autoscaling:
hpaEnabled: false
service:
serviceEnabled: true
primary-email:
name: email-send-primary
newRelicAppName: "notification-celery-email-send-primary-dev"
Expand All @@ -119,6 +125,8 @@ nodes:
minAvailable: 2
autoscaling:
hpaEnabled: false
service:
serviceEnabled: true
primary-sms:
name: sms-send-primary
newRelicAppName: "notification-celery-sms-send-primary-dev"
Expand All @@ -142,6 +150,8 @@ nodes:
minAvailable: 2
autoscaling:
hpaEnabled: false
service:
serviceEnabled: true
# SCALABLE CONFIGS
scalable-main:
name: scalable
Expand Down Expand Up @@ -171,6 +181,8 @@ nodes:
pdb:
pdbEnabled: true
minAvailable: 2
service:
serviceEnabled: true
scalable-email:
name: email-send-scalable
newRelicAppName: "notification-celery-email-send-scalable-dev"
Expand Down Expand Up @@ -199,6 +211,8 @@ nodes:
pdb:
pdbEnabled: true
minAvailable: 2
service:
serviceEnabled: true
scalable-sms:
name: sms-send-scalable
newRelicAppName: "notification-celery-sms-send-scalable-dev"
Expand Down Expand Up @@ -227,7 +241,9 @@ nodes:
pdb:
pdbEnabled: true
minAvailable: 2

service:
serviceEnabled: true

image:
repository: public.ecr.aws/cds-snc/notify-api
pullPolicy: Always
Expand Down
2 changes: 1 addition & 1 deletion helmfile/helmfile.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ repositories:
url: https://okgolove.github.io/helm-charts

releases:
{{ if eq .Environment.Name "dev" }}
{{ if or (eq .Environment.Name "dev") (eq .Environment.Name "staging") }}
- name: notify-documentation
namespace: notification-canada-ca
labels:
Expand Down
Loading