Don't make project and zone positional args

Hopefully this will prevent the filestore_names list picking up things it shouldn't

Author: sgibson91

gcpFilestoreBackups/templates/deployment.yaml

@@ -34,13 +34,13 @@ spec:
             {{- range .Values.filestoreNames | required "filestoreNames is required" }}
             - '{{ . }}'
             {{- end }}
-            - '{{ .Values.project | required "project is required" }}'
-            - '{{ .Values.zone | required "zone is required" }}'
+            - --project {{ .Values.project | required "project is required" }}
+            - --zone {{ .Values.zone | required "zone is required" }}
             {{- if .Values.retentionDays }}
-            - --retention-days '{{ .Values.retentionDays }}'
+            - --retention-days {{ .Values.retentionDays }}
             {{- end }}
             {{- if .Values.backupFreqDays }}
-            - --backup-freq-days '{{ .Values.backupFreqDays }}'
+            - --backup-freq-days {{ .Values.backupFreqDays }}
             {{- end }}
           securityContext:
             runAsUser: 1000

images/gcp-filestore-backups/gcp-filestore-backups.py

@@ -236,12 +236,12 @@ def main(args):
         help="The name of one or more GCP Filestores to backup",
     )
     parser.add_argument(
-        "project",
+        "--project",
         type=str,
         help="The GCP project the Filestore belongs to",
     )
     parser.add_argument(
-        "zone",
+        "--zone",
         type=str,
         help="The GCP zone the Filestore is deployed in, e.g. us-central1-b",
     )