syncthing/enterprise - sync app from latest test (#2403)

* syncthing/enterprise - sync app from latest test * fix ci
2026-04-14 10:40:31 +08:00 · 2024-04-25 15:45:56 +03:00
parent d122099fdc
commit ec2fc382be
9 changed files with 268 additions and 117 deletions
--- a/library/ix-dev/enterprise/syncthing/Chart.yaml
+++ b/library/ix-dev/enterprise/syncthing/Chart.yaml
@@ -3,9 +3,9 @@ description: Syncthing is a continuous file synchronization program.
 annotations:
  title: Syncthing
 type: application
-version: 1.1.8
+version: 1.2.3
 apiVersion: v2
-appVersion: '1.27.0'
+appVersion: '1.27.6'
 kubeVersion: '>=1.16.0-0'
 maintainers:
  - name: truenas
--- a/library/ix-dev/enterprise/syncthing/ci/basic-no-hostnet-values.yaml
+++ b/library/ix-dev/enterprise/syncthing/ci/basic-no-hostnet-values.yaml
@@ -1,16 +1,12 @@
 syncthingStorage:
  home:
-    type: hostPath
-    hostPath: /mnt/{{ .Release.Namespace }}/home
+    type: pvc
  additionalStorages:
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir1
+    - type: pvc
      mountPath: /mnt/dir1
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir2
+    - type: pvc
      mountPath: /mnt/dir2
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir3
+    - type: pvc
      mountPath: /mnt/dir3

 syncthingID:
--- a/library/ix-dev/enterprise/syncthing/ci/basic-values.yaml
+++ b/library/ix-dev/enterprise/syncthing/ci/basic-values.yaml
@@ -1,16 +1,12 @@
 syncthingStorage:
  home:
-    type: hostPath
-    hostPath: /mnt/{{ .Release.Namespace }}/home
+    type: pvc
  additionalStorages:
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir1
+    - type: pvc
      mountPath: /mnt/dir1
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir2
+    - type: pvc
      mountPath: /mnt/dir2
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir3
+    - type: pvc
      mountPath: /mnt/dir3

 syncthingID:
--- a/library/ix-dev/enterprise/syncthing/ci/https-no-hostnet-values.yaml
+++ b/library/ix-dev/enterprise/syncthing/ci/https-no-hostnet-values.yaml
@@ -1,16 +1,12 @@
 syncthingStorage:
  home:
-    type: hostPath
-    hostPath: /mnt/{{ .Release.Namespace }}/home
+    type: pvc
  additionalStorages:
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir1
+    - type: pvc
      mountPath: /mnt/dir1
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir2
+    - type: pvc
      mountPath: /mnt/dir2
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir3
+    - type: pvc
      mountPath: /mnt/dir3

 syncthingID:
--- a/library/ix-dev/enterprise/syncthing/ci/https-values.yaml
+++ b/library/ix-dev/enterprise/syncthing/ci/https-values.yaml
@@ -1,16 +1,12 @@
 syncthingStorage:
  home:
-    type: hostPath
-    hostPath: /mnt/{{ .Release.Namespace }}/home
+    type: pvc
  additionalStorages:
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir1
+    - type: pvc
      mountPath: /mnt/dir1
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir2
+    - type: pvc
      mountPath: /mnt/dir2
-    - type: hostPath
-      hostPath: /mnt/{{ .Release.Namespace }}/dir3
+    - type: pvc
      mountPath: /mnt/dir3

 syncthingID:
--- a/library/ix-dev/enterprise/syncthing/migrations/migrate
+++ b/library/ix-dev/enterprise/syncthing/migrations/migrate
@@ -0,0 +1,73 @@
+#!/usr/bin/python3
+import json
+import os
+import sys
+
+
+def storage_migrate(storage):
+    delete_keys = []
+    if storage['type'] == 'hostPath':
+        # Check if the key exists, if not we have already migrated
+        if not storage.get('hostPath'):
+            return storage
+
+        storage['hostPathConfig'] = {'hostPath': storage['hostPath']}
+        delete_keys.append('hostPath')
+
+    elif storage['type'] == 'ixVolume':
+        # Check if the key exists, if not we have already migrated
+        if not storage.get('datasetName'):
+            return storage
+
+        storage['ixVolumeConfig'] = {'datasetName': storage['datasetName']}
+        delete_keys.append('datasetName')
+
+    elif storage['type'] == 'smb-pv-pvc':
+        # Check if the key exists, if not we have already migrated
+        if not storage.get('server'):
+            return storage
+
+        storage['smbConfig'] = {
+            'server': storage['server'],
+            'share': storage['share'],
+            'domain': storage['domain'],
+            'username': storage['username'],
+            'password': storage['password'],
+            'size': storage['size'],
+        }
+        delete_keys.extend(['server', 'share', 'domain', 'username', 'password', 'size'])
+
+    for key in delete_keys:
+        storage.pop(key, None)
+
+    return storage
+
+
+def migrate(values):
+    storage_key = 'syncthingStorage'
+    storages = ['home']
+
+    for storage in storages:
+        check_val = values.get(storage_key, {}).get(storage, {})
+        if not isinstance(check_val, dict) or not check_val:
+            raise Exception(f'Storage section {storage} is malformed')
+
+        values[storage_key][storage] = storage_migrate(check_val)
+
+    additionalStorages = values.get(storage_key, {}).get('additionalStorages', [])
+    for idx, storage in enumerate(additionalStorages):
+        if not isinstance(storage, dict) or not storage:
+            raise Exception(f'Item {idx} in additionalStorages is malformed')
+
+        values[storage_key]['additionalStorages'][idx] = storage_migrate(storage)
+
+    return values
+
+
+if __name__ == '__main__':
+    if len(sys.argv) != 2:
+        exit(1)
+
+    if os.path.exists(sys.argv[1]):
+        with open(sys.argv[1], 'r') as f:
+            print(json.dumps(migrate(json.loads(f.read()))))
--- a/library/ix-dev/enterprise/syncthing/questions.yaml
+++ b/library/ix-dev/enterprise/syncthing/questions.yaml
@@ -206,24 +206,66 @@ questions:
                      description: Host Path (Path that already exists on the system)
                    - value: "ixVolume"
                      description: ixVolume (Dataset created automatically by the system)
-              - variable: datasetName
-                label: Dataset Name
+              - variable: ixVolumeConfig
+                label: ixVolume Configuration
+                description: The configuration for the ixVolume dataset.
                schema:
-                  type: string
+                  type: dict
                  show_if: [["type", "=", "ixVolume"]]
-                  required: true
-                  hidden: true
-                  immutable: true
-                  default: "home"
                  $ref:
                    - "normalize/ixVolume"
-              - variable: hostPath
-                label: Host Path
+                  attrs:
+                    - variable: aclEnable
+                      label: Enable ACL
+                      description: Enable ACL for the dataset.
+                      schema:
+                        type: boolean
+                        hidden: true
+                        default: false
+                    - variable: datasetName
+                      label: Dataset Name
+                      description: The name of the dataset to use for storage.
+                      schema:
+                        type: string
+                        required: true
+                        immutable: true
+                        hidden: true
+                        default: "home"
+                    - variable: aclEntries
+                      label: ACL Configuration
+                      schema:
+                        type: dict
+                        show_if: [["aclEnable", "=", true]]
+                        attrs: []
+              - variable: hostPathConfig
+                label: Host Path Configuration
                schema:
-                  type: hostpath
+                  type: dict
                  show_if: [["type", "=", "hostPath"]]
-                  immutable: true
-                  required: true
+                  attrs:
+                    - variable: aclEnable
+                      label: Enable ACL
+                      description: Enable ACL for the dataset.
+                      schema:
+                        type: boolean
+                        hidden: true
+                        default: false
+                    - variable: acl
+                      label: ACL Configuration
+                      schema:
+                        type: dict
+                        show_if: [["aclEnable", "=", true]]
+                        attrs: []
+                        $ref:
+                          - "normalize/acl"
+                    - variable: hostPath
+                      label: Host Path
+                      description: The host path to use for storage.
+                      schema:
+                        type: hostpath
+                        show_if: [["aclEnable", "=", false]]
+                        required: true
+
        - variable: additionalStorages
          label: Additional Storage
          description: Additional storage for Syncthing.
@@ -245,7 +287,7 @@ questions:
                      label: Type
                      description: |
                        Host Path: Is a path that already exists on the system.</br>
-                        SMB Share: Mounts a persistent volume claim to a SMB share.
+                        SMB Share: Is a SMB share that is mounted to a persistent volume claim.
                      schema:
                        type: string
                        required: true
@@ -256,63 +298,110 @@ questions:
                            description: Host Path (Path that already exists on the system)
                          - value: "smb-pv-pvc"
                            description: SMB Share (Mounts a persistent volume claim to a SMB share)
+                    - variable: readOnly
+                      label: Read Only
+                      description: Mount the volume as read only.
+                      schema:
+                        type: boolean
+                        hidden: true
+                        default: false
                    - variable: mountPath
                      label: Mount Path
                      description: The path inside the container to mount the storage.
                      schema:
                        type: path
                        required: true
-                    - variable: hostPath
-                      label: Host Path
-                      description: The host path to use for storage.
+                    - variable: hostPathConfig
+                      label: Host Path Configuration
                      schema:
-                        type: hostpath
+                        type: dict
                        show_if: [["type", "=", "hostPath"]]
-                        required: true
-                    - variable: server
-                      label: Server
-                      description: The server for the SMB share.
+                        attrs:
+                          - variable: aclEnable
+                            label: Enable ACL
+                            description: Enable ACL for the dataset.
+                            schema:
+                              type: boolean
+                              hidden: true
+                              default: false
+                          - variable: acl
+                            label: ACL Configuration
+                            schema:
+                              type: dict
+                              show_if: [["aclEnable", "=", true]]
+                              attrs: []
+                              $ref:
+                                - "normalize/acl"
+                          - variable: hostPath
+                            label: Host Path
+                            description: The host path to use for storage.
+                            schema:
+                              type: hostpath
+                              show_if: [["aclEnable", "=", false]]
+                              required: true
+                    - variable: smbConfig
+                      label: SMB Share Configuration
+                      description: The configuration for the SMB Share.
                      schema:
-                        type: string
+                        type: dict
                        show_if: [["type", "=", "smb-pv-pvc"]]
-                        required: true
-                    - variable: share
-                      label: Share
-                      description: The share name for the SMB share.
-                      schema:
-                        type: string
-                        show_if: [["type", "=", "smb-pv-pvc"]]
-                        required: true
-                    - variable: domain
-                      label: Domain (Optional)
-                      description: The domain for the SMB share.
-                      schema:
-                        type: string
-                        show_if: [["type", "=", "smb-pv-pvc"]]
-                    - variable: username
-                      label: Username
-                      description: The username for the SMB share.
-                      schema:
-                        type: string
-                        show_if: [["type", "=", "smb-pv-pvc"]]
-                        required: true
-                    - variable: password
-                      label: Password
-                      description: The password for the SMB share.
-                      schema:
-                        type: string
-                        show_if: [["type", "=", "smb-pv-pvc"]]
-                        required: true
-                        private: true
-                    - variable: size
-                      label: Size (in Gi)
-                      description: The size of the volume quota.
-                      schema:
-                        type: int
-                        show_if: [["type", "=", "smb-pv-pvc"]]
-                        required: true
-                        min: 1
-                        default: 1
+                        attrs:
+                          - variable: migrationMode
+                            label: Migration Mode
+                            description: |
+                              NOTE: This only works properly with TrueNAS SCALE 24.04.0 or newer.<br/>
+
+                              Enabling this will force the following:</br>
+                              - Read Only Mount, regardless of the value of the Read Only checkbox.</br>
+                              - SMB Mount Options will be set to "vers=3.0", "cifsacl", "noperm"</br>
+
+                              This option is used to migrate data from third party</br>
+                              NAS platforms onto TrueNAS SCALE.</br>
+
+                              Keep in mind that the ACL preservation is not guaranteed when:</br>
+                              - Non-AD environment</br>
+                              - ACL or remote server contains local users</br>
+                            schema:
+                              type: boolean
+                              default: false
+                          - variable: server
+                            label: Server
+                            description: The server for the SMB share.
+                            schema:
+                              type: string
+                              required: true
+                          - variable: share
+                            label: Share
+                            description: The share name for the SMB share.
+                            schema:
+                              type: string
+                              required: true
+                          - variable: domain
+                            label: Domain (Optional)
+                            description: The domain for the SMB share.
+                            schema:
+                              type: string
+                          - variable: username
+                            label: Username
+                            description: The username for the SMB share.
+                            schema:
+                              type: string
+                              required: true
+                          - variable: password
+                            label: Password
+                            description: The password for the SMB share.
+                            schema:
+                              type: string
+                              required: true
+                              private: true
+                          - variable: size
+                            label: Size (in Gi)
+                            description: The size of the volume quota.
+                            schema:
+                              type: int
+                              required: true
+                              min: 1
+                              default: 1

  - variable: resources
    group: Resources Configuration
@@ -345,7 +434,7 @@ questions:
                schema:
                  type: string
                  max_length: 12
-                  valid_chars: '^[1-9][0-9]*([EPTGMK]i?|e[0-9]+)?$'
+                  valid_chars: "^[1-9][0-9]*([EPTGMK]i?|e[0-9]+)?$"
                  valid_chars_error: |
                    Valid Memory limit formats are</br>
                    - Suffixed with E/P/T/G/M/K - eg. 1G</br>
--- a/library/ix-dev/enterprise/syncthing/templates/_persistence.tpl
+++ b/library/ix-dev/enterprise/syncthing/templates/_persistence.tpl
@@ -2,9 +2,8 @@
 persistence:
  home:
    enabled: true
-    type: {{ .Values.syncthingStorage.home.type }}
-    datasetName: {{ .Values.syncthingStorage.home.datasetName | default "" }}
-    hostPath: {{ .Values.syncthingStorage.home.hostPath | default "" }}
+    {{- include "syncthing.storage.ci.migration" (dict "storage" .Values.syncthingStorage.home) }}
+    {{- include "ix.v1.common.app.storageOptions" (dict "storage" .Values.syncthingStorage.home) | nindent 4 }}
    targetSelector:
      syncthing:
        syncthing:
@@ -37,25 +36,20 @@ persistence:
  {{- end -}}

  {{- range $idx, $storage := .Values.syncthingStorage.additionalStorages }}
+  {{- if eq $storage.type "smb-pv-pvc" -}}
+    {{- if $storage.smbConfig.migrationMode -}}
+      {{- $_ := set $storage "readOnly" true -}}
+      {{- $_ := set $storage.smbConfig "mountOptions" (list
+        (dict "key" "noperm")
+        (dict "key" "cifsacl")
+        (dict "key" "vers" "value" "3.0")
+      ) -}}
+    {{- end -}}
+  {{- end }}
  {{ printf "sync-%v" (int $idx) }}:
-    {{- $size := "" -}}
-    {{- if $storage.size -}}
-      {{- $size = (printf "%vGi" $storage.size) -}}
-    {{- end }}
    enabled: true
-    type: {{ $storage.type }}
-    datasetName: {{ $storage.datasetName | default "" }}
-    hostPath: {{ $storage.hostPath | default "" }}
-    server: {{ $storage.server | default "" }}
-    share: {{ $storage.share | default "" }}
-    domain: {{ $storage.domain | default "" }}
-    username: {{ $storage.username | default "" }}
-    password: {{ $storage.password | default "" }}
-    size: {{ $size }}
-    {{- if eq $storage.type "smb-pv-pvc" }}
-    mountOptions:
-      - key: noperm
-    {{- end }}
+    {{- include "syncthing.storage.ci.migration" (dict "storage" $storage) }}
+    {{- include "ix.v1.common.app.storageOptions" (dict "storage" $storage) | nindent 4 }}
    targetSelector:
      syncthing:
        syncthing:
@@ -85,3 +79,13 @@ scaleCertificate:
    id: {{ .Values.syncthingNetwork.certificateID }}
    {{- end -}}
 {{- end -}}
+
+{{/* TODO: CI only migration, remove on next version bump*/}}
+{{- define "syncthing.storage.ci.migration" -}}
+  {{- $storage := .storage -}}
+
+  {{- if $storage.hostPath -}}
+    {{- $_ := set $storage "hostPathConfig" dict -}}
+    {{- $_ := set $storage.hostPathConfig "hostPath" $storage.hostPath -}}
+  {{- end -}}
+{{- end -}}
--- a/library/ix-dev/enterprise/syncthing/values.yaml
+++ b/library/ix-dev/enterprise/syncthing/values.yaml
@@ -1,6 +1,6 @@
 image:
  repository: syncthing/syncthing
-  tag: '1.27.0'
+  tag: '1.27.6'
  pullPolicy: IfNotPresent

 resources:
@@ -35,5 +35,6 @@ syncthingNetwork:
 syncthingStorage:
  home:
    type: ixVolume
-    datasetName: home
+    ixVolumeConfig:
+      datasetName: home
  additionalStorages: []