From 1452c3f580aefa5b5fb8c171f32f2e6d18e5fffc Mon Sep 17 00:00:00 2001
From: Evan Krall <krall@yelp.com>
Date: Mon, 7 Aug 2023 11:47:30 -0700
Subject: [PATCH] Revert "Add support to use in-cluster service account (#338)"

This reverts commit ea2b8370aba65bf3d11d4cc6a6f854494d72b429.
---
 clusterman/kubernetes/util.py | 13 +------------
 tests/kubernetes/util_test.py | 15 ---------------
 2 files changed, 1 insertion(+), 27 deletions(-)

diff --git a/clusterman/kubernetes/util.py b/clusterman/kubernetes/util.py
index 0e9253a5f..ab3ff183f 100644
--- a/clusterman/kubernetes/util.py
+++ b/clusterman/kubernetes/util.py
@@ -60,18 +60,7 @@ def __init__(self, kubeconfig_path: str, client_class: Type) -> None:
         :param Type client_class: k8s client class to initialize
         """
         try:
-            """
-            https://kubernetes.io/docs/concepts/containers/container-environment/#container-environment
-            Every pod in k8s gets some default environment variable injected, including KUBERNETES_SERVICE_HOST
-            which points to default kuberbetes service. We are using this variable to distinguise between
-            when cluterman is started in a pod vs when it's started on host. For clusterman instances running inside
-            a k8s cluster, we prioritise using K8s Service account since that let us avoid creating any kubeconfig
-            in advance. For clusterman CLI invocation we continue using provided KUBECONFIG file
-            """
-            if os.getenv("KUBERNETES_SERVICE_HOST"):
-                kubernetes.config.load_incluster_config()
-            else:
-                kubernetes.config.load_kube_config(kubeconfig_path, context=os.getenv("KUBECONTEXT"))
+            kubernetes.config.load_kube_config(kubeconfig_path, context=os.getenv("KUBECONTEXT"))
         except TypeError:
             error_msg = "Could not load KUBECONFIG; is this running on Kubernetes master?"
             if "yelpcorp" in socket.getfqdn():
diff --git a/tests/kubernetes/util_test.py b/tests/kubernetes/util_test.py
index 3316a4660..ef4da0799 100644
--- a/tests/kubernetes/util_test.py
+++ b/tests/kubernetes/util_test.py
@@ -26,21 +26,6 @@ def test_cached_corev1_api_no_kubeconfig(caplog):
         assert "Could not load KUBECONFIG" in caplog.text
 
 
-def test_cached_corev1_api_use_load_incluster_config_when_running_in_pod():
-    with mock.patch.dict(os.environ, {"KUBERNETES_SERVICE_HOST": "ABC"}):
-        with mock.patch(
-            "clusterman.kubernetes.util.kubernetes.config.load_incluster_config"
-        ) as mock_load_incluster_config:
-            _ = CachedCoreV1Api("/foo/bar/admin.conf")
-            assert mock_load_incluster_config.called
-
-
-def test_cached_corev1_api_use_load_kubeconfig_config_when_running_as_cli():
-    with mock.patch("clusterman.kubernetes.util.kubernetes.config.load_kube_config") as mock_load_kube_config:
-        _ = CachedCoreV1Api("/foo/bar/admin.conf")
-        assert mock_load_kube_config.called
-
-
 def test_cached_corev1_api_caches_non_cached_function(mock_cached_core_v1_api):
     mock_cached_core_v1_api.list_namespace()
     assert mock_cached_core_v1_api._client.list_namespace.call_count == 1