From efd9f7ddd5fea49c017e090b07d2e4a91e7860ac Mon Sep 17 00:00:00 2001 From: Bavneet Singh <33008256+bavneetsingh16@users.noreply.github.com> Date: Mon, 19 Aug 2024 18:29:39 -0700 Subject: [PATCH] [connectedk8s] fix connect cmd issue of namespace delete (#7894) Co-authored-by: Bavneet Singh --- src/connectedk8s/HISTORY.rst | 4 ++++ src/connectedk8s/azext_connectedk8s/_precheckutils.py | 2 -- src/connectedk8s/setup.py | 2 +- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/connectedk8s/HISTORY.rst b/src/connectedk8s/HISTORY.rst index 18bbac5c5d0..42f6162f7de 100644 --- a/src/connectedk8s/HISTORY.rst +++ b/src/connectedk8s/HISTORY.rst @@ -3,6 +3,10 @@ Release History =============== +1.9.1 +++++++ +* Fixed the issue where executing a second `connectedk8s connect` command would inadvertently delete the "azure-arc-release" namespace. + 1.9.0 ++++++ * Upgrading to v0.2.2 of ArcK8sDiagnoser diff --git a/src/connectedk8s/azext_connectedk8s/_precheckutils.py b/src/connectedk8s/azext_connectedk8s/_precheckutils.py index 9a3f79fc53a..8ffe215129e 100644 --- a/src/connectedk8s/azext_connectedk8s/_precheckutils.py +++ b/src/connectedk8s/azext_connectedk8s/_precheckutils.py @@ -271,8 +271,6 @@ def executing_cluster_diagnostic_checks_job(corev1_api_instance, batchv1_api_ins # Clearing all the resources after fetching the cluster diagnostic checks container logs Popen(cmd_helm_delete, stdout=PIPE, stderr=PIPE) - delete_release_namespace = [kubectl_client_location, "delete", "namespace", "azure-arc-release"] - Popen(delete_release_namespace, stdout=PIPE, stderr=PIPE) # To handle any exception that may occur during the execution except Exception as e: diff --git a/src/connectedk8s/setup.py b/src/connectedk8s/setup.py index 4432358b1ee..7936ed63211 100644 --- a/src/connectedk8s/setup.py +++ b/src/connectedk8s/setup.py @@ -17,7 +17,7 @@ # TODO: Confirm this is the right version number you want and it matches your # HISTORY.rst entry. -VERSION = '1.9.0' +VERSION = '1.9.1' # The full list of classifiers is available at # https://pypi.python.org/pypi?%3Aaction=list_classifiers