chore: merge branch 'main' into build-warnings

uselagoon · Nov 24, 2023 · baed884 · baed884
2 parents bf22431 + 3d23c2e
commit baed884
Show file tree

Hide file tree

Showing 2 changed files with 78 additions and 23 deletions.
diff --git a/internal/templating/backups/template_prebackuppod.go b/internal/templating/backups/template_prebackuppod.go
@@ -19,6 +19,11 @@ import (
 	"sigs.k8s.io/yaml"
 )
 
+type PreBackupPodTmpl struct {
+	Service   generator.ServiceValues
+	Namespace string
+}
+
 func GeneratePreBackupPod(
 	lValues generator.BuildValues,
 ) ([]byte, error) {
@@ -77,7 +82,11 @@ func GeneratePreBackupPod(
 
 				var pbp bytes.Buffer
 				tmpl, _ := template.New("").Funcs(funcMap).Parse(preBackupPodSpecs[serviceValues.Type])
-				err := tmpl.Execute(&pbp, serviceValues)
+				tmplVals := PreBackupPodTmpl{
+					Service:   serviceValues,
+					Namespace: lValues.Namespace,
+				}
+				err := tmpl.Execute(&pbp, tmplVals)
 				if err != nil {
 					return nil, err
 				}
@@ -163,7 +172,11 @@ func GeneratePreBackupPod(
 
 				var pbp bytes.Buffer
 				tmpl, _ := template.New("").Funcs(funcMap).Parse(preBackupPodSpecs[serviceValues.Type])
-				err := tmpl.Execute(&pbp, serviceValues)
+				tmplVals := PreBackupPodTmpl{
+					Service:   serviceValues,
+					Namespace: lValues.Namespace,
+				}
+				err := tmpl.Execute(&pbp, tmplVals)
 				if err != nil {
 					return nil, err
 				}
@@ -285,7 +298,7 @@ var preBackupPodSpecs = PreBackupPods{
   $BACKUP_DB_DATABASE
   >> $dump
   && cat $dump && rm $dump"
-fileExtension: .{{ .Name }}.sql
+fileExtension: .{{ .Service.Name }}.sql
 pod:
   spec:
     containers:
@@ -296,26 +309,26 @@ pod:
       - name: BACKUP_DB_HOST
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_HOST
+            key: {{ .Service.Name | VarFix }}_HOST
             name: lagoon-env
       - name: BACKUP_DB_USERNAME
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_USERNAME
+            key: {{ .Service.Name | VarFix }}_USERNAME
             name: lagoon-env
       - name: BACKUP_DB_PASSWORD
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_PASSWORD
+            key: {{ .Service.Name | VarFix }}_PASSWORD
             name: lagoon-env
       - name: BACKUP_DB_DATABASE
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_DATABASE
+            key: {{ .Service.Name | VarFix }}_DATABASE
             name: lagoon-env
       image: uselagoon/database-tools:latest
       imagePullPolicy: Always
-      name: {{ .Name }}-prebackuppod`,
+      name: {{ .Service.Name }}-prebackuppod`,
 	"postgres-dbaas": `backupCommand: >
   /bin/sh -c  "if [ ! -z $BACKUP_DB_READREPLICA_HOSTS ]; then
   BACKUP_DB_HOST=$(echo $BACKUP_DB_READREPLICA_HOSTS | cut -d ',' -f1);
@@ -325,7 +338,7 @@ pod:
   --dbname=$BACKUP_DB_DATABASE
   --username=$BACKUP_DB_USERNAME
   --format=t -w"
-fileExtension: .{{ .Name }}.tar
+fileExtension: .{{ .Service.Name }}.tar
 pod:
   spec:
     containers:
@@ -336,28 +349,28 @@ pod:
       - name: BACKUP_DB_HOST
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_HOST
+            key: {{ .Service.Name | VarFix }}_HOST
             name: lagoon-env
       - name: BACKUP_DB_USERNAME
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_USERNAME
+            key: {{ .Service.Name | VarFix }}_USERNAME
             name: lagoon-env
       - name: BACKUP_DB_PASSWORD
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_PASSWORD
+            key: {{ .Service.Name | VarFix }}_PASSWORD
             name: lagoon-env
       - name: BACKUP_DB_DATABASE
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_DATABASE
+            key: {{ .Service.Name | VarFix }}_DATABASE
             name: lagoon-env
       image: uselagoon/database-tools:latest
       imagePullPolicy: Always
-      name: {{ .Name }}-prebackuppod`,
-	"mongodb-dbaas": `backupCommand: /bin/sh -c "mongodump --uri=mongodb://${BACKUP_DB_USERNAME}:${BACKUP_DB_PASSWORD}@${BACKUP_DB_HOST}:${BACKUP_DB_PORT}/${BACKUP_DB_DATABASE}?ssl=true&sslInsecure=true&tls=true&tlsInsecure=true --archive"
-fileExtension: .{{ .Name }}.bson
+      name: {{ .Service.Name }}-prebackuppod`,
+	"mongodb-dbaas": `backupCommand: /bin/sh -c "dump=$(mktemp) && mongodump --quiet --ssl --tlsInsecure --username=${BACKUP_DB_USERNAME} --password=${BACKUP_DB_PASSWORD} --host=${BACKUP_DB_HOST}:${BACKUP_DB_PORT} --db=${BACKUP_DB_DATABASE} --authenticationDatabase=${BACKUP_DB_AUTHSOURCE} --authenticationMechanism=${BACKUP_DB_AUTHMECHANISM} --archive=$dump && cat $dump && rm $dump"
+fileExtension: .{{ .Service.Name }}.bson
 pod:
   spec:
     containers:
@@ -368,24 +381,44 @@ pod:
       - name: BACKUP_DB_HOST
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_HOST
+            key: {{ .Service.Name | VarFix }}_HOST
             name: lagoon-env
       - name: BACKUP_DB_USERNAME
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_USERNAME
+            key: {{ .Service.Name | VarFix }}_USERNAME
             name: lagoon-env
       - name: BACKUP_DB_PASSWORD
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_PASSWORD
+            key: {{ .Service.Name | VarFix }}_PASSWORD
             name: lagoon-env
       - name: BACKUP_DB_DATABASE
         valueFrom:
           configMapKeyRef:
-            key: {{ .Name | VarFix }}_DATABASE
+            key: {{ .Service.Name | VarFix }}_DATABASE
+            name: lagoon-env
+      - name: BACKUP_DB_PORT
+        valueFrom:
+          configMapKeyRef:
+            key: {{ .Service.Name | VarFix }}_PORT
+            name: lagoon-env
+      - name: BACKUP_DB_AUTHSOURCE
+        valueFrom:
+          configMapKeyRef:
+            key: {{ .Service.Name | VarFix }}_AUTHSOURCE
+            name: lagoon-env
+      - name: BACKUP_DB_AUTHMECHANISM
+        valueFrom:
+          configMapKeyRef:
+            key: {{ .Service.Name | VarFix }}_AUTHMECHANISM
+            name: lagoon-env
+      - name: BACKUP_DB_AUTHTLS
+        valueFrom:
+          configMapKeyRef:
+            key: {{ .Service.Name | VarFix }}_AUTHTLS
             name: lagoon-env
       image: uselagoon/database-tools:latest
       imagePullPolicy: Always
-      name: {{ .Name }}-prebackuppod`,
+      name: {{ .Service.Name }}-prebackuppod`,
 }
diff --git a/internal/templating/backups/test-resources/result-prebackuppod3.yaml b/internal/templating/backups/test-resources/result-prebackuppod3.yaml
@@ -19,8 +19,10 @@ metadata:
     prebackuppod: mongodb-database
   name: mongodb-database-prebackuppod
 spec:
-  backupCommand: /bin/sh -c "mongodump --uri=mongodb://${BACKUP_DB_USERNAME}:${BACKUP_DB_PASSWORD}@${BACKUP_DB_HOST}:${BACKUP_DB_PORT}/${BACKUP_DB_DATABASE}?ssl=true&sslInsecure=true&tls=true&tlsInsecure=true
-    --archive"
+  backupCommand: /bin/sh -c "dump=$(mktemp) && mongodump --quiet --ssl --tlsInsecure
+    --username=${BACKUP_DB_USERNAME} --password=${BACKUP_DB_PASSWORD} --host=${BACKUP_DB_HOST}:${BACKUP_DB_PORT}
+    --db=${BACKUP_DB_DATABASE} --authenticationDatabase=${BACKUP_DB_AUTHSOURCE} --authenticationMechanism=${BACKUP_DB_AUTHMECHANISM}
+    --archive=$dump && cat $dump && rm $dump"
   fileExtension: .mongodb-database.bson
   pod:
     metadata: {}
@@ -50,6 +52,26 @@ spec:
             configMapKeyRef:
               key: MONGODB_DATABASE_DATABASE
               name: lagoon-env
+        - name: BACKUP_DB_PORT
+          valueFrom:
+            configMapKeyRef:
+              key: MONGODB_DATABASE_PORT
+              name: lagoon-env
+        - name: BACKUP_DB_AUTHSOURCE
+          valueFrom:
+            configMapKeyRef:
+              key: MONGODB_DATABASE_AUTHSOURCE
+              name: lagoon-env
+        - name: BACKUP_DB_AUTHMECHANISM
+          valueFrom:
+            configMapKeyRef:
+              key: MONGODB_DATABASE_AUTHMECHANISM
+              name: lagoon-env
+        - name: BACKUP_DB_AUTHTLS
+          valueFrom:
+            configMapKeyRef:
+              key: MONGODB_DATABASE_AUTHTLS
+              name: lagoon-env
         - name: BACKUP_DB_READREPLICA_HOSTS
           valueFrom:
             configMapKeyRef: