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

Feature add environment name to resources #38

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
a61a21a
updated configmap name in all files
AngeloB-AIS Aug 30, 2024
fca06a2
feature: updated postgres name
AngeloB-AIS Sep 4, 2024
1594468
feature: updated server name, debugged changes
AngeloB-AIS Sep 4, 2024
905916a
feature: updated ui name and debugged
AngeloB-AIS Sep 4, 2024
8b68e75
feature: fixed worker name
AngeloB-AIS Sep 4, 2024
727fc2a
feature: fixed temporal naming
AngeloB-AIS Sep 4, 2024
8b36a70
revert namespace back to just multiwoven
AngeloB-AIS Sep 6, 2024
00ba000
change env name from suffix to prefix: configmap
AngeloB-AIS Sep 6, 2024
d345ef3
change env name from suffix to prefix: ingress
AngeloB-AIS Sep 6, 2024
8d69f2c
change env name from suffix to prefix: postgresql deployment/service
AngeloB-AIS Sep 6, 2024
85d2353
change env name from suffix to prefix: postgresql pvc
AngeloB-AIS Sep 6, 2024
01a8cc8
change env name from suffix to prefix: server
AngeloB-AIS Sep 6, 2024
bf0a40f
change env name from suffix to prefix: ui
AngeloB-AIS Sep 6, 2024
0b29add
change env name from suffix to prefix: worker
AngeloB-AIS Sep 6, 2024
9f21bcd
change env name from suffix to prefix: temporal
AngeloB-AIS Sep 6, 2024
17f661a
change env name from suffix to prefix: temporal in ingress template
AngeloB-AIS Sep 6, 2024
3f4406e
Merge branch 'main' into feature--add-environment-name-to-resources
AngeloB-AIS Sep 23, 2024
a9435fa
fix: added app env to workflow deployment names
AngeloB-AIS Sep 23, 2024
5b3b949
Merge branch 'main' into feature--add-environment-name-to-resources
AngeloB-AIS Sep 28, 2024
c31a8f2
fix: add appEnv prefix to hpa resources
AngeloB-AIS Sep 28, 2024
fe1dea6
fix: update chart version
AngeloB-AIS Sep 28, 2024
521736c
fix: add more descriptive default for pushapikey
AngeloB-AIS Sep 28, 2024
c7dbba1
fix: fix lint error where pushapikey description comment had too few …
AngeloB-AIS Sep 28, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/workflows/deploy-staging.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,6 @@ jobs:
- name: Deploy chart to staging
run: |
helm upgrade -i --reuse-values multiwoven ./charts/multiwoven
kubectl rollout restart deployment/multiwoven-worker -n multiwoven
kubectl rollout restart deployment/multiwoven-server -n multiwoven
kubectl rollout restart deployment/multiwoven-ui -n multiwoven
kubectl rollout restart deployment ${{ vars.APP_ENV }}-multiwoven-worker -n multiwoven
kubectl rollout restart deployment ${{ vars.APP_ENV }}-multiwoven-server -n multiwoven
kubectl rollout restart deployment ${{ vars.APP_ENV }}-multiwoven-ui -n multiwoven
4 changes: 2 additions & 2 deletions charts/multiwoven/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ apiVersion: v2
name: multiwoven
description: Open-source reverse ETL, an alternative to Hightouch, Census etc. 🔥
type: application
version: 0.27.0
appVersion: "0.27.0"
version: 0.28.0
appVersion: "0.28.0"
maintainers:
- name: subintp
- name: RafaelOAiSquared
Expand Down
4 changes: 2 additions & 2 deletions charts/multiwoven/templates/multiwoven-config.yaml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
apiVersion: v1
kind: ConfigMap
metadata:
name: {{ include "chart.fullname" . }}-config
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-config
namespace: {{ .Values.kubernetesNamespace }}
labels:
io.kompose.service: {{ include "chart.fullname" . }}-config
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-config
{{- include "chart.labels" . | nindent 4 }}
data:
ALLOWED_HOST: {{ .Values.multiwovenConfig.allowedHost | quote }}
Expand Down
8 changes: 4 additions & 4 deletions charts/multiwoven/templates/multiwoven-ingress.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
name: {{ include "chart.fullname" . }}-ingress
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ingress
namespace: {{ .Values.kubernetesNamespace }}

labels:
Expand Down Expand Up @@ -32,7 +32,7 @@ spec:
pathType: Prefix
backend:
service:
name: '{{ include "chart.fullname" . }}-ui'
name: '{{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui'
port:
number: {{ (index .Values.multiwovenUI.ports 0).port }}
- host: {{ .Values.multiwovenConfig.apiHost }}
Expand All @@ -42,7 +42,7 @@ spec:
pathType: Prefix
backend:
service:
name: '{{ include "chart.fullname" . }}-server'
name: '{{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server'
port:
number: {{ (index .Values.multiwovenServer.ports 0).port }}
{{ if .Values.temporal.enabled }}
Expand All @@ -51,7 +51,7 @@ spec:
paths:
- backend:
service:
name: '{{ include "chart.fullname" . }}-temporal-ui'
name: '{{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-temporal-ui'
port:
number: {{ (index .Values.temporalUi.ports 0).port }}
path: /(.*)
Expand Down
20 changes: 10 additions & 10 deletions charts/multiwoven/templates/multiwoven-postgresql-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,22 +2,22 @@
apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ include "chart.fullname" . }}-postgresql
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql
namespace: {{ .Values.kubernetesNamespace }}
labels:
io.kompose.service: {{ include "chart.fullname" . }}-postgresql
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql
{{- include "chart.labels" . | nindent 4 }}
spec:
replicas: {{ .Values.multiwovenPostgresql.replicas }}
selector:
matchLabels:
io.kompose.service: {{ include "chart.fullname" . }}-postgresql
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql
{{- include "chart.selectorLabels" . | nindent 6 }}
template:
metadata:
labels:
io.kompose.network/multiwoven-server-default: "true"
io.kompose.service: {{ include "chart.fullname" . }}-postgresql
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql
{{- include "chart.selectorLabels" . | nindent 8 }}
spec:
containers:
Expand All @@ -26,12 +26,12 @@ spec:
valueFrom:
configMapKeyRef:
key: DB_PASSWORD
name: {{ include "chart.fullname" . }}-config
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-config
- name: POSTGRES_USER
valueFrom:
configMapKeyRef:
key: DB_USERNAME
name: {{ include "chart.fullname" . }}-config
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-config
- name: KUBERNETES_CLUSTER_DOMAIN
value: {{ quote .Values.kubernetesClusterDomain }}
image: {{ .Values.multiwovenPostgresql.multiwovenPostgresql.image.repository }}:{{ .Values.multiwovenPostgresql.multiwovenPostgresql.image.tag | default .Chart.AppVersion }}
Expand All @@ -40,7 +40,7 @@ spec:
periodSeconds: 10
tcpSocket:
port: 5432
name: {{ include "chart.fullname" . }}-postgresql
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql
ports:
- containerPort: 5432
readinessProbe:
Expand All @@ -52,9 +52,9 @@ spec:
volumeMounts:
- mountPath: /var/lib/postgresql/data
subPath: postgres
name: {{ include "chart.fullname" . }}-postgresql-pvc
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql-pvc
volumes:
- name: {{ include "chart.fullname" . }}-postgresql-pvc
- name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql-pvc
persistentVolumeClaim:
claimName: {{ include "chart.fullname" . }}-postgresql-pvc
claimName: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql-pvc
{{ end }}
4 changes: 2 additions & 2 deletions charts/multiwoven/templates/multiwoven-postgresql-pvc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
name: {{ include "chart.fullname" . }}-postgresql-pvc
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql-pvc
namespace: {{ .Values.kubernetesNamespace }}
labels:
io.kompose.service: {{ include "chart.fullname" . }}-postgresql-pvc
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql-pvc
{{- include "chart.labels" . | nindent 4 }}
spec:
accessModes:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,15 @@
apiVersion: v1
kind: Service
metadata:
name: {{ include "chart.fullname" . }}-postgresql
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql
namespace: {{ .Values.kubernetesNamespace }}
labels:
io.kompose.service: {{ include "chart.fullname" . }}-postgresql
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql
{{- include "chart.labels" . | nindent 4 }}
spec:
type: {{ .Values.multiwovenPostgresql.type }}
selector:
io.kompose.service: {{ include "chart.fullname" . }}-postgresql
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-postgresql
{{- include "chart.selectorLabels" . | nindent 4 }}
ports:
{{- .Values.multiwovenPostgresql.ports | toYaml | nindent 2 -}}
Expand Down
18 changes: 9 additions & 9 deletions charts/multiwoven/templates/multiwoven-server-deployment.yaml
Original file line number Diff line number Diff line change
@@ -1,24 +1,24 @@
apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ include "chart.fullname" . }}-server
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
namespace: {{ .Values.kubernetesNamespace }}
labels:
app: {{ include "chart.fullname" . }}-server
io.kompose.service: {{ include "chart.fullname" . }}-server
app: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
{{- include "chart.labels" . | nindent 4 }}
spec:
replicas: {{ .Values.multiwovenServer.replicas }}
selector:
matchLabels:
app: {{ include "chart.fullname" . }}-server
io.kompose.service: {{ include "chart.fullname" . }}-server
app: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
{{- include "chart.selectorLabels" . | nindent 6 }}
template:
metadata:
labels:
app: {{ include "chart.fullname" . }}-server
io.kompose.service: {{ include "chart.fullname" . }}-server
app: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
{{- include "chart.selectorLabels" . | nindent 8 }}
spec:
containers:
Expand All @@ -27,15 +27,15 @@ spec:
value: {{ quote .Values.kubernetesClusterDomain }}
envFrom:
- configMapRef:
name: {{ include "chart.fullname" . }}-config
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-config
image: {{ .Values.multiwovenServer.multiwovenServer.image.repository }}:{{ .Values.multiwovenServer.multiwovenServer.image.tag | default .Chart.AppVersion }}
livenessProbe:
httpGet:
path: /
port: {{ (index .Values.multiwovenServer.ports 0).port }}
initialDelaySeconds: 15
periodSeconds: 10
name: {{ include "chart.fullname" . }}-server
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
ports:
- containerPort: {{ (index .Values.multiwovenServer.ports 0).port }}
readinessProbe:
Expand Down
4 changes: 2 additions & 2 deletions charts/multiwoven/templates/multiwoven-server-hpa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: autoscaling/v2
kind: HorizontalPodAutoscaler
metadata:
name: {{ include "chart.fullname" . }}-server-hpa
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server-hpa
namespace: {{ .Values.kubernetesNamespace }}
labels:
{{- include "chart.labels" . | nindent 4 }}
Expand All @@ -11,7 +11,7 @@ spec:
scaleTargetRef:
apiVersion: apps/v1
kind: Deployment
name: {{ include "chart.fullname" . }}-server
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
minReplicas: {{ .Values.hpa.multiwovenServer.minReplicas }}
maxReplicas: {{ .Values.hpa.multiwovenServer.maxReplicas }}
behavior:
Expand Down
6 changes: 3 additions & 3 deletions charts/multiwoven/templates/multiwoven-server-service.yaml
Original file line number Diff line number Diff line change
@@ -1,19 +1,19 @@
apiVersion: v1
kind: Service
metadata:
name: {{ include "chart.fullname" . }}-server
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
namespace: {{ .Values.kubernetesNamespace }}

labels:
io.kompose.service: {{ include "chart.fullname" . }}-server
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
{{- include "chart.labels" . | nindent 4 }}
annotations:
kompose.cmd: kompose convert
kompose.version: 1.32.0 (HEAD)
spec:
type: {{ .Values.multiwovenServer.type }}
selector:
io.kompose.service: {{ include "chart.fullname" . }}-server
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-server
{{- include "chart.selectorLabels" . | nindent 4 }}
ports:
{{- .Values.multiwovenServer.ports | toYaml | nindent 2 -}}
18 changes: 9 additions & 9 deletions charts/multiwoven/templates/multiwoven-ui-deployment.yaml
Original file line number Diff line number Diff line change
@@ -1,39 +1,39 @@
apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ include "chart.fullname" . }}-ui
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
namespace: {{ .Values.kubernetesNamespace }}
labels:
app: {{ include "chart.fullname" . }}-ui
io.kompose.service: {{ include "chart.fullname" . }}-ui
app: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
{{- include "chart.labels" . | nindent 4 }}
spec:
replicas: {{ .Values.multiwovenUI.replicas }}
selector:
matchLabels:
app: {{ include "chart.fullname" . }}-ui
io.kompose.service: {{ include "chart.fullname" . }}-ui
app: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
{{- include "chart.selectorLabels" . | nindent 6 }}
template:
metadata:
labels:
app: {{ include "chart.fullname" . }}-ui
io.kompose.service: {{ include "chart.fullname" . }}-ui
app: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
{{- include "chart.selectorLabels" . | nindent 8 }}
spec:
containers:
- env:
envFrom:
- configMapRef:
name: {{ include "chart.fullname" . }}-config
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-config
image: {{ .Values.multiwovenUI.multiwovenUI.image.repository }}:{{ .Values.multiwovenUI.multiwovenUI.image.tag | default .Chart.AppVersion }}
livenessProbe:
httpGet:
path: /
port: {{ (index .Values.multiwovenUI.ports 0).port }}
initialDelaySeconds: 15
periodSeconds: 10
name: {{ include "chart.fullname" . }}-ui
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
ports:
- containerPort: {{ (index .Values.multiwovenUI.ports 0).port }}
readinessProbe:
Expand Down
4 changes: 2 additions & 2 deletions charts/multiwoven/templates/multiwoven-ui-hpa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: autoscaling/v2
kind: HorizontalPodAutoscaler
metadata:
name: {{ include "chart.fullname" . }}-ui-hpa
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui-hpa
namespace: {{ .Values.kubernetesNamespace }}
labels:
{{- include "chart.labels" . | nindent 4 }}
Expand All @@ -11,7 +11,7 @@ spec:
scaleTargetRef:
apiVersion: apps/v1
kind: Deployment
name: {{ include "chart.fullname" . }}-ui
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
minReplicas: {{ .Values.hpa.multiwovenUI.minReplicas }}
maxReplicas: {{ .Values.hpa.multiwovenUI.maxReplicas }}
behavior:
Expand Down
6 changes: 3 additions & 3 deletions charts/multiwoven/templates/multiwoven-ui-service.yaml
Original file line number Diff line number Diff line change
@@ -1,19 +1,19 @@
apiVersion: v1
kind: Service
metadata:
name: {{ include "chart.fullname" . }}-ui
name: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
namespace: {{ .Values.kubernetesNamespace }}

labels:
io.kompose.service: {{ include "chart.fullname" . }}-ui
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
{{- include "chart.labels" . | nindent 4 }}
annotations:
kompose.cmd: kompose convert
kompose.version: 1.32.0 (HEAD)
spec:
type: {{ .Values.multiwovenUI.type }}
selector:
io.kompose.service: {{ include "chart.fullname" . }}-ui
io.kompose.service: {{ .Values.multiwovenConfig.appEnv }}-{{ include "chart.fullname" . }}-ui
{{- include "chart.selectorLabels" . | nindent 4 }}
ports:
{{- .Values.multiwovenUI.ports | toYaml | nindent 2 -}}
Loading
Loading