Upgrade CI Helm version and revert unnecessary template changes

The helm lint failures in CI were caused by using outdated Helm 3.3.4 (from 2020).
The template syntax `ne .Values.server.enabled false` works correctly with modern Helm versions.

Changes:
- Upgraded CI Helm version from 3.3.4 to 3.19.0 (latest stable)
- Reverted commit b82a8f2 template changes (not needed with modern Helm)

Testing confirmed:
- All helm lint tests pass with Helm 3.14+ and 3.19.0
- Customer issue (empty dataConfigSources entries) remains fixed
- Original template syntax is compatible with modern Helm

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: EliMoshkovich

.github/workflows/master.yaml

@@ -16,7 +16,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: azure/setup-helm@v1
         with:
-          version: "3.3.4"
+          version: "3.19.0"
       - name: helm lint
         run: |
           jq --version

templates/deployment-client.yaml

@@ -1,5 +1,5 @@
 {{- if .Values.client }}
-{{- if ne (.Values.client.enabled | default true) false }}
+{{- if ne .Values.client.enabled false  }}
 {{- $nm := include "opal.clientName" . | quote }}
 apiVersion: apps/v1
 kind: Deployment

templates/deployment-server.yaml

@@ -1,5 +1,5 @@
 {{- if .Values.server }}
-{{- if ne (.Values.server.enabled | default true) false }}
+{{- if ne .Values.server.enabled false  }}
 {{- $nm := include "opal.serverName" . | quote }}
 apiVersion: apps/v1
 kind: Deployment

templates/service-client.yaml

@@ -1,5 +1,5 @@
 {{- if .Values.client }}
-{{- if ne (.Values.client.enabled | default true) false }}
+{{- if ne .Values.client.enabled false  }}
 apiVersion: v1
 kind: Service
 metadata:

templates/service-server.yaml

@@ -1,5 +1,5 @@
 {{- if .Values.server }}
-{{- if ne (.Values.server.enabled | default true) false }}
+{{- if ne .Values.server.enabled false  }}
 apiVersion: v1
 kind: Service
 metadata: