Merge pull request #415 from UffizziCloud/feature/412-use-current-con…

…text-namespace-for-uffizzi-uninstall [#412] use current context namespace for uffizzi uninstall
UffizziCloud · Apr 2, 2024 · 35acaef · 35acaef
2 parents 77ad3d7 + 6415c92
commit 35acaef
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 1 deletion.
diff --git a/lib/uffizzi/cli/uninstall.rb b/lib/uffizzi/cli/uninstall.rb
@@ -86,7 +86,7 @@ def unset_account_installation
     end
 
     def namespace
-      options[:namespace] || InstallService::DEFAULT_NAMESPACE
+      options[:namespace] || InstallService.current_namespace
     end
 
     def server

diff --git a/lib/uffizzi/services/install_service.rb b/lib/uffizzi/services/install_service.rb
@@ -130,6 +130,14 @@ def helm_values_dir_path
       File.dirname(Uffizzi::ConfigFile.config_path)
     end
 
+    def current_namespace
+      path = KubeconfigService.default_path
+      kubeconfig = KubeconfigService.read_kubeconfig(path)
+      return if kubeconfig.nil?
+
+      KubeconfigService.get_current_namespace(kubeconfig) || DEFAULT_NAMESPACE
+    end
+
     private
 
     def execute_command(command, say: true, skip_error: false)

diff --git a/lib/uffizzi/services/kubeconfig_service.rb b/lib/uffizzi/services/kubeconfig_service.rb
@@ -50,6 +50,12 @@ def get_current_cluster_name(kubeconfig)
         .dig('context', 'cluster')
     end
 
+    def get_current_namespace(kubeconfig)
+      get_cluster_contexts(kubeconfig)
+        .detect { |c| c['name'] == get_current_context(kubeconfig) }
+        .dig('context', 'namespace')
+    end
+
     def get_cluster_contexts(kubeconfig)
       kubeconfig.fetch('contexts', [])
     end