From b3ae332776afa6d5da57ebb7655bffe087fbdc50 Mon Sep 17 00:00:00 2001
From: Kulwant Singh <ikulwant@amazon.com>
Date: Wed, 14 Feb 2024 16:55:56 +0000
Subject: [PATCH] Disable AppSignal on EKS Windows (#76)

# Problem
AppSignal is having issues around initializing k8s client on EKS Windows
when running it as Host process container. Similar issues existed for
CI Windows but workaround was implemented for CI, similar workaround is
required for AppSignal or We can upgrade container to 1.7 which solves this
issue.
---
 helm/templates/cloudwatch-agent-daemonset-windows.yaml |  2 +-
 helm/values.yaml                                       | 10 ++++++++++
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/helm/templates/cloudwatch-agent-daemonset-windows.yaml b/helm/templates/cloudwatch-agent-daemonset-windows.yaml
index 335d5cf53..507a83917 100644
--- a/helm/templates/cloudwatch-agent-daemonset-windows.yaml
+++ b/helm/templates/cloudwatch-agent-daemonset-windows.yaml
@@ -18,7 +18,7 @@ spec:
   {{- if .Values.agent.config }}
   config: {{ .Values.agent.config | toJson | quote }}
   {{- else }}
-  config: {{ .Values.agent.defaultConfig | toJson | quote }}
+  config: {{ .Values.agent.windowsDefaultConfig | toJson | quote }}
   {{- end }}
   resources:
     requests:
diff --git a/helm/values.yaml b/helm/values.yaml
index f22ae7e12..7a689de58 100644
--- a/helm/values.yaml
+++ b/helm/values.yaml
@@ -165,4 +165,14 @@ agent:
           "app_signals": { }
         }
       }
+    }
+  windowsDefaultConfig:
+    {
+      "logs": {
+        "metrics_collected": {
+          "kubernetes": {
+            "enhanced_container_insights": true
+          },
+        }
+      }
     }
\ No newline at end of file