bump helm version

Author: mainred

src/aks-agent/azext_aks_agent/_consts.py

@@ -44,3 +44,6 @@
 AGENT_NAMESPACE = "kube-system"
 AGENT_LABEL_SELECTOR = "app.kubernetes.io/name=aks-agent"
 AKS_MCP_LABEL_SELECTOR = "app.kubernetes.io/name=aks-mcp"
+
+# Helm Configuration
+HELM_VERSION = "3.16.0"

src/aks-agent/azext_aks_agent/agent/k8s/helm_manager.py

@@ -15,6 +15,7 @@
 from pathlib import Path
 from typing import List, Optional, Tuple
 
+from azext_aks_agent._consts import HELM_VERSION
 from knack.log import get_logger
 
 logger = get_logger(__name__)
@@ -31,13 +32,13 @@ class HelmManager:
     - Share helm functionality across different chart deployments
     """
 
-    def __init__(self, helm_version: str = "3.14.0", local_bin_dir: Optional[str] = None,
+    def __init__(self, helm_version: str = HELM_VERSION, local_bin_dir: Optional[str] = None,
                  kubeconfig_path: Optional[str] = None):
         """
         Initialize the Helm Manager.
 
         Args:
-            helm_version: Helm version to use (default: '3.14.0')
+            helm_version: Helm version to use (default: HELM_VERSION from _consts)
             local_bin_dir: Local directory for helm binary (default: ~/.aks-agent/bin)
             kubeconfig_path: Path to kubeconfig file (default: None - use default config)
         """
@@ -311,13 +312,13 @@ def repo_update(self) -> bool:
         return success
 
 
-def create_helm_manager(helm_version: str = "3.14.0",
+def create_helm_manager(helm_version: str = HELM_VERSION,
                         local_bin_dir: Optional[str] = None) -> HelmManager:
     """
     Factory function to create a HelmManager instance.
 
     Args:
-        helm_version: Helm version to use
+        helm_version: Helm version to use (default: HELM_VERSION from _consts)
         local_bin_dir: Local directory for helm binary
 
     Returns:

src/aks-agent/azext_aks_agent/tests/latest/test_helm_manager.py

@@ -8,13 +8,12 @@
 """
 
 import os
-import platform
-import subprocess
 import tempfile
 import unittest
 from pathlib import Path
-from unittest.mock import MagicMock, Mock, mock_open, patch
+from unittest.mock import Mock, mock_open, patch
 
+from azext_aks_agent._consts import HELM_VERSION
 from azext_aks_agent.agent.k8s.helm_manager import HelmManager
 
 
@@ -23,7 +22,7 @@ class TestHelmManager(unittest.TestCase):
 
     def setUp(self):
         """Set up test fixtures."""
-        self.test_helm_version = "3.14.0"
+        self.test_helm_version = HELM_VERSION
         self.test_kubeconfig = "/mock/kubeconfig"
         self.test_bin_dir = "/mock/bin"
 
@@ -35,7 +34,7 @@ def test_init_default_values(self, mock_ensure_helm, mock_mkdir):
 
         manager = HelmManager()
 
-        self.assertEqual(manager.helm_version, "3.14.0")
+        self.assertEqual(manager.helm_version, "3.16.0")
         self.assertIsNone(manager.kubeconfig_path)
         self.assertIsNotNone(manager.local_bin_dir)
         mock_mkdir.assert_called_once()
@@ -171,15 +170,15 @@ def test_get_version(self, mock_ensure_helm, mock_mkdir, mock_run):
         mock_ensure_helm.return_value = "/mock/helm"
         mock_run.return_value = Mock(
             returncode=0,
-            stdout="version.BuildInfo{Version:\"v3.14.0\"}",
+            stdout="version.BuildInfo{Version:\"v3.16.0\"}",
             stderr=""
         )
 
         manager = HelmManager()
         version = manager.get_version()
 
         self.assertIsNotNone(version)
-        self.assertIn("3.14.0", version)
+        self.assertIn("3.16.0", version)
 
     @patch('azext_aks_agent.agent.k8s.helm_manager.subprocess.run')
     @patch('azext_aks_agent.agent.k8s.helm_manager.Path.mkdir')
@@ -227,7 +226,7 @@ class TestHelmManagerRealDownload(unittest.TestCase):
 
     def setUp(self):
         """Set up test fixtures."""
-        self.test_helm_version = "3.14.0"
+        self.test_helm_version = HELM_VERSION
         self.temp_dir = None
 
     def tearDown(self):
@@ -249,7 +248,7 @@ def test_real_download_all_platforms(self):
             ("darwin", "amd64"),
             ("darwin", "arm64"),
             ("windows", "amd64"),
-            # ("windows", "arm64"), # The blob does not exist for this platform
+            ("windows", "arm64"),
         ]
 
         for os_name, arch in platforms: