From e854ac8f9246ef8c551e969317c4c22fb93809df Mon Sep 17 00:00:00 2001
From: ronenk1 <ronenk1@rnd-hub.com>
Date: Sun, 25 Jan 2026 14:57:50 +0200
Subject: [PATCH] feat(global): enhance sub-chart dependency build process and
 update service configuration

---
 .github/workflows/build-and-push-helm.yaml | 26 ++++++++++++++++++++++
 charts/sftpgo-stack/values.yaml            | 20 +++++++++++------
 2 files changed, 39 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/build-and-push-helm.yaml b/.github/workflows/build-and-push-helm.yaml
index d65bdfe..c7fbd2d 100644
--- a/.github/workflows/build-and-push-helm.yaml
+++ b/.github/workflows/build-and-push-helm.yaml
@@ -41,6 +41,32 @@ jobs:
       - name: Set up Helm
         uses: azure/setup-helm@v4
 
+      - name: Build sub-chart dependencies
+        env:
+          CHART_NAME: ${{ steps.chart_metadata.outputs.chart_name }}
+        run: |
+          set -euo pipefail
+          
+          CHART_PATH="./charts/$CHART_NAME"
+          
+          # Find all sub-charts with dependencies and build them
+          if [[ -d "$CHART_PATH/charts" ]]; then
+            echo "Checking for sub-charts with dependencies..."
+            find "$CHART_PATH/charts" -name "Chart.yaml" -type f 2>/dev/null | while read -r subchart; do
+              subchart_dir=$(dirname "$subchart")
+              subchart_name=$(basename "$subchart_dir")
+              
+              if grep -q 'dependencies:' "$subchart"; then
+                echo "Building dependencies for sub-chart: $subchart_name"
+                helm dependency build "$subchart_dir"
+              else
+                echo "No dependencies for sub-chart: $subchart_name"
+              fi
+            done
+          else
+            echo "No sub-charts directory found, skipping sub-chart dependency build"
+          fi
+
       - name: Push Chart to ACR
         uses: appany/helm-oci-chart-releaser@v0.5.0
         with:
diff --git a/charts/sftpgo-stack/values.yaml b/charts/sftpgo-stack/values.yaml
index 2df9e51..046539b 100644
--- a/charts/sftpgo-stack/values.yaml
+++ b/charts/sftpgo-stack/values.yaml
@@ -69,14 +69,20 @@ sftpgo:
           averageUtilization: 80
   
   service:
-    type: LoadBalancer
-    annotations:
-      service.beta.kubernetes.io/azure-load-balancer-internal: 'true'
-      service.beta.kubernetes.io/azure-load-balancer-internal-subnet: master-subnet
-    ports:
-      sftp:
-        port: 2022        
+    type: ClusterIP    
 
+  services: 
+    lb:
+      type: LoadBalancer
+      ports: # Define extra ports if needed for LoadBalancer service
+        sftp: 
+          port: 2022
+        webdav:
+          port: 5005
+
+  webdavd:
+    enabled: true
+    
 sftpgo-node-exporter:
   resources:
     requests: