From 2d51fed0027916f59d865faf87bc7ab7b29d109a Mon Sep 17 00:00:00 2001
From: Glen Messenger <gmessenger@google.com>
Date: Tue, 29 Sep 2026 17:16:56 -0700
Subject: [PATCH] helm: bump litellm-helm to 1.1.4; unit-test the enabled
 k8s-aibom subchart

Addresses review: the parent chart version increments so the changed
chart publishes and selects reliably, and the enabled configuration is
exercised by helm unittest (controller Deployment, ClusterRole, and the
default AIBOMControllerConfig render when k8s-aibom.enabled=true). The
default-off path cannot be asserted as zero documents by helm-unittest
(disabled dependencies are not loaded) and is covered by every other
suite rendering with default values.

Signed-off-by: Glen Messenger <gmessenger@google.com>
---
 helm/litellm-helm/Chart.yaml                |  2 +-
 helm/litellm-helm/tests/k8s_aibom_test.yaml | 37 +++++++++++++++++++++
 2 files changed, 38 insertions(+), 1 deletion(-)
 create mode 100644 helm/litellm-helm/tests/k8s_aibom_test.yaml

diff --git a/helm/litellm-helm/Chart.yaml b/helm/litellm-helm/Chart.yaml
index 3421e8f619a..6145fcb52d4 100644
--- a/helm/litellm-helm/Chart.yaml
+++ b/helm/litellm-helm/Chart.yaml
@@ -18,7 +18,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 1.1.3
+version: 1.1.4
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to
diff --git a/helm/litellm-helm/tests/k8s_aibom_test.yaml b/helm/litellm-helm/tests/k8s_aibom_test.yaml
new file mode 100644
index 00000000000..770a8a41ec0
--- /dev/null
+++ b/helm/litellm-helm/tests/k8s_aibom_test.yaml
@@ -0,0 +1,37 @@
+# The k8s-aibom subchart is condition-gated (k8s-aibom.enabled, default
+# false). helm-unittest does not load a disabled dependency's templates at
+# all, so the default-off path cannot be asserted here as "zero documents";
+# it is covered by every other suite rendering with default values. These
+# tests exercise the enabled configuration end to end.
+suite: k8s-aibom optional subchart (enabled)
+templates:
+  - charts/k8s-aibom/templates/deployment.yaml
+  - charts/k8s-aibom/templates/clusterrole.yaml
+  - charts/k8s-aibom/templates/controllerconfig.yaml
+set:
+  k8s-aibom.enabled: true
+tests:
+  - it: renders the k8s-aibom controller Deployment when enabled
+    template: charts/k8s-aibom/templates/deployment.yaml
+    asserts:
+      - hasDocuments:
+          count: 1
+      - isKind:
+          of: Deployment
+      - equal:
+          path: spec.template.spec.containers[0].name
+          value: manager
+  - it: renders the k8s-aibom ClusterRole when enabled
+    template: charts/k8s-aibom/templates/clusterrole.yaml
+    asserts:
+      - hasDocuments:
+          count: 1
+      - isKind:
+          of: ClusterRole
+  - it: renders the default AIBOMControllerConfig when enabled
+    template: charts/k8s-aibom/templates/controllerconfig.yaml
+    asserts:
+      - hasDocuments:
+          count: 1
+      - isKind:
+          of: AIBOMControllerConfig