Fix Alluxio env variable syntax for worker hostname

- Removed invalid ${env.POD_NAME} syntax from alluxio-site.properties
- Alluxio doesn't support ${env.VAR} environment variable substitution in properties
- Changed approach to dynamically append hostname config at runtime
- Worker startup script now appends alluxio.worker.hostname to config file
- Uses POD_NAME, POD_NAMESPACE env vars to construct FQDN at container startup
- Removed alluxio.user.hostname from core-site.xml (not needed for Accumulo clients)

This fixes "Invalid property key env.POD_NAME" error from Alluxio workers.

Co-authored-by: phrocker <[email protected]>

Author: Copilot

charts/accumulo/templates/alluxio-worker-daemonset.yaml

@@ -61,6 +61,19 @@ spec:
             sleep 5
           done
 
+          # Set worker hostname to FQDN for Kubernetes DNS resolution
+          WORKER_FQDN="${POD_NAME}.{{ include "accumulo.fullname" . }}-alluxio-worker.${POD_NAMESPACE}.svc.cluster.local"
+          echo "Setting Alluxio worker hostname to: ${WORKER_FQDN}"
+          
+          # Export as environment variables for Alluxio to pick up
+          export ALLUXIO_WORKER_HOSTNAME="${WORKER_FQDN}"
+          export ALLUXIO_JOB_WORKER_HOSTNAME="${WORKER_FQDN}"
+          
+          # Add to alluxio-site.properties dynamically
+          cat >> /opt/alluxio/conf/alluxio-site.properties <<EOF
+alluxio.worker.hostname=${WORKER_FQDN}
+alluxio.job.worker.hostname=${WORKER_FQDN}
+EOF
 
           # Create directories
           mkdir -p /opt/ramdisk
@@ -119,10 +132,6 @@ spec:
           valueFrom:
             fieldRef:
               fieldPath: metadata.namespace
-        - name: ALLUXIO_WORKER_HOSTNAME
-          value: "$(POD_NAME).{{ include "accumulo.fullname" . }}-alluxio-worker.$(POD_NAMESPACE).svc.cluster.local"
-        - name: ALLUXIO_JOB_WORKER_HOSTNAME
-          value: "$(POD_NAME).{{ include "accumulo.fullname" . }}-alluxio-worker.$(POD_NAMESPACE).svc.cluster.local"
         - name: ALLUXIO_MASTER_HOSTNAME
           value: {{include "accumulo.fullname" .}}-alluxio-master
         {{- if eq .Values.storage.provider "s3" }}

charts/accumulo/templates/configmap.yaml

@@ -289,7 +289,7 @@ data:
     alluxio.master.hostname={{ include "accumulo.fullname" . }}-alluxio-master
     alluxio.master.rpc.port=19998
     alluxio.master.web.port=19999
-    # Worker hostname set via ALLUXIO_WORKER_HOSTNAME environment variable
+    # Worker hostname will be set via ALLUXIO_WORKER_HOSTNAME environment variable in deployment
     alluxio.worker.bind.host=0.0.0.0
     alluxio.worker.port=29999
     alluxio.worker.web.port=30000
@@ -298,19 +298,15 @@ data:
     alluxio.worker.rpc.bind.host=0.0.0.0
     alluxio.worker.data.bind.host=0.0.0.0
     alluxio.worker.web.bind.host=0.0.0.0
-    # Use FQDN for worker hostname resolution in Kubernetes
-    alluxio.worker.hostname=${env.POD_NAME}.{{ include "accumulo.fullname" . }}-alluxio-worker.${env.POD_NAMESPACE}.svc.cluster.local
     
     ## Job worker configuration
     alluxio.job.master.hostname={{ include "accumulo.fullname" . }}-alluxio-master
     alluxio.job.master.rpc.port=20001
-    # Job worker hostname set via ALLUXIO_JOB_WORKER_HOSTNAME environment variable
+    # Job worker hostname will be set via ALLUXIO_JOB_WORKER_HOSTNAME environment variable in deployment
     alluxio.job.worker.bind.host=0.0.0.0
     alluxio.job.worker.rpc.port=30002
     alluxio.job.worker.data.port=30003
     alluxio.job.worker.web.port=30004
-    # Use FQDN for job worker hostname resolution in Kubernetes
-    alluxio.job.worker.hostname=${env.POD_NAME}.{{ include "accumulo.fullname" . }}-alluxio-worker.${env.POD_NAMESPACE}.svc.cluster.local
 
     ## Memory and storage configuration
     alluxio.worker.memory.size={{ .Values.alluxio.properties.alluxio.worker.memory.size }}
@@ -383,11 +379,16 @@ data:
             <name>fs.AbstractFileSystem.alluxio.impl</name>
             <value>alluxio.hadoop.AlluxioFileSystem</value>
         </property>
-        <!-- Alluxio client configuration for Kubernetes DNS resolution -->
+        <!-- Alluxio client configuration -->
         <property>
-            <name>alluxio.user.hostname</name>
-            <value>${env.POD_NAME}.{{ include "accumulo.fullname" . }}-alluxio-worker.${env.POD_NAMESPACE}.svc.cluster.local</value>
-            <description>Client hostname for Alluxio operations</description>
+            <name>alluxio.master.hostname</name>
+            <value>{{ include "accumulo.fullname" . }}-alluxio-master</value>
+            <description>Alluxio master hostname</description>
+        </property>
+        <property>
+            <name>alluxio.master.rpc.port</name>
+            <value>19998</value>
+            <description>Alluxio master RPC port</description>
         </property>
         <property>
             <name>alluxio.user.network.netty.channel</name>