feat(kubernetes): enhance configuration and health checks

- Added support for environment variables prefixed with "ES_".
- Updated ReflectorOptions configuration to use nameof for better maintainability.
- Changed the container port from 25080 to 8080 in deployment.yaml.
- Modified liveness and readiness probes to point to new health check paths.
- Introduced a new Kubernetes setting for skipping TLS verification.

BREAKING CHANGE: Health check endpoints have been updated.

Author: winromulus

src/ES.Kubernetes.Reflector/Program.cs

@@ -12,13 +12,16 @@
 {
     var builder = WebApplication.CreateBuilder(args);
     builder.Logging.ClearProviders();
+
+    builder.Configuration.AddEnvironmentVariables("ES_");
+
     builder.Ignite();
     builder.IgniteSerilog();
     builder.IgniteSeqOpenTelemetryExporter();
     builder.Services.AddMediatR(config =>
         config.RegisterServicesFromAssembly(typeof(Program).Assembly));
 
-    builder.Services.Configure<ReflectorOptions>(builder.Configuration.GetSection("Reflector"));
+    builder.Services.Configure<ReflectorOptions>(builder.Configuration.GetSection(nameof(ES.Kubernetes.Reflector)));
 
     builder.Services.AddTransient(s =>
     {
@@ -45,7 +48,6 @@
 
     var app = builder.Build();
     app.Ignite();
-
     await app.RunAsync();
     return 0;
 });

src/ES.Kubernetes.Reflector/Properties/launchSettings.json

@@ -4,7 +4,7 @@
       "commandName": "Project",
       "launchBrowser": false,
       "environmentVariables": {
-        "ES_ENVIRONMENT": "Development"
+        "ASPNETCORE_ENVIRONMENT": "Development"
       },
       "applicationUrl": "http://0.0.0.0:8080",
       "dotnetRunMessages": false

src/helm/reflector/templates/cron.yaml

@@ -68,6 +68,11 @@ spec:
                   value: {{ .Values.configuration.logging.minimumLevel | quote }}
                 - name: ES_Reflector__Watcher__Timeout
                   value: {{ .Values.configuration.watcher.timeout | quote }}
+                - name: ES_Reflector__Kubernetes__SkipTlsVerify
+                  value: {{ .Values.configuration.kubernetes.skipTlsVerify | quote }}
+                {{- with .Values.extraEnv }}
+                  {{- toYaml . | nindent 12 }}
+                {{- end }}
               resources:
                 {{- toYaml .Values.resources | nindent 16 }}
 {{- end }}

src/helm/reflector/templates/deployment.yaml

@@ -57,21 +57,15 @@ spec:
 
           ports:
             - name: http
-              containerPort: 25080
+              containerPort: 8080
               protocol: TCP
           livenessProbe:
-            {{- toYaml .Values.healthcheck | nindent 12 }}
-            initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds }}
-            periodSeconds: {{ .Values.livenessProbe.periodSeconds }}
+            {{- toYaml .Values.livenessProbe | nindent 12 }}
           readinessProbe:
-            {{- toYaml .Values.healthcheck | nindent 12 }}
-            initialDelaySeconds: {{ .Values.readinessProbe.initialDelaySeconds }}
-            periodSeconds: {{ .Values.readinessProbe.periodSeconds }}
+            {{- toYaml .Values.readinessProbe | nindent 12 }}
           {{- if semverCompare ">= 1.18-0" .Capabilities.KubeVersion.Version }}
           startupProbe:
-            {{- toYaml .Values.healthcheck | nindent 12 }}
-            failureThreshold: {{ .Values.startupProbe.failureThreshold }}
-            periodSeconds: {{ .Values.startupProbe.periodSeconds }}
+            {{- toYaml .Values.startupProbe | nindent 12 }}
           {{- end }}
           resources:
             {{- toYaml .Values.resources | nindent 12 }}

src/helm/reflector/values.yaml

@@ -61,21 +61,31 @@ securityContext:
   runAsNonRoot: true
   runAsUser: 1000
 
-healthcheck:
+livenessProbe:
   httpGet:
-    path: /healthz
+    path: /health/live
     port: http
-
-livenessProbe:
+  timeoutSeconds: 10
   initialDelaySeconds: 5
   periodSeconds: 10
+  failureThreshold: 5
 readinessProbe:
+  httpGet:
+    path: /health/ready
+    port: http
+  timeoutSeconds: 10
   initialDelaySeconds: 5
   periodSeconds: 10
+  failureThreshold: 5
 startupProbe:
-  # The application will have a maximum of 50s (10 * 5 = 50s) to finish its startup.
-  failureThreshold: 10
-  periodSeconds: 5
+  httpGet:
+    path: /health/ready
+    port: http
+  timeoutSeconds: 10
+  initialDelaySeconds: 5
+  periodSeconds: 10
+  failureThreshold: 5
+
 
 resources:
   {}