Skip to content

Commit 23bf22f

Browse files
committed
chore: remove webhooks2tasks and extend webhook-handler secrets, add needed secrets to api-sidecar
1 parent bd5cbb1 commit 23bf22f

File tree

10 files changed

+37
-244
lines changed

10 files changed

+37
-244
lines changed

Makefile

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -593,7 +593,6 @@ endif
593593
--set logs2notifications.logs2webhooks.disabled=$(LOGS2WEBHOOKS_DISABLED) \
594594
$$([ $(IMAGE_REGISTRY) ] && [ $(INSTALL_STABLE_CORE) != true ] && echo '--set ssh.image.repository=$(IMAGE_REGISTRY)/ssh') \
595595
$$([ $(IMAGE_REGISTRY) ] && [ $(INSTALL_STABLE_CORE) != true ] && echo '--set webhookHandler.image.repository=$(IMAGE_REGISTRY)/webhook-handler') \
596-
$$([ $(IMAGE_REGISTRY) ] && [ $(INSTALL_STABLE_CORE) != true ] && echo '--set webhooks2tasks.image.repository=$(IMAGE_REGISTRY)/webhooks2tasks') \
597596
--set s3BAASAccessKeyID=lagoonFilesAccessKey \
598597
--set s3BAASSecretAccessKey=lagoonFilesSecretKey \
599598
--set s3FilesAccessKeyID=lagoonFilesAccessKey \

charts/lagoon-core/README.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,6 @@ logs2webhook:
2525
enabled: false
2626
webhookHandler:
2727
enabled: false
28-
webhooks2tasks:
29-
enabled: false
3028
sshPortal:
3129
enabled: false
3230
```

charts/lagoon-core/ci/linter-values.yaml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -113,14 +113,6 @@ authServer:
113113
requests:
114114
cpu: "10m"
115115

116-
webhooks2tasks:
117-
replicaCount: 1
118-
image:
119-
repository: uselagoon/webhooks2tasks
120-
resources:
121-
requests:
122-
cpu: "10m"
123-
124116
webhookHandler:
125117
replicaCount: 1
126118
image:

charts/lagoon-core/ci/testlagoon-main-override.yaml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,6 @@ authServer:
3636
image:
3737
repository: testlagoon/auth-server
3838

39-
webhooks2tasks:
40-
image:
41-
repository: testlagoon/webhooks2tasks
42-
4339
webhookHandler:
4440
image:
4541
repository: testlagoon/webhook-handler

charts/lagoon-core/templates/_helpers.tpl

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -284,37 +284,6 @@ app.kubernetes.io/instance: {{ .Release.Name }}
284284
{{- end }}
285285

286286

287-
288-
{{/*
289-
Create a default fully qualified app name for webhooks2tasks.
290-
*/}}
291-
{{- define "lagoon-core.webhooks2tasks.fullname" -}}
292-
{{- include "lagoon-core.fullname" . }}-webhooks2tasks
293-
{{- end }}
294-
295-
{{/*
296-
Common labels webhooks2tasks
297-
*/}}
298-
{{- define "lagoon-core.webhooks2tasks.labels" -}}
299-
helm.sh/chart: {{ include "lagoon-core.chart" . }}
300-
{{ include "lagoon-core.webhooks2tasks.selectorLabels" . }}
301-
{{- if .Chart.AppVersion }}
302-
app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
303-
{{- end }}
304-
app.kubernetes.io/managed-by: {{ .Release.Service }}
305-
{{- end }}
306-
307-
{{/*
308-
Selector labels webhooks2tasks
309-
*/}}
310-
{{- define "lagoon-core.webhooks2tasks.selectorLabels" -}}
311-
app.kubernetes.io/name: {{ include "lagoon-core.name" . }}
312-
app.kubernetes.io/component: {{ include "lagoon-core.webhooks2tasks.fullname" . }}
313-
app.kubernetes.io/instance: {{ .Release.Name }}
314-
{{- end }}
315-
316-
317-
318287
{{/*
319288
Create a default fully qualified app name for webhookHandler.
320289
*/}}

charts/lagoon-core/templates/api.deployment.yaml

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,33 @@ spec:
8484
{{- toYaml .Values.api.securityContext | nindent 10 }}
8585
image: "{{ .Values.apiSidecarHandler.image.repository }}:{{ coalesce .Values.apiSidecarHandler.image.tag .Values.imageTag .Chart.AppVersion }}"
8686
imagePullPolicy: {{ .Values.apiSidecarHandler.image.pullPolicy }}
87+
env:
88+
- name: JWTSECRET
89+
valueFrom:
90+
secretKeyRef:
91+
name: {{ include "lagoon-core.fullname" . }}-secrets
92+
key: JWTSECRET
93+
- name: LAGOON_VERSION
94+
value: {{ .Chart.AppVersion | replace "-" "." }}
95+
- name: RABBITMQ_HOST
96+
value: {{ include "lagoon-core.broker.fullname" . }}
97+
- name: RABBITMQ_PASSWORD
98+
valueFrom:
99+
secretKeyRef:
100+
name: {{ include "lagoon-core.broker.fullname" . }}
101+
key: RABBITMQ_PASSWORD
102+
- name: RABBITMQ_USERNAME
103+
valueFrom:
104+
secretKeyRef:
105+
name: {{ include "lagoon-core.broker.fullname" . }}
106+
key: RABBITMQ_USERNAME
107+
{{- range $key, $val := .Values.api.sidecar.additionalEnvs }}
108+
- name: {{ $key }}
109+
value: {{ $val | quote }}
110+
{{- end }}
111+
envFrom:
112+
- secretRef:
113+
name: {{ include "lagoon-core.api.fullname" . }}
87114
livenessProbe:
88115
httpGet:
89116
path: /status

charts/lagoon-core/templates/webhook-handler.deployment.yaml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ spec:
3535
image: "{{ .Values.webhookHandler.image.repository }}:{{ coalesce .Values.webhookHandler.image.tag .Values.imageTag .Chart.AppVersion }}"
3636
imagePullPolicy: {{ .Values.webhookHandler.image.pullPolicy }}
3737
env:
38-
- name: RABBITMQ_HOST
38+
- name: RABBITMQ_ADDRESS
3939
value: {{ include "lagoon-core.broker.fullname" . }}
4040
- name: RABBITMQ_PASSWORD
4141
valueFrom:
@@ -47,6 +47,13 @@ spec:
4747
secretKeyRef:
4848
name: {{ include "lagoon-core.broker.fullname" . }}
4949
key: RABBITMQ_USERNAME
50+
- name: JWT_SECRET
51+
valueFrom:
52+
secretKeyRef:
53+
name: {{ include "lagoon-core.fullname" . }}-secrets
54+
key: JWTSECRET
55+
- name: GRAPHQL_ENDPOINT
56+
value: http://{{ include "lagoon-core.api.fullname" . }}:{{ .Values.api.service.port }}/graphql
5057
{{- range $key, $val := .Values.webhookHandler.additionalEnvs }}
5158
- name: {{ $key }}
5259
value: {{ $val | quote }}

charts/lagoon-core/templates/webhooks2tasks.deployment.yaml

Lines changed: 0 additions & 137 deletions
This file was deleted.

charts/lagoon-core/templates/webhooks2tasks.hpa.yaml

Lines changed: 0 additions & 32 deletions
This file was deleted.

charts/lagoon-core/values.yaml

Lines changed: 2 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -155,6 +155,8 @@ api:
155155

156156
additionalEnvs:
157157
# FOO: Bar
158+
sidecar:
159+
additionalEnvs:
158160

159161
autoscaling:
160162
enabled: false
@@ -494,34 +496,6 @@ authServer:
494496
targetCPUUtilizationPercentage: 80
495497
# targetMemoryUtilizationPercentage: 80
496498

497-
webhooks2tasks:
498-
enabled: true
499-
replicaCount: 2
500-
image:
501-
repository: uselagoon/webhooks2tasks
502-
pullPolicy: Always
503-
# Overrides the image tag whose default is the chart appVersion.
504-
tag: ""
505-
506-
podAnnotations: {}
507-
508-
securityContext: {}
509-
510-
resources:
511-
requests:
512-
memory: "64Mi"
513-
cpu: "50m"
514-
515-
additionalEnvs:
516-
# FOO: Bar
517-
518-
autoscaling:
519-
enabled: false
520-
minReplicas: 1
521-
maxReplicas: 100
522-
targetCPUUtilizationPercentage: 80
523-
# targetMemoryUtilizationPercentage: 80
524-
525499
webhookHandler:
526500
enabled: true
527501
replicaCount: 2

0 commit comments

Comments
 (0)