WIP: The prior commit (51133b5) on this PR added the cachePath constant but never updated the three call sites that still used the raw "/cache" literal. I replaced all three: the two corev1.VolumeMount MountPath fields (in buildBackupJob and b #3

Draft
bstein wants to merge 2 commits from hermes-repair/sonar-AZ9pTqVcN0JrBQvDGDs3 into main

View File

@ -14,6 +14,8 @@ import (
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
)
const cachePath = "/cache"
func buildBackupJob(cfg *config.Config, req api.BackupRequest, jobName, secretName, repository string, dedupeEnabled bool, keepLast int) *batchv1.Job {
labels := map[string]string{
labelAppName: "soteria",
@ -42,7 +44,7 @@ func buildBackupJob(cfg *config.Config, req api.BackupRequest, jobName, secretNa
Resources: jobResources(cfg),
VolumeMounts: []corev1.VolumeMount{
{Name: "data", MountPath: "/data", ReadOnly: true},
{Name: "cache", MountPath: "/cache"},
{Name: "cache", MountPath: cachePath},
},
},
},
@ -115,7 +117,7 @@ func buildRestoreJob(cfg *config.Config, req api.RestoreTestRequest, jobName, se
Resources: jobResources(cfg),
VolumeMounts: []corev1.VolumeMount{
{Name: "restore", MountPath: "/restore"},
{Name: "cache", MountPath: "/cache"},
{Name: "cache", MountPath: cachePath},
},
},
},
@ -258,7 +260,7 @@ func resticEnv(cfg *config.Config, secretName, repository string) []corev1.EnvVa
}
env := []corev1.EnvVar{
{Name: "RESTIC_REPOSITORY", Value: repository},
{Name: "RESTIC_CACHE_DIR", Value: "/cache"},
{Name: "RESTIC_CACHE_DIR", Value: cachePath},
{
Name: "AWS_ACCESS_KEY_ID",
ValueFrom: &corev1.EnvVarSource{SecretKeyRef: &corev1.SecretKeySelector{LocalObjectReference: corev1.LocalObjectReference{Name: secretName}, Key: "AWS_ACCESS_KEY_ID"}},