address comments

Author: mainred

src/aks-agent/README.rst

@@ -105,7 +105,7 @@ Run in non-interactive batch mode
 
 .. code-block:: bash
 
-    az aks agent "Diagnose networking issues" --no-interactive --max-steps 15 --model azure/my-gpt4.1-deployment
+    az aks agent "Diagnose networking issues" --no-interactive --name MyManagedCluster --resource-group MyResourceGroup --model azure/my-gpt4.1-deployment
 
 Clean up the AKS agent
 -----------------------

src/aks-agent/azext_aks_agent/_help.py

@@ -56,7 +56,7 @@
         - name: Ask about pod issues in the cluster with last configured model
           text: |-
             az aks agent "Why are my pods not starting?" --resource-group myResourceGroup --name myAKSCluster
-          - name: Check AKS agent deployment status
+        - name: Check AKS agent deployment status
           text: |-
             az aks agent --status --resource-group myResourceGroup --name myAKSCluster
         - name: Ask about pod issues in the cluster with Azure OpenAI

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

@@ -39,7 +39,7 @@ def get_aks_credentials(
     kubeconfig_path = _get_kubeconfig_file_path(resource_group_name, cluster_name)
 
     # Ensure the kubeconfig file exists and write kubeconfig to it
-    with os.fdopen(os.open(kubeconfig_path, os.O_CREAT | os.O_WRONLY, 0o600), 'wt') as f:
+    with os.fdopen(os.open(kubeconfig_path, os.O_RDWR | os.O_CREAT | os.O_TRUNC, 0o600), 'wt') as f:
         f.write(kubeconfig)
 
     logger.info("Kubeconfig downloaded successfully to: %s", kubeconfig_path)
@@ -64,7 +64,7 @@ def _get_kubeconfig_file_path(  # pylint: disable=unused-argument
             if ex.errno != errno.EEXIST:
                 raise
 
-    kubeconfig_filename = f"kubeconfig-{cluster_name}"
+    kubeconfig_filename = f"kubeconfig-{resource_group_name}-{cluster_name}"
     kubeconfig_path = os.path.join(kubeconfig_dir, kubeconfig_filename)
 
     return kubeconfig_path

src/aks-agent/azext_aks_agent/custom.py

@@ -174,7 +174,7 @@ def aks_agent_init(cmd,
 
     except Exception as e:
         console.print(f"❌ Error during initialization: {str(e)}", style=ERROR_COLOR)
-        logger.error("Agent initialization failed: %s", e)
+        raise AzCLIError("Agent initialization failed")
 
 
 def _get_existing_cluster_role(aks_agent_manager):
@@ -431,10 +431,9 @@ def aks_agent_cleanup(
         console.print(
             "❌ Cleanup failed. Please run 'az aks agent --status' to verify cleanup completion.", style=ERROR_COLOR)
 
+
 # pylint: disable=unused-argument
 # pylint: disable=too-many-locals
-
-
 def aks_agent(
     cmd,
     client,